? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
9 Sep 2019

Summary of Changes

#26223 by @SharkyKZ and merged (without tests) by @wilsonge introduced at least one error

Testing Instructions

administrator/index.php?option=com_finder&view=maps

Expected result

list of maps

Actual result

syntax error in the query

Summary

The original query was

$query->select([$query->qn('b.id', 'branch_id'), $query->qn('b.title', 'branch_title')])
			->leftJoin($query->qn('#__finder_taxonomy', 'b') . ' ON b.level = 1 AND b.lft <= a.lft AND a.rgt <= b.rgt');

This was changed to

$query->select([$db->quote('b.id', 'branch_id'), $db->quote('b.title', 'branch_title')])
			->leftJoin($db->quote('#__finder_taxonomy', 'b') . ' ON b.level = 1 AND b.lft <= a.lft AND a.rgt <= b.rgt');

Which is obviously incorrect as qn==> quoteName not quote

This PR fixes that

avatar brianteeman brianteeman - open - 9 Sep 2019
avatar brianteeman brianteeman - change - 9 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2019
Category Administration com_finder
avatar SharkyKZ SharkyKZ - test_item - 10 Sep 2019 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 10 Sep 2019

I have tested this item successfully on 38ed7e5


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

avatar wilsonge wilsonge - change - 10 Sep 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-09-10 12:14:52
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 10 Sep 2019
avatar wilsonge wilsonge - merge - 10 Sep 2019
avatar wilsonge
wilsonge - comment - 10 Sep 2019

Thanks!

avatar brianteeman
brianteeman - comment - 10 Sep 2019

Thanks

Add a Comment

Login with GitHub to post a comment