? bug PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Mar 2023

Pull Request for Issue #40123 .

Summary of Changes

add a meaningful title to the feed by using the document page title

Testing Instructions

Create the following menu items
Blog Category
Featured Articles
Tags (List All)
Contacts (list)

Add page titles to some of them

Actual result BEFORE applying this Pull Request

If you view the source you will find the following in the header

<link href="seite.feed?type=rss" rel="alternate" type="application/rss+xml" title="RSS 2.0">
<link href="seite.feed?type=atom" rel="alternate" type="application/atom+xml" title="Atom 1.0">

Expected result AFTER applying this Pull Request

<link href="seite.feed?type=rss" rel="alternate" type="application/rss+xml" title="xxxxxxxxxx">
<link href="seite.feed?type=atom" rel="alternate" type="application/atom+xml" title="xxxxxxxx">

Where xxxxxxxx is the page title according to the settings so it is the same as the browser page title eg

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 - 22 Mar 2023
Category Front End com_content com_finder com_tags Libraries
avatar brianteeman brianteeman - open - 22 Mar 2023
avatar brianteeman brianteeman - change - 22 Mar 2023
Status New Pending
avatar brianteeman brianteeman - change - 22 Mar 2023
The description was changed
avatar brianteeman brianteeman - edited - 22 Mar 2023
avatar viocassel viocassel - test_item - 22 Mar 2023 - Tested successfully
avatar viocassel
viocassel - comment - 22 Mar 2023

I have tested this item successfully on 630886d


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

avatar Mich-es
Mich-es - comment - 22 Mar 2023

Tested successfully - Thanks!! 630886d

avatar Quy
Quy - comment - 22 Mar 2023

Should escape title just in case it contains an ".
<link href="/joomla-cms-4.3-dev/index.php/all-tags?format=feed&amp;type=rss" rel="alternate" type="application/rss+xml" title="All "Tags"">

avatar brianteeman brianteeman - change - 22 Mar 2023
Labels Added: PR-4.3-dev
avatar SharkyKZ
SharkyKZ - comment - 22 Mar 2023

Should escape title just in case it contains an ". <link href="/joomla-cms-4.3-dev/index.php/all-tags?format=feed&amp;type=rss" rel="alternate" type="application/rss+xml" title="All "Tags"">

This should be done in the renderer and not here.

avatar brianteeman
brianteeman - comment - 22 Mar 2023

and that would be where?

avatar Mich-es Mich-es - test_item - 23 Mar 2023 - Tested successfully
avatar Mich-es
Mich-es - comment - 23 Mar 2023

I have tested this item successfully on 6137e4d


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

avatar PjotrClaas PjotrClaas - test_item - 12 May 2023 - Tested successfully
avatar PjotrClaas
PjotrClaas - comment - 12 May 2023

I have tested this item successfully on 6137e4d

Tested successfully! 6137e4d


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

avatar richard67 richard67 - change - 12 May 2023
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 12 May 2023

RTC


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

avatar obuisard obuisard - change - 12 May 2023
Labels Added: ? bug
avatar obuisard obuisard - close - 12 May 2023
avatar obuisard obuisard - merge - 12 May 2023
avatar obuisard obuisard - change - 12 May 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-12 17:38:24
Closed_By obuisard
avatar obuisard
obuisard - comment - 12 May 2023

Thank you Brian @brianteeman !

avatar brianteeman
brianteeman - comment - 12 May 2023

Thanks

Add a Comment

Login with GitHub to post a comment