Error thrown in with usage of string in closure about limit-orders HOT 7 CLOSED

Lemme guess, PHP 7.1 or older 🤦

from limit-orders.

That is correct. Seeing this issue on at least 7.3 and 7.4.

from limit-orders.

Actually, seeing this in PHP 7.0 as well.

from limit-orders.

Hmm, perhaps this isn't as simple as I had thought; PHP added typehint support for the string scalar type back in PHP 7.0.0, so this shouldn't be a matter of PHP interpreting the "string" typehint as a class name.

Our autoloader also follows the FIG example of a PSR-4 autoloader pretty closely (though they forego typehinting). I'll rework the autoloader to explicitly cast to string rather than typehint, but could you please post a list of other plugins running in case there's a conflict with another autoloader?

from limit-orders.

I have only been able to replicate this when trying to install a plugin through the Nexcess Dashboard Installer.

from limit-orders.

Once 1.2.0 is tagged, let's make sure to retest this; everything in the codebase should be PHP 7.0+ compliant, so this may be a problem in the dashboard installer.

from limit-orders.

Closing this, as we were able to sort out the issue (resolved in version 1.2.1 of the Nexcess Installer plugin).

from limit-orders.