? Success

User tests: Successful: Unsuccessful:

avatar phproberto
phproberto
20 Feb 2015

In the catefories view the parent div was created in a layout. That's a bad practice that can cause unexpected issues when overriding views.

Test

Ensure that all the categories views are showed correctly. The PR includes articles, newsfeeds & contact categories because they were the only calling that layout but if you find more test it there please. Also in the menu item of each category view add a "Page class" in the "Page display" tab and ensure that is applied correctly.

Backward compatibility issues

This should only cause issues if someone is already using that layout. In that case they may have created a a dirty fix to close the orphan div. But this is a bug so we should fix it.

avatar phproberto phproberto - open - 20 Feb 2015
avatar joomla-cms-bot joomla-cms-bot - change - 20 Feb 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 20 Feb 2015
Easy No Yes
avatar zero-24 zero-24 - change - 21 Feb 2015
Category Layout
avatar zero-24 zero-24 - change - 21 Feb 2015
Category Layout Administration Layout
avatar bertmert
bertmert - comment - 10 Mar 2015

@test Success

avatar zero-24 zero-24 - alter_testresult - 10 Mar 2015 - bertmert: Tested successfully
avatar designbengel designbengel - test_item - 14 Mar 2015 - Tested successfully
avatar continga
continga - comment - 14 Mar 2015

Test OK

avatar brianteeman brianteeman - change - 14 Mar 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 14 Mar 2015
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 14 Mar 2015

Thanks for testing - RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6134.
avatar phproberto phproberto - change - 15 Mar 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-03-15 09:15:33
avatar phproberto phproberto - close - 15 Mar 2015
avatar phproberto phproberto - close - 15 Mar 2015
avatar phproberto
phproberto - comment - 15 Mar 2015

Merged. Thanks!

avatar phproberto phproberto - head_ref_deleted - 15 Mar 2015

Add a Comment

Login with GitHub to post a comment