? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
21 May 2021

Pull Request for #29315

Steps to reproduce the issue

install sample data
login to frontend as super admin
Click Site Settings
Set "Site Name in Page Titles" to After
Save

Expected result

On every page, on the frontend, the site name should be appended to the page title.

Actual result

The site name is missing from the following pages
Click Site Settings menu item
Click Template Settings menu item
Click to edit a module

In addition Site Settings and Template Settings ignore all the other menu settings such as show heading and pageclass
Edit a module doesnt have a menu item so that doesnt apply for this one

This PR should resolve all of this.

@PhilETaylor could you please check the docblocks - I didn't really know what I was supposed to be doing there.

avatar brianteeman brianteeman - open - 21 May 2021
avatar brianteeman brianteeman - change - 21 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 May 2021
Category Front End com_config
avatar PhilETaylor PhilETaylor - test_item - 21 May 2021 - Tested successfully
avatar PhilETaylor
PhilETaylor - comment - 21 May 2021

I have tested this item successfully on 030b0a5


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

avatar Abernyte-Git Abernyte-Git - test_item - 21 May 2021 - Tested successfully
avatar Abernyte-Git
Abernyte-Git - comment - 21 May 2021

I have tested this item successfully on 030b0a5


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

avatar richard67
richard67 - comment - 21 May 2021

@brianteeman Could you check the review comments? Thanks in advance.

avatar brianteeman brianteeman - change - 21 May 2021
Labels Added: ? ?
avatar brianteeman brianteeman - change - 22 May 2021
Labels Removed: ?
avatar joomdonation
joomdonation - comment - 22 May 2021

Sorry @brianteeman . I forgot the Templates view in my PR. Just made another PR brianteeman#126 . Could you please merge that PR, too ? Then we will get someone here to test it. Thanks

avatar joomdonation joomdonation - test_item - 22 May 2021 - Tested successfully
avatar joomdonation
joomdonation - comment - 22 May 2021

I have tested this item successfully on 0bd1922


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

avatar richard67
richard67 - comment - 22 May 2021

@Abernyte-Git @PhilETaylor Could you test again? Thanks in advance.

avatar alikon alikon - test_item - 22 May 2021 - Tested successfully
avatar alikon
alikon - comment - 22 May 2021

I have tested this item successfully on 0bd1922


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

avatar alikon alikon - change - 22 May 2021
Status Pending Ready to Commit
avatar alikon
alikon - comment - 22 May 2021

RTC


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

avatar richard67 richard67 - change - 22 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-22 10:32:29
Closed_By richard67
Labels Added: ?
avatar richard67 richard67 - close - 22 May 2021
avatar richard67 richard67 - merge - 22 May 2021
avatar richard67
richard67 - comment - 22 May 2021

Thanks!

avatar brianteeman
brianteeman - comment - 22 May 2021

thank you

Add a Comment

Login with GitHub to post a comment