Flynax is not ready for PHP 7.1

[Bahram Soltanirad]

Hello flynax team and all,

As i got some errors in PHP 7.1 which was not in PHP <=7.0, i was encouraged to find the problem.

The error is something like this:

Fatal error: Uncaught ArgumentCountError: Too few arguments to function ...

I got this error at some programs like:

rlPayment.class.php:

Only registered members can view the code.

when it does not pass enough parameters to upgradeBanner function in rlBanners.class.php

OR in flydroid apk, add listing module, it unables to load screen. I guess the problem is the same like above.
OR in Multifield plugin admin panel:

multi_formats.inc.php:

Only registered members can view the code.

and

Only registered members can view the code.

It passes few parameters to the corresponding functions from rlMultifield.class.php

Or maybe in some other places that i don't know.

But, what the problem is. According PHP formal document

previously, a warning would be emitted for invoking user-defined functions with too few arguments. Now, this warning has been promoted to an Error exception.

Example:

Only registered members can view the code. 

Output in PHP 7.1:

Fatal error: Uncaught ArgumentCountError: Too few arguments to function test(), 0 passed in %s on line %d and exactly 1 expected in %s:%d
But, solution is one of belows:

1. Using default value for missed parametrs in target function. like

Only registered members can view the code.

2. Pass all parametrs to the target function. like

Only registered members can view the code.

3. Using

Only registered members can view the code.

method, like

Only registered members can view the code.

But, since it is an error message (Fatal error) you probably needs to use Throwable instead of Exception like this:

Only registered members can view the code.

So, it seems flynax is not ready for PHP 7.1 yet.

[Rudi]

Hello Bahram,

We are aware of it, but nevertheless, thanks for the reminder,

Actually, we didn't reccommend our customers to use php 7.1 or higher cause the script is only adapted to 7.0

But we'll try to solve it soon and let you know

Thanks

[Toofan Karvanian]

Hello,

@Rudi
What about now (v4.6.2), is it fully support PHP 7.1 or not yet? And what about plugins?

[Rudi]

Hello,

The script now supports php 7.2 but some plugins not yet,
for example, multi-field plugin, some modifications are required to adjust it to 7.2

create tickets if you have problems with plugins which don't work with php 7.1-7.2

[Toofan Karvanian]

multi-field plugin is one of the required plugins!!

So to avoid all kind of issues, we need to stick with a PHP 5.6 environment? even if PHP 5.x is not supported anymore except security fixes (end of life is 3 Dec 2018).

I'm just looking for the best environment specs in details so we can avoid issues related to the environment and focus on the app.

[Viktor]

You can run 7.X version of php on your server and check the system. When you find any problems you can create ticket and we will solve it asap.

[Toofan Karvanian]

OK, thank you for the reply.
if said you will solve it ASAP, OK, but I wish it will be a reasonable SLA.

[Pasquale De leonardis]

when will be the plugin "Multifield" updated for PHP7.2? why get in admin this error:

Fatal error: Uncaught ArgumentCountError: Too few arguments to function rlMultiField::getLevel(), 1 passed in doc/plugins/multiField/admin/multi_formats.inc.php on line 446 and exactly 2 expected in doc/plugins/multiField/rlMultiField.class.php:1268 Stack trace: #0 doc/plugins/multiField/admin/multi_formats.inc.php(446): rlMultiField->getLevel(NULL) #1 /index.php(216): require_once('/var/www/strafb...') #2 {main} thrown in doc/plugins/multiField/rlMultiField.class.php on line 1268

[Rudi]

Could you create a ticket so that we would fix it adjusting the plugin to php 7.2

the plugin update will be released a bit later

[Pasquale De leonardis]

done, thank you