search

Ecoxl / celero

Updating Celero
4 stars 1 forks source link

Mouseclick on "Import UPS values" results to "A PHP Error was encountered" #89

Open Ecoxl opened 2 years ago

Ecoxl commented 2 years ago

Mouseclick on "Import UPS values" results to "A PHP Error was encountered"
ImportingUBP values is an essential function for the case study the students are working on. image

tcgumus commented 2 years ago

Ok I got it. PHP 7.4+ has deprecated curly brace syntax for accessing array elements and string offsets.

$array = [1, 2]; // works
$array = {1, 2}; // Parse error: syntax error, unexpected '{'

That's why we are getting errors in pages. So, we need a lot of change in our codebase.

There are some other people that faced the same problem and codeigniter team responded with saying new versions will support it. source

Actually I support upgrading the PHP version because it can cause security and performance issues in the future if we don't update our systems. To use PHP 7.4+ we need to upgrade our codebase to codeigniter 4. (CI4 is built for PHP7.3+)

We can use this document to do this big job. I read it and can summarize it in our next meeting.

Best regards, Tuna