Remove submitted values from data if field is disabled#25682
Remove submitted values from data if field is disabled#25682HLeithner merged 5 commits intojoomla:stagingfrom
Conversation
|
I have tested this item ✅ successfully on 4bb2e3a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25682. |
1 similar comment
|
I have tested this item ✅ successfully on 4bb2e3a This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25682. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25682. |
|
@laoneo can you have a look at this because i'm not quite confident with this. |
|
Me neither, pretty sure this will introduce a regression. |
|
@laoneo Could you please describe in more detail why this would introduce a regression? |
|
@laoneo @HLeithner I have restored the previous state, as I do not really fully understand when a default value is needed. The only adjustment now concerns what, according to the description, anyway should not be. |
|
@degobbis can't say exactly what, it is just so a feeling. It took a couple of iterations till we were on a stable state with all these readonly, disabled states and access level settings including the passed array. What I can remember is that we had issues with values not being present in the data array as they were needed to clear values, etc. |
|
@laoneo However, we are currently receiving an error message because the values from disabled fields are still sent. With my fix, I'm only making sure that the described procedure of the Bowser is actually executed. Either way, the same result. |
|
@gogicomputers @viocassel Could You please test my changes again? |
|
I have tested this item ✅ successfully on 174136c This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25682. |
1 similar comment
|
I have tested this item ✅ successfully on 174136c This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25682. |
|
Status "Ready To Commit". Thanks for tests. |
|
thx |
Pull Request for Issue #25652 .
Testing Instructions
see #25628