Skip to content

Command sync:conditional-fields shows warnings in PHP 8.2 #3884

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
Closed
matthewjohns0n opened this issue Nov 8, 2023 · 1 comment · Fixed by #3885
Closed

Command sync:conditional-fields shows warnings in PHP 8.2 #3884

matthewjohns0n opened this issue Nov 8, 2023 · 1 comment · Fixed by #3885
Assignees
@matthewjohns0n

Comments

@matthewjohns0n
Copy link
Member

matthewjohns0n commented Nov 8, 2023 •
edited
Loading

No description provided.

matthewjohns0n added a commit that referenced this issue Nov 8, 2023
@matthewjohns0n
Add properties that are used in the command. Fixes #3884
d87fd21
@matthewjohns0n matthewjohns0n self-assigned this Nov 8, 2023
@intoeetive intoeetive mentioned this issue Dec 12, 2023
Merged
bryannielsen pushed a commit that referenced this issue Jan 9, 2024
@matthewjohns0n @bryannielsen
Add properties that are used in the command. Fixes #3884
9502231
bryannielsen added a commit that referenced this issue Jan 15, 2024
@bryannielsen
Merge pull request #3885 from ExpressionEngine/bug/7.dev/warnings-in-…
cbf5d5d 
…php8-in-cli-command

Resolved #3884 where properties were undefined on the class and would through an error in PHP 8.2 when using the `sync:conditional-fields` command
@bryannielsen
Copy link
Contributor

This should be resolved now in version 7.3.15

intoeetive added a commit that referenced this issue Jan 24, 2024
@intoeetive @matthewjohns0n
@lnicely @bryannielsen
7.x into 7.dev after EE 7.3.15 release (#4014)
b6cf939 
* fix: Adds padding to Publisher button dropdowns on mobile devices

* Resolved #3749 where TLS 1.3 was showing up as option in email settings for PHP versions that don't support it

* Update mt_rand to use random_int

* ExpressionEngine 7.3.15

* Resolved issue where saving entry with Fluid field using groups could throw PHP error if field was removed from group

* Add properties that are used in the command. Fixes #3884

* Resolved #3989 where Role permissions could have been saved during validation request

* cover with Cypress tests

* Fix the redirect library so it always works with quick links

* Run full test suite on release branches (not -alpha)

* Improve handling of redirects for root relative urls

---------

Co-authored-by: Matt Johnson <matt@packettide.com>
Co-authored-by: lny08017 <lucas.nicely@kub.org>
Co-authored-by: Matt Johnson <moosematt11@gmail.com>
Co-authored-by: Bryan Nielsen <bryan@packettide.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matthewjohns0n matthewjohns0n

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@bryannielsen @matthewjohns0n