RTC bug PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
24 Apr 2024

Summary of Changes

While going through the code with phpstan, this showed up as error. Instead of assigning the string to a new entry in the array, we are also trying to read from this array by trying to concatenate a string... I'm struggling to explain it, but if you look at the code, you see the problem. Right now this does not throw an error in PHP, but it is just a matter of time when PHP becomes strict enough to complain about this.

Testing Instructions

Codereview

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 Hackwar Hackwar - open - 24 Apr 2024
avatar Hackwar Hackwar - change - 24 Apr 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Apr 2024
Category Front End com_contact
avatar Quy Quy - test_item - 25 Apr 2024 - Tested successfully
avatar Quy Quy - change - 25 Apr 2024
Labels Added: PR-4.4-dev
avatar Quy Quy - test_item - 25 Apr 2024 - Tested successfully
avatar Quy
Quy - comment - 25 Apr 2024

I have tested this item ✅ successfully on 59e7346


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

avatar Quy Quy - alter_testresult - 28 Apr 2024 - HLeithner: Tested successfully
avatar Quy Quy - change - 28 Apr 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 28 Apr 2024

RTC


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

avatar Quy Quy - change - 28 Apr 2024
Labels Added: RTC
avatar Quy Quy - change - 5 May 2024
Labels Added: bug
avatar MacJoom MacJoom - change - 6 May 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-05-06 17:42:19
Closed_By MacJoom
avatar MacJoom
MacJoom - comment - 6 May 2024

Thank you!

Add a Comment

Login with GitHub to post a comment