Required fields in form are still required even if field/group is hidden about pkp-lib HOT 7 CLOSED

@ewhanson This should resolve the workaround you are having in OrcidForm.vue. What you think?

ui-library: pkp/ui-library#357

from pkp-lib.

Hey @jardakotesovec, thanks for this! I think this gets most of the way to solving my issues. It is possible to save a form with conditionally hidden required elements blank, but if there are previous errors, they are not dismissed when the fields are hidden (see video).

from pkp-lib.

@ewhanson Ok.. additionally added removing error on field unmount, which can be think about as clean up for field which is not relevant anymore.

At this point I think this should work well.. can't think of where it could cause problems at this point.. but if you can think of something please let me know!

from pkp-lib.

Thanks @jardakotesovec! This is most of the way there now, but I'm still running into an error. It seems that only one error is removed at a time when the components are unmounted. I have an example where there are 3 "required fields not completed" errors. And each time I hide/unmount the components, another one of the errors is removed.

from pkp-lib.

@ewhanson Great catch, thanks! I did not test it with more than one field, so I did not notice it.

It went wrong, because how the state management is structured currently there. So every removing error created new errors object with that one error removed. But the errors object prop don't get propagated between these calls.. so all of them were modifying the original object and basically last change win.

Not fan of the solution I end up with.. hence the long comment in the code.. but it does work and I still think that this is how it should behave and it should remove the errors if the field gets removed.

from pkp-lib.

Hey @jardakotesovec, that makes sense. I've done some testing with the latest changes and it's working as expected now. 👍

from pkp-lib.

@ewhanson Thank you, merged.

from pkp-lib.