? ? ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
26 Dec 2017

Pull Request for Issue #19149.

Summary of Changes

Remove ?amp; from Help URLs and update to Help38

Testing Instructions

Install PR.
Delete configuration.php.
Go to the front end to reinstall Joomla.
Install Test English (GB) Sample Data.
Go to front end.
Under All Modules, click Smart Search.
Click Help in the article.

Expected result

https://help.joomla.org/proxy/index.php?keyref=Help38:Extensions_Module_Manager_Smart_Search

Actual result

https://help.joomla.org/proxy/index.php?amp;keyref=Help31:Extensions_Module_Manager_Smart_Search

Documentation Changes Required

avatar Quy Quy - open - 26 Dec 2017
avatar Quy Quy - change - 26 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Dec 2017
Category SQL Installation Postgresql MS SQL Unit Tests
avatar brianteeman
brianteeman - comment - 27 Dec 2017

If your going to correct it then let's have the correct url with at least 38 not 31:)

avatar Quy Quy - change - 27 Dec 2017
Labels Added: ? ?
avatar Quy Quy - change - 27 Dec 2017
Title
Fix Help URLs
Fix Help URLs and update to Help38
avatar Quy Quy - edited - 27 Dec 2017
avatar Quy Quy - change - 27 Dec 2017
The description was changed
avatar Quy Quy - edited - 27 Dec 2017
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 27 Dec 2017 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017

I have tested this item ? unsuccessfully on 17075be

Wrong Link stay: https://help.joomla.org/proxy/index.php?amp;keyref=Help31:Extensions_Module_Manager_Smart_Search

Uninstalling PR give:
bildschirmfoto 2017-12-27 um 07 50 46

System information

3.8.4-dev
Multilanguage Site (4 Lang.) & Test English (GB) Sample Data
macOS Sierra, 10.12.6
Firefox 57 (64-bit)

MAMP 4.2

  • Apache-Server
  • PHP 7.0.22
  • MySQLi 5.6.35

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19181.
avatar Quy Quy - change - 27 Dec 2017
The description was changed
avatar Quy Quy - edited - 27 Dec 2017
avatar Quy
Quy - comment - 27 Dec 2017

You will have to reinstall Joomla. Testing instructions updated.

avatar Quy Quy - change - 27 Dec 2017
The description was changed
avatar Quy Quy - edited - 27 Dec 2017
avatar Quy Quy - change - 27 Dec 2017
The description was changed
avatar Quy Quy - edited - 27 Dec 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017

I tested first in installed 3.8.4 and got wrong URL. Then i installed a fresh staging with same Result (wrong URL).

avatar Quy
Quy - comment - 27 Dec 2017

Please follow testing instructions.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017

Reinstall Joomla.

means "Joomla Update > Reinstall Joomla Core files"?

avatar Quy
Quy - comment - 27 Dec 2017

No. Once the configuration file is deleted, access the front end which will force an installation.

avatar Quy Quy - change - 27 Dec 2017
The description was changed
avatar Quy Quy - edited - 27 Dec 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017
  • patch installed
  • configuration.php deleted
  • copy "Installation"-Folder from downloaded staging to Joomla3-Folder
  • open Frontend and reinstall Joomla

Same Result: Wrong Link https://help.joomla.org/proxy/index.php?amp;keyref=Help31:Extensions_Module_Manager_Smart_Search

avatar mbabker
mbabker - comment - 27 Dec 2017

copy "Installation"-Folder from downloaded staging to Joomla3-Folder

This is the part you're messing up. You need the installation folder from this PR, not from staging (staging has the bad URL, this PR has the good one).

Download the ZIP from https://github.com/Quy/joomla-cms/tree/helpurl and install that.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 27 Dec 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 27 Dec 2017

I have tested this item successfully on 17075be

Thanks for Help @Quy @mbabker


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

avatar alikon
alikon - comment - 28 Feb 2018

I have tested this item successfully on 17075be


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

avatar alikon alikon - test_item - 28 Feb 2018 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Feb 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Feb 2018

Ready to Commit after two successful tests.

avatar mbabker mbabker - change - 17 Mar 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-03-17 15:13:43
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 17 Mar 2018
avatar mbabker mbabker - merge - 17 Mar 2018

Add a Comment

Login with GitHub to post a comment