Feeling Lucky
?
[#17148] - [4.0] Initial form fields namespaces
- Closed
- 18 Jul 2017
- Medium
- Build: 3.8-dev
- # 17148
- Diff
- fastslack:fields-namespace
User tests: Successful: Unsuccessful:
Summary of Changes
This PR pretend to replace old fields classes to namespaces
Testing Instructions
Install and run Joomla! normaly without issues in forms
Expected result
All work as usual
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Title |
|
||||||
| Labels |
Added:
?
|
||
mbabker
- comment
- 17 Jul 2017
The two in libraries/legacy are fine to do for 3.8. But the rest in libraries/joomla have to wait for 4.0 since the directory they are in now is not autoloaded and this move would make the classes autoloaded and we've already run into problems with duplicated classes from the core field offerings.
| Title |
|
||||||
laoneo
- comment
- 17 Jul 2017
You can change the base branch of the pr. What you need to do then is to merge it with the 4.0-dev branch.
fastslack
- comment
- 18 Jul 2017
Hmm better to close this and open 2 PR separately
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-07-18 03:20:49 |
| Closed_By | ⇒ | fastslack |
| Category | Libraries | ⇒ | Unit Tests Administration com_admin SQL Postgresql MS SQL com_banners com_categories com_config com_contact com_content com_fields com_finder com_joomlaupdate com_menus |
According to the discussion in Krakow with @mbabker we can't namespace the fields in libraries/joomla in the 3 series, because of autoloading. So this pr should be targeted against the 4.0-dev branch.