Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PHP7.4: Deprecated warning - serializationContext.php on line 152 #1111

Closed
rbairwell opened this issue Aug 7, 2019 · 2 comments
Closed

PHP7.4: Deprecated warning - serializationContext.php on line 152 #1111

rbairwell opened this issue Aug 7, 2019 · 2 comments
Labels
bug

Comments

@rbairwell
Copy link

Q A
Bug report? yes
Feature request? no
BC Break report? no
RFC? no

Steps required to reproduce the problem

  1. Setup a PHP7.4 environment with deprecated warnings turned on.
  2. Serialize something (I will expand on this once I've identified the exact cause)
  3. Receive:
    <b>Deprecated</b>: Unparenthesized a ? b : c ? d : eis deprecated. Use either(a ? b : c) ? d : eora ? b : (c ? d : e) in <b>/data/vendor/jms/serializer/src/SerializationContext.php</b> on line <b>152</b>

Expected Result

No deprecated warning

Actual Result

<b>Deprecated</b>: Unparenthesized a ? b : c ? d : eis deprecated. Use either(a ? b : c) ? d : eora ? b : (c ? d : e) in <b>/data/vendor/jms/serializer/src/SerializationContext.php</b> on line <b>152</b>
@goetas goetas mentioned this issue Aug 8, 2019
Merged
@goetas
Copy link
Collaborator

goetas commented Aug 8, 2019

Hi, thanks for reporting this.
I started to work on PHP 7.4 compatibility in #1113

@goetas goetas added the bug label Aug 8, 2019
@goetas
Copy link
Collaborator

goetas commented Aug 18, 2019

fixed in #1113

@goetas goetas closed this as completed Aug 18, 2019
markelarauzo7 added a commit to markelarauzo7/ddd that referenced this issue Oct 7, 2020
@markelarauzo7
ugrade jms/serializer to 3.9.0 so that it does not lock version in 1.…
cd99bef 
…X which is not compatible with php7.4

See: schmittjoh/serializer#1111
@markelarauzo7 markelarauzo7 mentioned this issue Oct 7, 2020
Closed
mhujer added a commit to mhujer/jms-serializer-uuid that referenced this issue Dec 31, 2020
@mhujer
lock JMS/Serializer to 3.2+ which is first with PHP 7.4 compatibility
3b021e9 
See schmittjoh/serializer#1111 and schmittjoh/serializer#1113
mhujer added a commit to mhujer/jms-serializer-uuid that referenced this issue Dec 31, 2020
@mhujer
lock JMS/Serializer to 3.2+ which is first with PHP 7.4 compatibility
5f1d516 
See schmittjoh/serializer#1111 and schmittjoh/serializer#1113
mhujer added a commit to mhujer/jms-serializer-uuid that referenced this issue Dec 31, 2020
@mhujer
lock JMS/Serializer to 3.2+ which is first with PHP 7.4 compatibility
135bb48 
See schmittjoh/serializer#1111 and schmittjoh/serializer#1113
mhujer added a commit to mhujer/consistence-jms-serializer that referenced this issue Mar 5, 2021
@mhujer
require jms/serializer 3.2+ to prevent errors on PHP 7.4
db60dc4 
Ref schmittjoh/serializer#1111
mhujer added a commit to mhujer/consistence-jms-serializer that referenced this issue Mar 5, 2021
@mhujer
require jms/serializer 3.2+ to prevent errors on PHP 7.4
7d5ebd6 
Ref schmittjoh/serializer#1111
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rbairwell @goetas