? ? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
5 Aug 2016

There is a parameter in the menu item Tags:Tagged Items, item selection options called "maximum"

This shouldnt be there at all as there is no code to use the param. It looks like a siple copy paste error from another tag menu item

Summary of Changes

This simple PR removes the option
Even if a user had set a value here there is no b/c issue as there was no code that would ever use the value

avatar joomla-cms-bot joomla-cms-bot - change - 5 Aug 2016
Category Tags Front End Components
avatar brianteeman brianteeman - open - 5 Aug 2016
avatar brianteeman brianteeman - change - 5 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Aug 2016
Labels Added: ?
avatar alikon alikon - test_item - 6 Aug 2016 - Tested successfully
avatar alikon
alikon - comment - 6 Aug 2016

I have tested this item successfully on 25f3cbe


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

avatar jeckodevelopment jeckodevelopment - test_item - 6 Aug 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 6 Aug 2016

I have tested this item successfully on 25f3cbe


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

avatar brianteeman
brianteeman - comment - 6 Aug 2016

@alikon I did them as seperate pr as they were wrong for different reasons and I wanted to keep them seperate in case I had missed some weird way they were being used.


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

avatar brianteeman brianteeman - change - 6 Aug 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 6 Aug 2016

Rtc


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

avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - close - 13 Aug 2016
avatar rdeutz rdeutz - merge - 13 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 13 Aug 2016
avatar rdeutz rdeutz - change - 13 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-13 18:05:56
Closed_By rdeutz

Add a Comment

Login with GitHub to post a comment