Labels |
Added:
?
|
Category | ⇒ | com_search |
Status | New | ⇒ | Discussion |
Status | Discussion | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-10 19:18:02 |
Closed_By | ⇒ | alikon |
As class names are not case sensitive in PHP and we have a convention that form files do exist in lowercase name this code loads the class in < 3.8. Now it is different, so we need to find a way to have the same set up. I will check it in the next few days for a proper solution. Till then I suggest to close the pr #17477 and reopen this one and mark it as release blocker.
That line is only problematic if you're scanning the filesystem to find a not-yet-loaded class, it won't be hit if something hits the autoloader first. What we have in the class mapping file won't work for these fields because their properly declared cases aren't anything that were ever autoloaded to begin with, so for form fields we might need to just uppercase the first character after JFormField
and leave everything after lowercased.
Labels |
Added:
?
|
reopen as requested
Status | Closed | ⇒ | New |
Closed_Date | 2017-08-10 19:18:02 | ⇒ | |
Closed_By | alikon | ⇒ |
Status | New | ⇒ | Discussion |
Status | Discussion | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-17 14:04:01 |
Closed_By | ⇒ | franz-wohlkoenig |
Closed_By | franz-wohlkoenig | ⇒ | joomla-cms-bot |
closed as having Pull Request #17584
Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/17460
Labels |
Removed:
?
|
Can't confirm. Search Tools are shown and working as expected.
System information
3.8-dev
Multilanguage Site
macOS Sierra, 10.12.6
Firefox 55 (64-bit)
MAMP 4.1.1