PR-5.0-dev PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Mar 2024

Summary of Changes

In the large Rich Snippets PR #41151 a line was changed that has resulted in invalid markup and broken functionality of the language code plugin

Testing Instructions

  1. Create an article in and view the article in the front end.
    Check the source and you will see the invalid markup
    <meta content="en-GB">

  2. open the language code plugin, enable and save
    go to the new tab in that plugin and change the lanbguage code to en-BT
    Refresh the front and refresh the page. View the source and you will see the same invalid markup as in test 1

  3. Apply the one line change in this PR
    Disable the language code plugin
    Refresh the front and refresh the page. View the source and you will see the <meta itemprop="inLanguage" content="en-GB">
    This is the correct and valid markup

  4. Re-enable the language code plugin and check the frone end again and you will see <meta itemprop="inLanguage" content="en-BT">

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 brianteeman brianteeman - open - 22 Mar 2024
avatar brianteeman brianteeman - change - 22 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Mar 2024
Category Front End com_content
avatar brianteeman brianteeman - change - 22 Mar 2024
The description was changed
avatar brianteeman brianteeman - edited - 22 Mar 2024
avatar viocassel viocassel - test_item - 22 Mar 2024 - Tested successfully
avatar viocassel
viocassel - comment - 22 Mar 2024

I have tested this item ✅ successfully on e371631


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

avatar fgsw fgsw - test_item - 10 Apr 2024 - Tested successfully
avatar fgsw
fgsw - comment - 10 Apr 2024

I have tested this item ✅ successfully on e371631


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

avatar LadySolveig
LadySolveig - comment - 10 Apr 2024

Thank you for this contribution. I would really like to merge this in 5.1 if you would find the time to rebase @brianteeman

avatar brianteeman brianteeman - change - 10 Apr 2024
Title
[5.0] language code rich snippet revert
[5.1] language code rich snippet revert
avatar brianteeman brianteeman - edited - 10 Apr 2024
avatar brianteeman brianteeman - change - 10 Apr 2024
Labels Added: PR-5.0-dev PR-5.1-dev
avatar brianteeman
brianteeman - comment - 10 Apr 2024

rebased to 5.1 as requested

avatar bembelimen bembelimen - change - 11 Apr 2024
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-04-11 00:53:56
Closed_By bembelimen
avatar bembelimen bembelimen - close - 11 Apr 2024
avatar bembelimen bembelimen - merge - 11 Apr 2024
avatar bembelimen
bembelimen - comment - 11 Apr 2024

Thx

avatar Scrabble96
Scrabble96 - comment - 17 Apr 2024

I thought the idea in J5 was to remove itemprops from the template php files and let Joomla's schema.org add the correct code. Now, on updating from 5.0.3 to 5.1.0 I see that itemprop="inLanguage" has appeared in html/com_content/article/default.php. Is that right?

Add a Comment

Login with GitHub to post a comment