? Pending

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
22 Aug 2016

Pull Request for Issue #11714 (part 2).

Summary of Changes

Only load the plg_content_pagebreak and plg_content_vote languages files when they are needed.
Example: don't load them if the show_vote is not active or there is no pagebreak class.

For more info see #11714

Testing Instructions

Reproduce

  • Use latest staging
  • Enable plugin content vote and plugin content pagebreak
  • Enable debug and debug lang in global config
  • Enable debug system plugin with all options activated
  • Go to frontend and check the debug console
  • Notice JROOT/administrator/language/en-GB/en-GB.plg_content_vote.ini and JROOT/administrator/language/en-GB/en-GB.plg_content_pagebreak.ini language file is loaded no matter what page you are (even in backend pages they are loaded).

image

Test patch

  • Apply patch
  • Repeat the steps above and checkthe two language files are NOT loaded (except when needed).
  • Check plugin work as before if language files are required to load (ex: show_vote enabled / article with pagebreak, etc).

Documentation Changes Required

None.

avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Category Plugins Front End
avatar andrepereiradasilva andrepereiradasilva - open - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Category Plugins Front End Libraries Plugins Front End
avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Aug 2016

@mbabker i updated this PR to follow the logic you added for modules in #11698, but this time for the plugins.

Can you check if all ok?

avatar mbabker
mbabker - comment - 22 Aug 2016

Looks fine.

avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Aug 2016

thanks

avatar alikon alikon - test_item - 22 Aug 2016 - Tested successfully
avatar alikon
alikon - comment - 22 Aug 2016

I have tested this item successfully on 3b028a3


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

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

I have tested this item successfully on 3b028a3


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

avatar jeckodevelopment jeckodevelopment - change - 22 Aug 2016
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 22 Aug 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 25 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-25 20:24:02
Closed_By rdeutz
avatar rdeutz rdeutz - close - 25 Aug 2016
avatar rdeutz rdeutz - merge - 25 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 25 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment