PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar svenbluege
svenbluege
8 Mar 2023

Deprecation messages are shown for some tagged elements:
image

Summary of Changes

In com_tags, a tagged element's text-variable is transformed into null if the value is empty.
That causes deprecation messages in plugins like loadmodules and emailcloak. They usually don't need to check for null values in $article->text.

Testing Instructions

  1. configure PHP to show deprecation messages
  2. create a new contact using com_contact: just enter a name and tag it with "testtag"
  3. open the frontend /index.php?option=com_tags&view=tags
  4. click on the new tag 'testtag'.
  5. see the deprecation messages.

Actual result BEFORE applying this Pull Request

Following the test instructions, deprecation messages are shown

Expected result AFTER applying this Pull Request

Following the test instructions, deprecation messages are now shown

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 joomla-cms-bot joomla-cms-bot - change - 8 Mar 2023
Category com_tags Front End
avatar svenbluege svenbluege - open - 8 Mar 2023
avatar svenbluege svenbluege - change - 8 Mar 2023
Status New Pending
avatar svenbluege
svenbluege - comment - 8 Mar 2023

This issue was fixed with the same code in 4.2.7, I'm missing the change in 4.3. Here is my try to add it to 4.3 again.
Original PR which was included in 4.2.7: #39640

avatar Quy Quy - test_item - 9 Mar 2023 - Tested successfully
avatar Quy
Quy - comment - 9 Mar 2023

I have tested this item successfully on 548adf2


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

avatar Quy
Quy - comment - 9 Mar 2023

@obuisard This was missed in the upmerge. Should be ok to merge without an additional test.

avatar obuisard obuisard - change - 10 Mar 2023
Labels Added: PR-4.3-dev
avatar obuisard obuisard - change - 10 Mar 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-03-10 03:32:58
Closed_By obuisard
avatar obuisard obuisard - close - 10 Mar 2023
avatar obuisard obuisard - merge - 10 Mar 2023
avatar obuisard
obuisard - comment - 10 Mar 2023

Thank you Sven @svenbluege for the PR!

Add a Comment

Login with GitHub to post a comment