? ? Pending

User tests: Successful: Unsuccessful:

avatar nibra
nibra
17 Mar 2017

Pull Request for Issue #13382 .

Summary of Changes

Added test for that issue, demonstrating how to use JFormHelper::loadFieldClass() properly until JForm fields are covered by the autoloader.

Testing Instructions

$ ./libraries/vendor/bin/phpunit tests/unit/suites/issues/Issue13382Test

Documentation Changes Required

Since the form fields do not work with JLoader (due to the folder name being form/fields instead of form/field), the JForm field class must be loaded manually, if (and only then) the filename of a field in any additional field path matches the filename of core field. In the case from the original issue, the field class should look like

<?php
require_once JPATH_LIBRARIES . '/joomla/form/fields/radio.php';

class TestFormFieldRadio extends JFormFieldRadio
{

}
avatar nibra nibra - open - 17 Mar 2017
avatar nibra nibra - change - 17 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Mar 2017
Category Unit Tests
avatar nibra nibra - change - 17 Mar 2017
Labels Added: ? ?
avatar brianteeman brianteeman - change - 9 Aug 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-08-09 13:44:05
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 9 Aug 2018

This is going to have to be closed as the tests are no longer part of this repo. Please submit it to the tests repo

avatar brianteeman brianteeman - close - 9 Aug 2018

Add a Comment

Login with GitHub to post a comment