zofe / rapyd-laravel

deprecated rewritten in rapyd-livewire
MIT License
866 stars 297 forks source link

PHP 7.3 breaks composer with continue/targeting error #458

Closed butchewing closed 4 years ago

butchewing commented 5 years ago

PHP 7.3 Composer 1.8.5

I'm getting the following error:

ErrorException in Field.php line 579:
"continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"?
in Field.php line 579
at HandleExceptions->handleError('2', '"continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"?', '/home/application/public_html/vendor/zofe/rapyd/src/DataForm/Field/Field.php', '579', array('file' => '/home/application/public_html/vendor/composer/../zofe/rapyd/src/DataForm/Field/Field.php')) in ClassLoader.php line 444

https://www.php.net/manual/en/migration73.incompatible.php

As a quick and dirty fix, I commented out the if ($data=='') statement in lines 578-580 on vendor/zofe/rapyd/src/DataForm/Field/Field.php.