pkp / pkp-lib

The library used by PKP's applications OJS, OMP and OPS, open source software for scholarly publishing.
https://pkp.sfu.ca
GNU General Public License v3.0
304 stars 444 forks source link

Improve DublinCoreMetaPlugin #8406

Closed bozana closed 1 year ago

bozana commented 1 year ago

There are a few things that should be considered in the DublinCoreMetaPlugin:

bozana commented 1 year ago

@asmecher, should the DC meta elements be displayed on the galley i.e. file view page in OJS, as this is the case in OMP? Or should the OMP be change so that the DC elements are only displayed on the book/chapter view page (as this is the case for GoogleScholarPlugin)? Currently, in OMP, the only difference between the DC elements on the submission and representation/file view page are the elements DC.Identifier and DC.Identifier.URI. If we would like to keep the DC elements on both pages (submission and representation), then eventually also DC.Format , DC.Identifier.DOI, DC.Language and DC.Type could be changed to only display values of that particular representation? Something else? :thinking:

EDIT: And one more question: what is 'sponsor' here https://github.com/pkp/ojs/blob/main/plugins/generic/dublinCoreMeta/DublinCoreMetaPlugin.php#L79? 'Supporting Agencies' i.e. 'submissionAgency' in DB table controlled_vocabs?

bozana commented 1 year ago

This Issue changes/improves the following: OJS and OMP:

additionally in OMP:

The way the subjects and keywords are get is also changed in GoogleScholar plugin

bozana commented 1 year ago

PRs:

bozana commented 1 year ago

Hi @asmecher, For the representation i.e. submission file view pages I left everything as it was. This I can approach later, if needed. Else s. https://github.com/pkp/pkp-lib/issues/8406#issuecomment-1419244982 for the changes I have made now. Could you please review the PRs above?

bozana commented 1 year ago

Hi @asmecher, thanks a lot for the review! I considered everything (s. the last commits). Additionally: In OMP I also considered chapter for datePublished and pages, and I changed the way we assure the DC meta elements are displayed only for the current version when viewing a file (see the comment there). In OJS now I get the section from the TemplateManger. OK so?

asmecher commented 1 year ago

Yes, I can't think of any problem with those approaches. Please go ahead, thanks!