Feeling Lucky
[#46381] - [6.0] Remove default empty string param in addControlField()
- Pending
- Medium
- Build: 6.0-dev
- # 46381
- Diff
- Denitz:remove-addControlField-defaults
User tests: Successful: Unsuccessful:
Summary of Changes
New addControlField() method has $value as empty string by default, no sense to use empty string in method calls.
Testing Instructions
Apply patch.
Actual result BEFORE applying this Pull Request
useless default param in code
Expected result AFTER applying this Pull Request
No changes.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_associations com_banners com_cache com_categories com_checkin com_config com_contact com_content com_fields com_finder com_installer |
richard67
- comment
- 24 Nov 2025
Such bigger refactoring should not be done in a patch version, so this PR would have to be made for 6.1-dev, I think.
@softforge @Bodge-IT Please check and decide.
HLeithner
- | Title |
|
||||||
I have tested this item ✅ successfully on 1fbc46d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46381.