Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#12762] - fixed issue #12761. function total throws SQL exception in SQL Server

? Success

User tests: Successful: Unsuccessful:

avatar freegink
freegink
5 Nov 2016

Pull Request for Issue #12761 .

Summary of Changes

removed parameter values for "limit" and "offset" passed in "setQuery" in function "total"

Testing Instructions

  1. Go to "Menu->Mange"
  2. Select a menu
  3. Click "Edit" button

The editing interface should appear without SQL exception.

Documentation Changes Required

N/A

avatar freegink freegink - open - 5 Nov 2016
avatar freegink freegink - change - 5 Nov 2016
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Nov 2016
Category ⇒ Libraries
avatar phproberto
phproberto - comment - 5 Nov 2016

I'm ok with the fix because it's not really something that will make a performance improvement if you don't have millions of rows and if you have millions of user groups you have a bigger problem.

That said you are not fixing an issue. You are bypassing it. This query is correct in MySQL:

SELECT count(id)
FROM jml_usergroups
LIMIT 0,1

If the SQL Server driver is not working properly fix it or this will happen again.

avatar freegink
freegink - comment - 6 Nov 2016

You are right. The driver should be fixed.

avatar freegink freegink - change - 6 Nov 2016
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-11-06 03:13:43
Closed_By ⇒ freegink
avatar freegink freegink - close - 6 Nov 2016
avatar Tutunka
Tutunka - comment - 31 Mar 2017

Searched all over google for this fix so thanks for the easy solution.
I installed an older version back in December 2016 and gave up due to this bug and just today updated Joomla to the latest version so this fix definitely isn't part of the new files. Can it be added to the project release files? I'm moving DNN and Prestashop due to bugs that simply don't get fixed so this isn't really a great start with Joomla.

Add a Comment

Login with GitHub to post a comment