?
?
Success
-
Success
continuous-integration/drone
the build was successful
Details
-
Success
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
-
Success
continuous-integration/appveyor/pr
AppVeyor build succeeded
Details
User tests:
Successful:
Unsuccessful:
It is a part of the other more complicated PR #13262
Summary of Changes
Better workaround for sql query with offset
and limit
.
- do not create sub query if there is only
limit
applied, use SELECT TOP
Testing Instructions
Apply patch on installed joomla (on mssql db) with sample data.
Go to article list on backend and check whether pagination works.
Expected result
Works as before,
- start to work in conjunction with PR #13854 at administrator/index.php?option=com_finder&view=maps
Actual result
- sql error at administrator/index.php?option=com_finder&view=maps
- and does not work in conjunction with PR #13854
Documentation Changes Required
No
csthomas
-
open
-
3 Feb 2017
csthomas
-
edited
-
5 Feb 2017
joomla-cms-bot
-
change
-
5 Feb 2017
Category |
|
⇒ |
MS SQL
Libraries
Unit Tests
|
csthomas
-
change
-
5 Feb 2017
The description was changed
Labels |
Added:
?
?
|
csthomas
-
change
-
11 Feb 2017
Status |
New |
⇒ |
Closed |
Closed_Date |
0000-00-00 00:00:00 |
⇒ |
2017-02-11 12:50:49 |
Closed_By |
|
⇒ |
csthomas |
csthomas
-
close
-
11 Feb 2017
Add a Comment
Login with GitHub to post a comment
I close it in favour of #13262