Skip to content

[4.3] PHP8.2 Dynamic property warning in User class#39620

Merged
roland-d merged 5 commits intojoomla:4.2-devfrom
alikon:patch-7
Jan 14, 2023
Merged

[4.3] PHP8.2 Dynamic property warning in User class#39620
roland-d merged 5 commits intojoomla:4.2-devfrom
alikon:patch-7

Conversation

@alikon
Copy link
Copy Markdown
Contributor

@alikon alikon commented Jan 13, 2023

Pull Request for Issue php 8.2 - dynamic property Deprecated

Summary of Changes

declared property

Testing Instructions

open frontend with php 8.2 with Error Reporting = maximum

Actual result BEFORE applying this Pull Request

Deprecated: Creation of dynamic property Joomla\CMS\User\User::$aid is deprecated in /shared/httpd/dev4/joomla-cms/libraries/src/User/User.php on line 871

Expected result AFTER applying this Pull Request

no more deprecation

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Deprecated: Creation of dynamic property Joomla\CMS\User\User::$aid is deprecated in /shared/httpd/dev4/joomla-cms/libraries/src/User/User.php on line 871
@laoneo laoneo added the PHP 8.x PHP 8.x deprecated issues label Jan 13, 2023
@laoneo laoneo changed the title php 8.2 - dynamic property Deprecated [4.3] PHP8.2 Dynamic property warning in User class Jan 13, 2023
@laoneo
Copy link
Copy Markdown
Member

laoneo commented Jan 13, 2023

I have tested this item ✅ successfully on 5e3bffc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39620.

1 similar comment
@joomdonation
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 5e3bffc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39620.

@joomdonation
Copy link
Copy Markdown
Contributor

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39620.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 13, 2023
@joomdonation joomdonation added this to the Joomla! 4.2.7 milestone Jan 13, 2023
@roland-d roland-d merged commit 11e772c into joomla:4.2-dev Jan 14, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 14, 2023
@roland-d
Copy link
Copy Markdown
Contributor

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PHP 8.x PHP 8.x deprecated issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants