? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
20 Aug 2017

Summary of Changes

  1. We really shouldn't allow JHtml::_() to be extended/overridden if someone subclasses JHtml, so this makes the method final.
  2. Since the method is now final, we can typehint the $key argument with its scalar value.
  3. Now that we can allow PHP 5.6 features, lets make the JHtml::_() method signature variadic so it is now documented more efficiently that the method has more than one argument.

Testing Instructions

The CMS still works.

Documentation Changes Required

Note the B/C break in JHtml::_() being made final.

avatar joomla-cms-bot joomla-cms-bot - change - 20 Aug 2017
Category Libraries
avatar mbabker mbabker - open - 20 Aug 2017
avatar mbabker mbabker - change - 20 Aug 2017
Status New Pending
avatar wilsonge wilsonge - change - 21 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-21 03:12:31
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 21 Aug 2017
avatar wilsonge wilsonge - merge - 21 Aug 2017

Add a Comment

Login with GitHub to post a comment