User tests: Successful: Unsuccessful:
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.
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.
Category | ⇒ | Administration Components Installation SQL Postgresql MS SQL Unit Tests |
Status | New | ⇒ | Pending |
Labels |
Added:
?
?
|
Labels |
Added:
?
|
Labels |
I have tested this item
I have tested this item
RTC please
Status | Pending | ⇒ | Ready to Commit |
Labels |
Labels |
Added:
?
|
@brianteeman why does the bot respond you and not me?
@jeckodevelopment did you change the status to RTC on the issue tracker?
No, i didn't
Thats why then ;)
yeah :) thank you for the help!
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 |
Labels |
Removed:
?
?
|
I have tested this item✅ successfully on 6d1edb0
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11383.