Joomla! Issue Tracker | Joomla! CMS #5423 - Fix notice in content search plugin

?

Closed
17 Dec 2014
Medium
Build: staging
# 5423
Diff
Hackwar:patch-3

Success

Success The Travis CI build passed Details

User tests: Successful: Unsuccessful:

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.

577abb2 14 Dec 2014

Adding catid to query

Hackwar - open - 14 Dec 2014

jissues-bot - change - 14 Dec 2014

Labels

Added: ?

shur - comment - 14 Dec 2014

tested, all works fine now!

brianteeman - change - 15 Dec 2014

Category

⇒

Search SEF

waader - comment - 15 Dec 2014

@test works!

waader - test_item - 15 Dec 2014 - Tested successfully

zero-24 - change - 15 Dec 2014

Status

Pending

⇒

Ready to Commit

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.}

zero-24 - alter_testresult - 15 Dec 2014 - shur: Tested successfully

brianteeman - change - 15 Dec 2014

Labels

Added: ?

zero-24 - close - 17 Dec 2014

wilsonge - reference | d27dc54 - 17 Dec 14

wilsonge - merge - 17 Dec 2014

wilsonge - close - 17 Dec 2014

wilsonge - close - 17 Dec 2014

wilsonge - change - 17 Dec 2014

Status	Ready to Commit	⇒	Closed
Closed_Date	0000-00-00 00:00:00	⇒	2014-12-17 13:18:07

wilsonge - comment - 17 Dec 2014

Thanks Hannes! Merged

wilsonge - change - 17 Dec 2014

Milestone

Added:

zero-24 - change - 14 Oct 2015

Labels

Removed: ?

Hackwar - head_ref_deleted - 6 Jan 2016

Add a Comment

Login with GitHub to post a comment

Older
Newer