Change JATS document creation to a DOM based model

[asmecher]

@ipula and @withanage, this PR replaces https://github.com/pkp/jatsTemplate/pull/26.

It also adds...

• https://github.com/pkp/jatsTemplate/commit/7dd456e36cf3a4179a3a4f47f0776bb648229a80 for https://github.com/pkp/pkp-lib/issues/7916
• https://github.com/pkp/jatsTemplate/commit/d8c40e8c27f198e749cc4a782699e083306925fa for https://github.com/pkp/pkp-lib/issues/9205
• https://github.com/pkp/jatsTemplate/commit/603d9f82212adf0d228aca1ef3b6f8a14c686aeb:
  • Improvements to readability (less use of ->parentNode; placement of attributes first; etc)
  • Resolves https://github.com/pkp/jatsTemplate/pull/26#discussion_r1264263222
  • Resolves https://github.com/pkp/jatsTemplate/pull/26#discussion_r1264257142
  • Resolves https://github.com/pkp/jatsTemplate/pull/26#discussion_r1264262865 about htmlspecialchars / escaping
  • Removes "version-neutral" aspects of the code -- this is intended for release with 3.5 only.
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R379: meta-value should not be nested in meta-name
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R262: $submission->getLocale() should never be null
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R327: $ccBadge misplaced
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R340: $ccBadge misplaced
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R394: appendChild applied to wrong element; sections data not included in DOM
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R462: Author ORCiD should not appear here
  • https://github.com/pkp/jatsTemplate/pull/26/files#diff-11e457ccd0f48a0b36bf17dbf4eb71477dc3b75ddf8b9be8595fea6fe3516fb3R347: kwd should not have xml:lang attribute
  • ...and similar.

Some of these are trickier to catch in review than they are while coding! It pays to work on this kind of code progressively -- modify, test, rinse and repeat, with data in your installation that exercises each piece of code you touch. Be careful with copy/paste; I think a number of weird typos came in that way.

[withanage]

Thanks @asmecher for review!

[asmecher]

Sorry :fearful: I forgot to add! @ipula and @withanage, thanks for this, it's always bothered me that we've been building XML with string concatenation. It's finicky work and I really appreciate your willingness to rearrange it painfully based on review feedback!