Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#2243] - Replace '#__' with actual table prefix in getQuery function

? Success

User tests: Successful: Unsuccessful:

avatar gunjanpatel
gunjanpatel
14 Oct 2013
avatar gunjanpatel gunjanpatel - open - 14 Oct 2013
avatar brianteeman
brianteeman - comment - 14 Oct 2013

Please can you add a link to the related joomlacode item

avatar gunjanpatel
gunjanpatel - comment - 14 Oct 2013

Yes, I was creating tracker for the same. :smiley:

Here is a link to the tracker item http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=32301&start=0

avatar beat
beat - comment - 14 Oct 2013

This is imho a B/C (backwards-compatibility) breaking of API and should not be implemented.

avatar betweenbrain
betweenbrain - comment - 14 Oct 2013

I agree with @beat . What is the issue that this is solving?

avatar mbabker
mbabker - comment - 14 Oct 2013

The change you're proposing is a potentially major break in the API. Why not just call the code you're using in this PR when you need the query with the correct prefix?

avatar gunjanpatel
gunjanpatel - comment - 15 Oct 2013

Thank you guys for review.
As per @elinw suggestion, I should use $query->dump();
I am closing this PR.
Thanks again for review it. :bow:

avatar gunjanpatel gunjanpatel - close - 15 Oct 2013
avatar gunjanpatel gunjanpatel - head_ref_deleted - 15 Oct 2013

Add a Comment

Login with GitHub to post a comment