? Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
6 Jul 2016

This change enables the SEF plugin to run even when the SEF URLs are not enabled. The background is, that with the new routing system the component routers are always invoked and called with each URL in order to do the necessary changes, especially setting the right Itemid. Without this change, the URLs everywhere are processed, except for those in content, when the SEF URLs are disabled. That means that with the new system, URLs in content would not get the right Itemid.

avatar Hackwar Hackwar - open - 6 Jul 2016
avatar Hackwar Hackwar - change - 6 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jul 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 6 Jul 2016
Category Plugins Router / SEF
avatar hardiktailored hardiktailored - test_item - 1 Aug 2016 - Not tested
avatar hardiktailored
hardiktailored - comment - 1 Aug 2016

I have not tested this item.

I copied one article URL when SEF enabled and added in content of another article. Then I disabled the SEF and checked the URL, it doesn't work and redirecting me to the home page. Then I applied patch and again visited the same URL from article, this time also redirected me to home page.
The URL looks same in both the cases. I don't find any URL or parameters changes or anything.
One difference though, before applying patch home page image was not loading and after patch image loads.

Can you please provide more details on how to test? Do I need menu item to test this?


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

avatar brianteeman brianteeman - change - 3 Aug 2016
Labels Added: ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2016
Category Plugins Router / SEF Plugins Front End Router / SEF
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2016
Labels Added: ?
avatar Hackwar
Hackwar - comment - 2 Oct 2016
  1. Use 3.7-dev as base branch
  2. Disable SEF URLs
  3. Get a URL of an article and remove the Itemid
  4. Paste the URL into a link in an article
  5. Look at the article with the URL in the frontend and see that the Itemid is still missing.
  6. Apply this PR
  7. Reload the article with the URL in the frontend again and see that the Itemid is properly added.
avatar wilsonge
wilsonge - comment - 5 Oct 2016

Based on Hannes and myself discussion earlier I am merging this on review to help the testing of the 3.7.x router

avatar wilsonge wilsonge - change - 5 Oct 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-10-05 20:22:26
Closed_By wilsonge
avatar wilsonge wilsonge - close - 5 Oct 2016
avatar wilsonge wilsonge - merge - 5 Oct 2016

Add a Comment

Login with GitHub to post a comment