?
avatar toperf
toperf
14 Aug 2014

Steps to reproduce the issue

Deleting on a JModelAdmin with an SqlSrv database, the returned query fails because it tries to "limit" the delete query

Expected result

DELETE FROM CommercialOffer WHERE [id] = '15'

Actual result

SELECT * FROM ( DELETE , ROW_NUMBER() OVER (ORDER BY (select 0)) AS RowNumber FROM CommercialOffer WHERE [id] = '15') _myResults WHERE RowNumber BETWEEN 1 AND 0

System information (as much as possible)

Joomla! 3.3.3

Additional comments

On libraries\joomla\database\query.php I put a "return $query;" on line 411, which looks to have made the trick. But I feel the other "cases" should be looked too as by nature the update should not have a "limit". Although I have encountered no problems with updates as it is.

avatar perfeito perfeito - open - 14 Aug 2014
avatar brianteeman brianteeman - change - 2 Sep 2014
Category MS SQL SQL
avatar nicksavov nicksavov - change - 16 Oct 2014
Labels Added: ?
avatar brianteeman brianteeman - change - 1 Jan 2015
Labels Removed: ?
avatar brianteeman brianteeman - change - 3 Jan 2015
Labels Added: ?
avatar roland-d
roland-d - comment - 8 Nov 2015

Hello @toperf

Thank you for your contribution.

The last comment here was on August 14 2014. So the question is, Is this issue/pull request still valid?
If no reply is received within 4 weeks we will close this issue.

Thanks for understanding!


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

avatar roland-d roland-d - change - 8 Nov 2015
Status New Information Required
avatar brianteeman brianteeman - change - 17 Dec 2015
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2015-12-17 11:11:59
Closed_By brianteeman
avatar brianteeman brianteeman - close - 17 Dec 2015
avatar brianteeman brianteeman - close - 17 Dec 2015
avatar brianteeman
brianteeman - comment - 17 Dec 2015

Closed due to lack of response for further information - it can always be reopened if that is provided


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

Add a Comment

Login with GitHub to post a comment