Refactor validation and section selection in QuickSubmitForm

[evici]

Refactor issue validation and section selection logic in QuickSubmitForm.php. Improved error handling and streamlined the process for selecting sections and managing user groups.

[asmecher]

@evici, is there a particular problem that this solves, or is it just a rearrangement of the code?

[evici]

@evici, is there a particular problem that this solves, or is it just a rearrangement of the code?

In multi-journal setup we do not have manager users as such it was required to have by plugin logic. In those situations we had err 500 and could not work with QuickSubmit.

This fix applies "sanity check" on usergroups in journals.

[asmecher]

Thanks, that helps. I see the additional fallbacks on Role::ROLE_ID_SUB_EDITOR, followed by any role at all.

I agree that the Role::ROLE_ID_SUB_EDITOR fallback is a good idea, but I'm not sure about the last fallback; it could be something created by the Journal Manager that is completely unrelated to their work with QuickSubmit.

Could you adjust this PR so that it only includes the change that adds the Role::ROLE_ID_SUB_EDITOR fallback?