? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
28 Aug 2016

Summary of Changes

When the application renders the template, it injects the template parameters into the JDocument instance. However, only JDocumentHtml actually stores a parameters reference. Since our templates are in scope of JDocument (aside from the error.php layout all of them are in scope of JDocumentHtml), the template should reference the injected parameters instead of issuing another call to JFactory to fetch the data. By doing so, this breaks parameter injection by not using the injected data.

Testing Instructions

The Beez3 template should continue to work with its configured parameters.

Documentation Changes Required

It should be made clear what data is made available when a template file is included in the scope of JDocumentHtml and the examples updated to support this practice.

avatar mbabker mbabker - open - 28 Aug 2016
avatar mbabker mbabker - change - 28 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2016
Category Templates (site) Front End
avatar bhavikTailored
bhavikTailored - comment - 29 Aug 2016
avatar bhavikTailored bhavikTailored - test_item - 29 Aug 2016 - Tested successfully
avatar bhavikTailored
bhavikTailored - comment - 29 Aug 2016

I have tested this item successfully on f5201fb


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

avatar RonakParmar RonakParmar - test_item - 29 Aug 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 29 Aug 2016

I have tested this item successfully on f5201fb


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

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

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 29 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-29 17:28:14
Closed_By rdeutz
avatar rdeutz rdeutz - close - 29 Aug 2016
avatar rdeutz rdeutz - merge - 29 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 29 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment