[OMP] Add option to disable validation in Onix export plugin

[kaitlinnewson]

Currently validation is always enforced in the Onix export plugin, but we want to add the option to disable it in order to ensure that the export can still be done when there may be metadata gaps.

This currently exists in some other plugins, such as the DOAJ Export plugin.

What application are you using?
OMP 3.3/3.4/main

Additional information

Example from the DOAJ plugin:

[kaitlinnewson]

PRs for review, and will be forward-ported once approved:

• OMP 3_3_0: pkp/pkp-lib#10132 add checkbox for validation in Onix export plugin omp#1624
• OMP 3_4_0: pkp/pkp-lib#10132 add checkbox for validation in Onix export plugin omp#1640
• pkp-lib 3_4_0: pkp/pkp-lib#10132 add checkbox for validation in Onix export plugin #10218
• OMP main: pkp/pkp-lib#10132 add checkbox for validation in Onix export plugin omp#1641
• pkp-lib main: pkp/pkp-lib#10132 add checkbox for validation in Onix export plugin #10219

PRs for locale key changes (key from OJS to pkp-lib):

• OJS 3_3_0: pkp/pkp-lib#10132 move plugin validation locale key for use in omp ojs#4361
• OJS 3_4_0: pkp/pkp-lib#10132 move plugin validation locale key for use in omp ojs#4359
• OJS main: pkp/pkp-lib#10132 move plugin validation locale key for use in omp ojs#4360
• pkp-lib 3_3_0: pkp/pkp-lib#10132 move plugin validation locale key for use in omp #10185
• pkp-lib 3_4_0: pkp/pkp-lib#10132 move plugin validation locale key for use in omp #10183
• pkp-lib main: pkp/pkp-lib#10132 move plugin validation locale key for use in omp #10184

If you want to force validation to break in the exports, you can replace the xsd schema files in plugins/importexport/onix30 with the latest versions (3.0.8).

@bozana would you be able to review?

One thing I had a question about is in the locale files. I copied over a locale key that is currently in OJS only - does it make sense to move this key into pkp-lib, or are duplicate keys in OJS and OMP acceptable? If duplicates are okay, then I can copy over the translations as well.

[bozana]

Hi @kaitlinnewson, I was traveling but am now back and will take a look at this soon... Thanks!

[kaitlinnewson]

Thanks @bozana - I'm looking at moving the locale key to pkp-lib so not ready to merge yet, but if you have time to review that would be great

[kaitlinnewson]

@bozana I've made the locale key changes and added them to the list of PRs above, ready for review on OMP 3_3_0

[bozana]

Hi @kaitlinnewson, great! All looks good.
When we make bigger changes to the locale keys, we ask Alec to copy the newest translations from Weblate -- so that there are no conflicts when introducing the new changes.
@asmecher, could you copy the newest translations to OJS and PKP-LIB stable-3_3_0, stable-3_4_0 and main?
@kaitlinnewson, you would then need to rebase and then we can merge.
Thanks a lot!

[bozana]

@kaitlinnewson, @asmecher has updated all locales. Could you please rebase? Then we will merge.
Thanks a lot!

[kaitlinnewson]

@bozana rebase is done for 3_4_0 and main (no changes in 3_3_0) - let me know if you see any issues as I haven't used rebasing very much before.

Working on forward-porting now and will update when that's done.

[bozana]

Hi @kaitlinnewson, I have just merged everything from here. Would you now like to port OMP changes to stable-3_4_0 and main?

[kaitlinnewson]

@bozana I've ported to 3_4_0 and main and added them to the list above. The PRs are slightly different from the changes in 3_3_0 as some things were moved to being handled in pkp-lib between the versions.

[bozana]

All merged, thus closing...