[#37460] - [4.2] Enable use of Form::process() in FormModel
- Fixed in Code Base
- 9 Apr 2022
- Medium
- Build: 4.2-dev
- # 37460
- Diff
- Fedik:form-process
User tests: Successful: Unsuccessful:
Summary of Changes
A couple years ago @Hackwar added a cool feature #12414, which allows to run Field::postProcessing().
However it was not enabled in our FormModel class, and so never used.
This PR enable this feature for FormModel.
Testing Instructions
Apply patch, edit any content, and make sure all works.
Then edit
joomla-cms/libraries/src/Form/FormField.php
Lines 1305 to 1308 in 05d3b14
Add dd('Works!'); before return. And try to save any content again.
Actual result BEFORE applying this Pull Request
Content saved, that means postProcess() was ignored.
Expected result AFTER applying this Pull Request
You got white page with Works! message, that means postProcess() was called.
Documentation Changes Required
As part of #12414
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37460.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37460.
Quy
- | Labels |
Added:
?
|
||
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-04-09 19:51:31 |
| Closed_By | ⇒ | roland-d |
I have tested this item✅ successfully on adaa0b6
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37460.