PR-5.3-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar QuyTon
QuyTon
26 Jan 2025

Testing Instructions

Code review.

/**
* The name of the form field direction (ltr or rtl).
*
* @var string
* @since 3.2
*/
protected $dirname;

Syntax should be name.dir but incorrectly changed in 3f4285e

avatar QuyTon QuyTon - open - 26 Jan 2025
avatar QuyTon QuyTon - change - 26 Jan 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Jan 2025
Category Layout
avatar QuyTon QuyTon - change - 26 Jan 2025
Labels Added: PR-5.3-dev
avatar brianteeman
brianteeman - comment - 26 Jan 2025

I dont think this is correct. see

// Set the dirname.
$dirname = ($dirname === 'dirname' || $dirname === 'true' || $dirname === '1');
$this->dirname = $dirname ? $this->getName($this->fieldname . '_dir') : false;

avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2025
Category Layout Layout Libraries
avatar QuyTon QuyTon - change - 3 Feb 2025
The description was changed
avatar QuyTon QuyTon - edited - 3 Feb 2025
avatar QuyTon QuyTon - change - 3 Feb 2025
The description was changed
avatar QuyTon QuyTon - edited - 3 Feb 2025

Add a Comment

Login with GitHub to post a comment