User tests: Successful: Unsuccessful:
This pull fix the issue when a couple JPagination instances used.
Test
Run next code:
$p1 = new JPagination(10, 0, 5, 'pref1');
$data1 = $p1->getData();
$p2 = new JPagination(20, 0, 10, 'pref2');
$data2 = $p2->getData();
echo '<pre>';
print_r($data1->next);
print_r($data2->next);
echo '</pre>';
Result (before patch):
JPaginationObject Object
(
[text] => Next
[base] => 5
[link] => /jdev/?pref1limitstart=5
[prefix] => pref1
[active] =>
)
JPaginationObject Object
(
[text] => Next
[base] => 5
[link] => /jdev/?pref1limitstart=5
[prefix] => pref1
[active] =>
)
Expected result:
JPaginationObject Object
(
[text] => Next
[base] => 5
[link] => /jdev/?pref1limitstart=5
[prefix] => pref1
[active] =>
)
JPaginationObject Object
(
[text] => Next
[base] => 10
[link] => /jdev/?pref2limitstart=10
[prefix] => pref2
[active] =>
)
Labels |
Added:
?
|
Title |
|
@test It work!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4521.
Category | ⇒ | Libraries |
Tested successfully(chrome)
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4521.
Multiple good tests setting to RTc - thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4521.
Status | Pending | ⇒ | Ready to Commit |
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-11-08 13:50:42 |
Merged, thanks.
Solved a conflict in pagination.php
manually.
Hi there,
this bugfix saved me a lot of work. But why isn't it on the latest Joomla release? I'm "up to date" and had to change the pagination.php manually.
It would be great if you could get this fix into the next Joomla release :-)
As you can see this was merged on Nov 8 and the last release was made 1st October. http://www.joomla.org/announcements/release-news/5569-joomla-3-3-6-released.html
So it will be in the next release (3.4) which hopefully is not too long away
Oh I did not see this. Alright, thanks a lot :-)
@Fedik See Fedik#1