? Success

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
14 Dec 2014

During #5276 I changed from catslug to catid in the call to ContentHelperRoute::getArticleRoute(). Unfortunately, we don't retrieve the catid from the database. That is why we get a notice here.

Testinstructions

Search something in your Joomla installation and make sure that you have the content search plugin enabled. Notice the notice in the output for every content link.
Apply the change, see the notice disappear.

Thanks @shur for spotting this bug.

avatar Hackwar Hackwar - open - 14 Dec 2014
avatar jissues-bot jissues-bot - change - 14 Dec 2014
Labels Added: ?
avatar shur
shur - comment - 14 Dec 2014

@Hackwar

tested, all works fine now!

avatar brianteeman brianteeman - change - 15 Dec 2014
Category Search SEF
avatar waader
waader - comment - 15 Dec 2014

@test works!

avatar waader waader - test_item - 15 Dec 2014 - Tested successfully
avatar zero-24 zero-24 - change - 15 Dec 2014
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 15 Dec 2014

RTC Thanks too all!


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5423.

avatar zero-24 zero-24 - alter_testresult - 15 Dec 2014 - shur: Tested successfully
avatar brianteeman brianteeman - change - 15 Dec 2014
Labels Added: ?
avatar zero-24 zero-24 - close - 17 Dec 2014
avatar wilsonge wilsonge - reference | d27dc54 - 17 Dec 14
avatar wilsonge wilsonge - merge - 17 Dec 2014
avatar wilsonge wilsonge - close - 17 Dec 2014
avatar wilsonge wilsonge - close - 17 Dec 2014
avatar wilsonge wilsonge - change - 17 Dec 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-12-17 13:18:07
avatar wilsonge
wilsonge - comment - 17 Dec 2014

Thanks Hannes! Merged

avatar wilsonge wilsonge - change - 17 Dec 2014
Milestone Added:
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?
avatar Hackwar Hackwar - head_ref_deleted - 6 Jan 2016

Add a Comment

Login with GitHub to post a comment