Task #1512
closedFeature approve terms block doesn't send value
100%
Description
EAV detail can't be saved because approve terms need be checked and after check this renderer doesn't return value to form.
This is probably block for project that use this feature.
Affected version 2.x.x
In version before 2.x.x isn't this problem because this renderer doesn't exist. Probably best solution for this is return value from the renderer AgreeCheckboxRenderer
Files
Updated by Ondřej Kopr over 5 years ago
- Status changed from New to In Progress
- Assignee changed from Peter Štrunc to Ondřej Kopr
- Target version set to 2.2.0
Updated by Ondřej Kopr over 5 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Ondřej Kopr to Radek Tomiška
- % Done changed from 0 to 90
I update renderer AgreeCheckboxRenderer with same behavior as BooleanFormAttributeRenderer. Now it works correctly. The changes was tested on verison 9.3.4, 9.4.0 and 9.6.0-SNAPSHOT.
Please Radek could you make a review? We can do review together and check the new behavior on my local environment.
Commit: https://git.bcvsolutions.eu/modules/reg/commit/b2b5d4f77a37abab2cd1d08d2ca0d2bd89bd17c6 (branch: develop)
Updated by Radek Tomiška over 5 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Radek Tomiška to Ondřej Kopr
- % Done changed from 90 to 100
I did test and code review, it works, thx!
Note: I installed module on existing environment, where previous version of this module was installed. I fixed some issues unrelated with this ticket - but i think backward compatibility not work correctly anyway (new form is not inited completelly, password field is required in each update ...) :-/
https://git.bcvsolutions.eu/modules/reg/commit/0c4b2476ff7d3dc6c224de3bbfc987a9696ed471
Updated by Ondřej Kopr over 5 years ago
Radek Tomiška wrote:
I did test and code review, it works, thx!
Note: I installed module on existing environment, where previous version of this module was installed. I fixed some issues unrelated with this ticket - but i think backward compatibility not work correctly anyway (new form is not inited completelly, password field is required in each update ...) :-/
https://git.bcvsolutions.eu/modules/reg/commit/0c4b2476ff7d3dc6c224de3bbfc987a9696ed471
For update behavior with password fields was created ticket #1677