? ? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
1 Aug 2016

Summary of Changes

The primary help screen system is no longer a CMS extension and the replacement application does not use the option=com_help query fragment so we can stop setting it as part of the help screen system's URLs for the default configuration.

Testing Instructions

For purposes of validating this patch, code review is all that is necessary for the SQL files.

For other test cases, first you'll want to re-save your global configuration. This will cause the helpurl variable to be saved with the new value (without the option=com_help query fragment). After that, clicking any help button in the backend should cause a new window to pop up and notice that in that URL the option=com_help query fragment isn't there but you still get the correct output.

To check the behavior on the "Help > Joomla! Help" menu, you'll need your browser's dev console open to inspect network requests. Clicking on any of the links on the left menu should get you that help page and validate the request does not have the option=com_help fragment.

Note: Though this will "fix" #11381 by removing the need for the ampersand in the request URL, it's really not a proper fix as it's just bypassing whatever change happened to cause the ampersand to stop being decoded.

avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2016
Category Administration Components Installation SQL Postgresql MS SQL Unit Tests
avatar mbabker mbabker - open - 1 Aug 2016
avatar mbabker mbabker - change - 1 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2016
Labels Added: ? ?
avatar brianteeman brianteeman - change - 1 Aug 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 1 Aug 2016
Labels
avatar brianteeman brianteeman - test_item - 1 Aug 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 1 Aug 2016

I have tested this item successfully on 6d1edb0


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

avatar jeckodevelopment jeckodevelopment - test_item - 1 Aug 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 1 Aug 2016

I have tested this item successfully on 6d1edb0


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

avatar Lavsteph Lavsteph - test_item - 1 Aug 2016 - Tested successfully
avatar Lavsteph
Lavsteph - comment - 1 Aug 2016

I have tested this item successfully on 6d1edb0


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

avatar jeckodevelopment
jeckodevelopment - comment - 1 Aug 2016

RTC please

avatar brianteeman brianteeman - change - 1 Aug 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 1 Aug 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2016
Labels Added: ?
avatar jeckodevelopment
jeckodevelopment - comment - 1 Aug 2016

@brianteeman why does the bot respond you and not me?

avatar brianteeman
brianteeman - comment - 1 Aug 2016

@jeckodevelopment did you change the status to RTC on the issue tracker?

avatar jeckodevelopment
jeckodevelopment - comment - 1 Aug 2016

No, i didn't

avatar brianteeman
brianteeman - comment - 1 Aug 2016

Thats why then ;)

avatar jeckodevelopment
jeckodevelopment - comment - 1 Aug 2016

yeah :) thank you for the help!

avatar wilsonge wilsonge - change - 1 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-01 13:17:56
Closed_By wilsonge
avatar brianteeman brianteeman - change - 1 Aug 2016
Labels Removed: ? ?

Add a Comment

Login with GitHub to post a comment