? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
10 Jul 2016

Summary of Changes

The $list data array is only used if the pagination_list_footer() function exists in the template. So only build it if the function exists.

Testing Instructions

No change to how pagination is built. Sorry, off hand I don't know the trace to get this specific function to trigger.

avatar mbabker mbabker - open - 10 Jul 2016
avatar mbabker mbabker - change - 10 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Jul 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 10 Jul 2016
Category Libraries
avatar zero-24
zero-24 - comment - 10 Jul 2016

@mbabker Travis is failing can you have a look / restart https://travis-ci.org/joomla/joomla-cms/jobs/143719779

avatar bertmert bertmert - test_item - 10 Jul 2016 - Tested successfully
avatar bertmert
bertmert - comment - 10 Jul 2016

I have tested this item successfully on e3dd5bd

All paginations below list views like
administrator/index.php?option=com_content
administrator/index.php?option=com_contact
...
work correctly after patch.

After removing
/administrator/templates/isis/html/pagination.php
or deleting function pagination_list_footer(...) in it

$list stays empty and paginations are provided by getPaginationLinks() instead.


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

avatar mbabker
mbabker - comment - 10 Jul 2016

@zero-24 It's unrelated to this change and quite frankly that test is flaky if it keeps randomly failing like it has been.

avatar andrepereiradasilva andrepereiradasilva - test_item - 19 Jul 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Jul 2016

I have tested this item successfully on e3dd5bd

on code review


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

avatar mbabker mbabker - change - 20 Jul 2016
Status Pending Ready to Commit
avatar mbabker mbabker - change - 13 Aug 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 13 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-13 18:24:22
Closed_By wilsonge
avatar wilsonge wilsonge - close - 13 Aug 2016
avatar wilsonge wilsonge - merge - 13 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 13 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 13 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment