Feeling Lucky
?
[#10716] - JModelForm - add parameter group to preprocessData
- Closed
- 30 Jul 2016
- Medium
- Build: 3.7.x
- # 10716
- Diff
- csthomas:predata
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
To be more compatibility with below:
protected function preprocessForm(JForm $form, $data, $group = 'content')I have added parameter $group = 'content' to preprocessData function.
protected function preprocessData($context, &$data, $group = 'content')New JModelForm will be more flexible to use with ex. user profile.
Testing Instructions
Go to front end article/category view and check does content event loads.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
n9iels
- comment
- 17 Jun 2016
I have tested this item
Works for me, like this feature!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10716.
| Status | Pending | ⇒ | Ready to Commit |
brianteeman
- comment
- 17 Jun 2016
| Labels |
Added:
?
|
||
roland-d
- | Milestone |
Added: |
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-07-30 14:43:14 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
I have tested this item✅ successfully on af4d3b3
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10716.