? ? Pending

User tests: Successful: Unsuccessful:

avatar Didldu-Florian
Didldu-Florian
6 Jul 2020

Summary of Changes

Site title "- Administration" should only be added at administrator.
This happens when component developers use the toolbar helper in the frontend

Testing Instructions

Using Toolbar Helper in a Frontend Compoennt and check the site title.

You can test it by changeing article view like these

4.0-dev...Didldu-Florian:patch-6

Open then an article in frontend, you will see then the problem in the <title>

Actual result BEFORE applying this Pull Request

The site title ist looking like these at frontend:

<title>Home - JADMINISTRATION</title>

Expected result AFTER applying this Pull Request

<title>Home</title>

The Problem is also that the language key JADMINISTRATION is not translated either.
So finaly we should not add "- JADMINISTRATION" at frontend.

This fix does this

avatar Didldu-Florian Didldu-Florian - open - 6 Jul 2020
avatar Didldu-Florian Didldu-Florian - change - 6 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jul 2020
Category Libraries
avatar Didldu-Florian Didldu-Florian - change - 6 Jul 2020
Labels Added: ?
avatar richard67 richard67 - change - 6 Jul 2020
Title
[4.0] Fix for wrond site title by using ToobarHelper in frontend
[4.0] Fix for wrong site title when using ToobarHelper in frontend
avatar richard67 richard67 - edited - 6 Jul 2020
avatar Didldu-Florian
Didldu-Florian - comment - 6 Jul 2020

Please fix again code style errors reported by Drone: https://ci.joomla.org/joomla/joomla-cms/33784/1/9

It's fixed

avatar richard67
richard67 - comment - 6 Jul 2020

It's fixed

@Didldu-Florian Yes, I've noticed. Do we have some component in core we can use for testing? Or do you have some 3rd party component you can tell?

avatar Didldu-Florian
Didldu-Florian - comment - 6 Jul 2020

@richard67 in core not, but you can test it by by changing article view

4.0-dev...Didldu-Florian:patch-6

Open then an article in frontend, you will see then the problem in the <title>

avatar Didldu-Florian Didldu-Florian - change - 6 Jul 2020
The description was changed
avatar Didldu-Florian Didldu-Florian - edited - 6 Jul 2020
avatar richard67 richard67 - test_item - 6 Jul 2020 - Tested successfully
avatar richard67
richard67 - comment - 6 Jul 2020

I have tested this item successfully on d34f026


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

avatar Quy
Quy - comment - 6 Jul 2020

@richard67 See link on to do multi-line comments/edits: https://twitter.com/github/status/1179101186437324801

avatar richard67
richard67 - comment - 6 Jul 2020

@Quy Yes, but that is just comment, not changes suggestions.

avatar Quy
Quy - comment - 6 Jul 2020

It is an outdated post, but applies to suggestions too. I have done it. Give it a try and you will see.

avatar pabloarias pabloarias - test_item - 9 Jul 2020 - Tested successfully
avatar pabloarias
pabloarias - comment - 9 Jul 2020

I have tested this item successfully on d34f026

I've added the code in com_content and see JADMINISTRATOR in title. After apply the path I don't see that word in title.


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

avatar Quy Quy - change - 9 Jul 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 9 Jul 2020

RTC


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

avatar infograf768 infograf768 - close - 10 Jul 2020
avatar infograf768 infograf768 - merge - 10 Jul 2020
avatar infograf768 infograf768 - change - 10 Jul 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-10 16:10:32
Closed_By infograf768
Labels Added: ?
avatar infograf768
infograf768 - comment - 10 Jul 2020

Tks.

Add a Comment

Login with GitHub to post a comment