-
Notifications
You must be signed in to change notification settings - Fork 7.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
T_INLINE_HTML changes between minor versions from 267 to 318 and back causing laravel blades to fail #18067
Comments
We do not do any guarantees on the values of the constants. The values don't change intentionally either, but are a side-effect of the parser generation. This is not a bug, but something which must not be relied upon. For purposes of storing tokens, we recommend the string representation as token names are unlikely to change - at least not without note in UPGRADING. |
@bwoebi thank you for explaining. |
NOTE. Laravel uses string representations like T_INLINE_HTML but these get changed during the php lifecycle as explained in that comment . |
@bwoebi I do have one follow-up question which i can't wrap my finger around. Possibly you or someone else from the PHP team can speed things up there. We encountered an issue as @macropay-solutions, described and linked to the original issue.
I already have a question open with the 3rd party that manages the server and performs those updates, to ask what exactly their update procedure looks like. But I still see to fail how it can have things mixed up between versions somehow. Assuming you have more experience with PHP itself, maybe you can explain why and how these situations can occur where the version is updated but not the constants or similar situations. As far as i can see/understand Laravel's code works perfectly fine on 8.3.16 and also on 8.3.17, it's just that it apparently is possible to somehow get a mixture of the two |
I don't know how that would happen. that being getting back the old value for T_INLINE_HTML and the new value for the php version in the same request. |
Description
The following code:
Resulted in this output:
This was discovered while debugging this blade rendering issue laravel/framework#52382 (reply in thread)
that is affecting laravel blade templating.
First step that lead to this laravel/framework#52382 (comment) was discovered by @JFBauer-ZeroPlex.
Tested on https://onlinephp.io/ and also on local and shared. More details in the thread.
PHP Version
8.0->8.4
Operating System
No response
Is this normal? Why change the constant value from minor version to minor version?
The text was updated successfully, but these errors were encountered: