[#7149] - MSSQL - Pagination fix
- Closed
- 27 Jun 2015
- Medium
- Build: staging
- # 7149
- Diff
- alikon:patch-33
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Steps to reproduce the issue
use pagination and set a limit
example go to Administration Article Manager Articles and set a limit for show only 5 articlesper page
Expected result
pagination works well show only 5 articles per page
Actual result
don't work show more articles
System information (as much as possible)
PHP Version 5.5.9
Web Server Apache/2.4.7 (Win32)
Joomla! Version Joomla! 3.4.1
Database MSSQL 2008 R2
Additional comments
fixed the process limit query on sqlsrv
SELECT * FROM (
the old $query
) A WHERE A.RowNumber BETWEEN start AND end;
i've followed these
http://blogs.msdn.com/b/sqlserver/archive/2006/10/25/limit-in-sql-server.aspx
https://technet.microsoft.com/en-us/library/ms186734.aspx
don't know about #5293 and azure
p.s
for #6414 the correct query was removed by mistake i think in #5293
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Labels |
Added:
?
|
||
| Category | ⇒ | MS SQL |
| Status | New | ⇒ | Pending |
@test OK
Issue confirmed on MS SQL. PR resolves issue.
Fixes the regression introduced by #5293 reported in #6414 !
Considering Azure works the same, it should be applied there as well.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7149.
Is there any free webhost service for testing MS SQL? I just tried somee.com but have some issues creating the ms sql Database there... :(
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7149.
i can recommend to use on a localhost the "express" edition is a free version of MSSQL
| Status | Pending | ⇒ | Ready to Commit |
RTC Thanks!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7149.
Merged - thanks!
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-06-27 00:11:35 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
This exact code was removed in #5293 - are we sure that we aren't introducing new bugs elsewhere?