Feeling Lucky
?
[#2429] - Save empty multi-value fields as empty array (instead of not at all).
- Closed
- 29 Nov 2013
- Medium
- Build: master
- # 2429
- Diff
- okonomiyaki3000:jform_store_empty_arrays
- Foreign ID 32564
- Failure The Travis CI build failed Details
User tests: Successful: Unsuccessful:
This change first of all simplifies an unnecessarily verbose if/else block. That part of the change affects only code readability, there is no change whatsoever to functionality at this point.
Then, if the field handles multiple values but no value has been set, the value is set to empty array. This way, if the field has one or more default values but the user chooses to deselect them, the field will not reselect the default values when the form is loaded again in the future.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2013-11-29 05:41:10 |
Started a new PR to staging instead of master. Closing.