• Closed
• 21 Nov 2013
• Medium
• Build: master
•   # 2566
•   Diff
•   brunowego:patch-16
• Foreign ID 32768

• Success The Travis CI build passed Details

User tests: Successful: Unsuccessful:

brunowego
20 Nov 2013

Tracker: #32768

joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=32768&start=0

7848a4c 20 Nov 2013 Fix wrong param to helper route

brunobatista - open - 20 Nov 2013

wilsonge - comment - 20 Nov 2013

This doesn't seem right. Because in the content helper router we're looking for an integer https://github.com/joomla/joomla-cms/blob/master/components/com_content/helpers/route.php#L39

brunobatista - comment - 21 Nov 2013

Yes @wilsonge , you are correct! I closing this issue. Thanks.

brunobatista - change - 21 Nov 2013

The description was changed

Title	Differences … Fix wrong param to helper route [#32768] Fix wrong param to helper route
Description		⇒	<p>Tracker: #32768</p> <p>joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&amp;tracker_item_id=32768&amp;start=0</p>
Status	New	⇒	Closed
Closed_Date	0000-00-00 00:00:00	⇒	2013-11-21 10:04:52
Labels	Added: ? ?

brunobatista - close - 21 Nov 2013

infograf768 - comment - 22 Nov 2013

I find 9 occurences of

ContentHelperRoute::getArticleRoute($item->slug, $item->catslug)

in core.

Hackwar - comment - 5 Dec 2013

FYI: The Joomla core routers don't require the catslug. Its a leftover from Joomla 1.5 and hasn't been necessary since Joomla 1.6

Add a Comment

• Older
• Newer