[#17497] - [Router] RulesStandard Supports `default` layout
- Closed
- 26 Feb 2018
- Medium
- Build: staging
- # 17497
- Diff
- asika32764:patch-2
User tests: Successful: Unsuccessful:
Summary of Changes
The Standard rule will remove view and layout if found menu item's layout name same as query.
For example: view=foo&layout=blog will generate route as /blog.html
But if we use default layout, mostly we will not add layout to URL, the StandardRule will generate a route like: /blog.html&view=foo
This PR try to remove view and layout if menu item and query both has no layout params.
Testing Instructions
Install Joomla will test sample data, in admin open Australian Parks article and add a link with URL: index.php?option=com_content&view=featured, then save it.
Go to frontend and you will see the SEF link is: .../index.php/using-joomla/extensions/templates/beez-2/home-page-beez-2?view=featured
Apply this PR should remove the ?view=featured
| Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
Actually, I lied. I am able to get the ?view=featured in the url, but the patch didn't remove it unless I'm missing a step.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17497.
@asika32764 Is this the same as #18429?
I'm not sure, I need test.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-02-26 16:50:55 |
| Closed_By | ⇒ | Quy |
Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/17497
No longer reproducible because of #18429
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17497.
I have tested this item? unsuccessfully on 53425f7
I could not duplicate the problem after following the steps. My link didn't have a ?view=featured in it to begin with. :(
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17497.