Feeling Lucky
[#15373] - Removed "readonly" and "disabled" fields parameters
- Fixed in Code Base
- 18 Apr 2017
- Medium
- Build: staging
- Easy Test
- # 15373
- Diff
- Bakual:RemoveReadonlyDisabled
User tests: Successful: Unsuccessful:
Pull Request for Issue #15369 .
Summary of Changes
This PR proposes to remove the "disabled" and "readonly" from the fields parameters as they don't make any sense for a field.
Testing Instructions
- Create a new field or save an existing one again so the readonly/disabled parameters aren't stored anymore.
- Test that the field works as expected.
- Test especially also that ACL restrictions still work, eg a user which isn't allowed to edit a value has a readonly/disabeld field.
Documentation Changes Required
Maybe if documentation is already written with those parameters mentioned.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_fields Language & Strings |
| Easy | No | ⇒ | Yes |
brianteeman
- comment
- 18 Apr 2017
Note to whoever merges. If this is merged for 3.7.0 (which it should) then its fine but if it waits until 3.7.1 then the language strings will have to be kept until 4.0 :(
rdeutz
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-04-18 13:23:26 |
| Closed_By | ⇒ | rdeutz |
I have tested this item✅ successfully on 115a27d
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15373.