?
avatar drjjw
drjjw
13 May 2014

Steps to reproduce the issue

  1. Install Joomla 3.3.0 without sample data
  2. Create test article, in uncategorized category, with text before and after a readmore link
  3. Create category list menu item that points to uncategorized Content => Article Managers => Options, set Show Unauthorised Links to Show

To reproduce:
1. When not logged into the front end, navigate the to the category list menu item
2. Click the Register to read more... link

Expected result

  1. Successful login and redirection to the requested article

Actual result

  1. Log in and I am redirected to a page that does not exist (404 error)

System information (as much as possible)

Joomla 3.3
PHP 5.4.26

Additional comments

We have patched this error in:

components\com_content\views\category\tmpl\default_articles.php

$returnURL = JRoute::_(ContentHelperRoute::getArticleRoute($article->slug));

with

$returnURL = JRoute::_(ContentHelperRoute::getArticleRoute($article->slug, $article->catid));

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
3.00

avatar drjjw drjjw - open - 13 May 2014
avatar betweenbrain
betweenbrain - comment - 13 May 2014

@hackwar is there any chance that this is an issue related to any of the recent router class improvements? Quickly checking the history of this file for the last couple of years, I don't see where this was changed. Likely, it was part of a bigger commit that isn't apparent.

avatar Hackwar
Hackwar - comment - 13 May 2014

Nope, this is not related to the router changes. The router changes did not touch any of this. If the category ID was missing, this is with 99.9% certainty an issue in earlier versions, too.

avatar betweenbrain
betweenbrain - comment - 13 May 2014

Thanks @Hackwar!

avatar nikosdion
nikosdion - comment - 26 Jul 2014

Please see 3979 which contains the PR for the code changes proposed by Matt. Closing this issue.

avatar betweenbrain
betweenbrain - comment - 26 Jul 2014

Thanks @nikosdion! I'll hit the close button for you ;-)

avatar betweenbrain betweenbrain - change - 26 Jul 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-07-26 13:28:54
avatar betweenbrain betweenbrain - close - 26 Jul 2014
avatar betweenbrain betweenbrain - close - 26 Jul 2014
avatar zero-24 zero-24 - close - 26 Jul 2014
avatar teubernet
teubernet - comment - 1 Sep 2014

I did find a similar Bug in Blog view.

I replaced:
$returnURL = JRoute::_(ContentHelperRoute::getArticleRoute($this->item->slug));

with:
$returnURL = JRoute::_(ContentHelperRoute::getArticleRoute($this->item->slug, $this->item->catid));

in blog_item.php

avatar zero-24
zero-24 - comment - 1 Sep 2014

@teubernet can you add a pull request aginst staging?

If you don't know how please see here.
http://docs.joomla.org/Using_the_Github_UI_to_Make_Pull_Requests

avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?

Add a Comment

Login with GitHub to post a comment