[#14031] - Fixing URLs for orphaned articles
- Fixed in Code Base
- 12 Feb 2017
- Medium
- Build: staging
- # 14031
- Diff
- Hackwar:patch-15
User tests: Successful: Unsuccessful:
Pull Request for Issue #13978 .
Summary of Changes
The issue is, that the home menu item is of type article and thus all orphaned articles are thought as being the homepage article. This is again because we are checking the path of views and even check the last view in the path. (the one without any further children) However, if we are at that point, we should be skipping that view, since we should either have matched that in the first step, where we match for the exactly matching menu item or not find a match at all.
Testing Instructions
Please see the instructions in #13978
Expected result
Actual result
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
This also works for me for the orphaned articles. I'm going to merge this with my and JM's test but the second half of the issue with the navigation from the category view needs fixing as he mentioned still so I will leave the original issue open for that
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-02-12 16:22:57 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
From Article 1 I clicked the Article 2 link and it takes me to Article 2 properly.
From Article 2 I clicked on Article 3 and it took me to Article 3 properly.
From Article 3 I clicked on Article 2 and it took me the the category list with Article 3 in it.
So I agree and can confirm what JM and George got too.
PS I have no idea what I'm doing with PRs. I manually replaced the code in the file in here with what was in the site via FTP and of course missed a { because I lack knowledge but eventually I got it! Was that the right way to do it? Sorry for cluttering this up with me not being quite clever enough to know what I don't know. ;)
Applying manually works. It's probably easier for you to use patchtester tho Jenn - you can find the instructions here :) https://docs.joomla.org/Testing_Joomla!_patches
Ah, yes, the patchtester! I forgot about it. Thanks, George.
Does this still need testing?
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14031.
@JoshuaLewis not at this moment
Tested;
From test instructions:
This one is solved by this PR.
BUT trying to to navigate to article 3 loads the category list menu item
That one is not solved: it loads the category list menu item
