PHP 8.x bug PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
11 Oct 2022

Summary of Changes

In several view classes we are missing defining properties of the class that we then use later on in the view causing us to have magic properties. This ensures that they are correctly set in the code base as explicit properties

Testing Instructions

Check tag list, categories, articles list and newsfeed list views and check that they continue to work as before with no changes

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

avatar wilsonge wilsonge - open - 11 Oct 2022
avatar wilsonge wilsonge - change - 11 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Oct 2022
Category Administration com_categories com_content com_newsfeeds com_tags
avatar wilsonge wilsonge - change - 29 Nov 2022
Labels Added: PR-4.3-dev
avatar chmst chmst - test_item - 11 Jan 2023 - Tested successfully
avatar chmst
chmst - comment - 11 Jan 2023

I have tested this item successfully on f818f17

code review


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

avatar laoneo laoneo - change - 11 Jan 2023
Title
Add missing class properties to views
[4.2] PHP8.2 Add missing class properties to views
avatar laoneo laoneo - edited - 11 Jan 2023
avatar laoneo laoneo - change - 11 Jan 2023
Labels Added: PHP 8.x
avatar laoneo laoneo - change - 11 Jan 2023
Title
[4.2] PHP8.2 Add missing class properties to views
[4.3] PHP8.2 Add missing class properties to views
avatar laoneo laoneo - edited - 11 Jan 2023
avatar laoneo laoneo - test_item - 11 Jan 2023 - Tested successfully
avatar laoneo
laoneo - comment - 11 Jan 2023

I have tested this item successfully on f818f17

Also code review


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

avatar laoneo
laoneo - comment - 12 Jan 2023

@wilsonge can you fix the code style errors?

avatar laoneo laoneo - change - 14 Apr 2023
Labels Added: ? bug
avatar laoneo
laoneo - comment - 14 Apr 2023

@wilsonge can you fix the code style issues here?

avatar laoneo laoneo - change - 17 Apr 2023
Title
[4.3] PHP8.2 Add missing class properties to views
[4.4] PHP8.2 Add missing class properties to views
avatar laoneo laoneo - edited - 17 Apr 2023
avatar laoneo laoneo - change - 17 Apr 2023
Labels Added: PR-4.4-dev
Removed: PR-4.3-dev
avatar alikon alikon - test_item - 29 Apr 2023 - Tested successfully
avatar alikon
alikon - comment - 29 Apr 2023

I have tested this item successfully on d3fb5cd


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

avatar laoneo laoneo - change - 29 Apr 2023
Labels Removed: ?
avatar laoneo laoneo - close - 29 Apr 2023
avatar laoneo laoneo - merge - 29 Apr 2023
avatar laoneo laoneo - change - 29 Apr 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-04-29 11:28:11
Closed_By laoneo
avatar laoneo
laoneo - comment - 29 Apr 2023

Thanks!

Add a Comment

Login with GitHub to post a comment