? Failure

User tests: Successful: Unsuccessful:

avatar asika32764
asika32764
2 Jul 2017

Summary of Changes

If a nestable view has a menu linked to some child items, then other items which not in children and havn't assigned to any menus will unable to create route.

Testing Instructions

Create a component (com_test) with new router rules and nesteable view, and add some items:

- Foo
    - Bar
        - Yoo
- Flower
    - Japan
        -Sakura

Assign a menu to Bar, go to Bar page, all routes linked to Japan and Sakura will not work, only link to Bar.

The route will be:

  • Bar: /bar
  • Yoo: /bar/3-yoo
  • Japan: /bar (Wrong URL)
  • Sakura: /bar (Wrong URL)

This PR fixed that if a view can not found parent menu, it will use an exists menu and add view alias after it.

Japan: /bar/5-japan
Sakura: /bar/5-japan/6-sakura

Actually, I think the real solution is fallback to /component/test/5-japan/6-sakura not after /bar, but I don't know how to do this. Router rules is too complex...

avatar asika32764 asika32764 - open - 2 Jul 2017
avatar asika32764 asika32764 - change - 2 Jul 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jul 2017
Category Libraries
avatar asika32764 asika32764 - change - 2 Jul 2017
The description was changed
avatar asika32764 asika32764 - edited - 2 Jul 2017
avatar asika32764 asika32764 - change - 2 Jul 2017
The description was changed
avatar asika32764 asika32764 - edited - 2 Jul 2017
avatar asika32764 asika32764 - change - 2 Jul 2017
The description was changed
avatar asika32764 asika32764 - edited - 2 Jul 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Jul 2017
The description was changed
avatar joomla-cms-bot joomla-cms-bot - edited - 2 Jul 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 Jul 2017
Category Libraries Libraries Router / SEF
avatar asika32764 asika32764 - change - 2 Jul 2017
The description was changed
avatar asika32764 asika32764 - edited - 2 Jul 2017
avatar wilsonge
wilsonge - comment - 4 Jul 2017
avatar csthomas
csthomas - comment - 11 Sep 2017

There is a more advanced proposition at #17746

avatar csthomas
csthomas - comment - 3 Jan 2018

Actually, I think the real solution is fallback to /component/test/5-japan/6-sakura

I have done it. You can test it at #19280.

avatar Quy
Quy - comment - 24 Jan 2018

@asika32764 @csthomas OK to close this PR?

avatar csthomas
csthomas - comment - 24 Jan 2018

If #19280 will be merged then yes.

avatar asika32764
asika32764 - comment - 25 Jan 2018

Since there has a PR for this issue, I'll close this PR first.

avatar asika32764 asika32764 - change - 25 Jan 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-01-25 03:58:26
Closed_By asika32764
avatar asika32764 asika32764 - close - 25 Jan 2018

Add a Comment

Login with GitHub to post a comment