locdb / locdb-frend

Fr(ont-)end for the Linked Open Citation Database.
https://locdb.github.io
GNU General Public License v3.0
6 stars 2 forks source link

SWB -> K10plus #445

Open LauraErhard opened 5 years ago

LauraErhard commented 5 years ago

The SWB is now integrated in the K10plus and the SWB database will not be updated anymore. The SWB PPN is still in the metadata but is not the primary ID anymore: 2019-04-03_ingest_ids

Ingest part: Uploading with the SWB-PPN is not possible at the moment: 2019-04-03_Ingest If I replace the SWB PPN with the K10plus ID the upload is possible.

abdelqader-mohammad commented 5 years ago

If the K10plus is the primary ID and mandatory, it makes sense to me that entries without a K10plus ID are rejected, so it shouldn't be possible to enter another type of ID in the frontend. If it should be possible to enter Ressources without a primary ID then the Backend has to resolve the new resource, in which case the actual error response from the Backend should show if this error could be solved in the frontend. We will try to reproduce that error.

abdelqader-mohammad commented 5 years ago

It seems like the the the backend ignores the identifier selected in the input form and treats the identifier as the expected type. So creating an electronic resource with an SWB_PPN results in an http 500 exception from the server without any details.

So as a quick fix, we replaced the SWB PPN with K10plus. @LauraErhard can you test the dev if everything works fine? before deploying the changes into production.

LauraErhard commented 5 years ago

Now neither the SWB PPN nor the K10plus ID works. Therefore I can't upload anything.

lgalke commented 5 years ago

Our input types as described by Anne (See also #251):

es sind aktuell erlaubt: 1) alle, wenn die resource schon existiert und man nur einen scan (also ein file) anhängen möchte. Da mache ich keinen unterschied. Dann 2) journal: required ist zdbid; 3) journalArticle: required ist doi, 4) book_chapter mit doi oder swb ppn oder 5) proceedings_article auch mit doi oder ppn, 6) monograph mit swb ppn und 7) book mit swbppn an alle kann man einen scan dranhängen außer bei journal

Translated:

In Anne's backend specification, I find:

/saveResource:
// [...] //
 parameters:
        - name: identifierScheme
          in: formData
          description: The scheme of the identifier of the resource
          required: true
          type: string
          enum: [SWB_PPN, ZDB_PPN, DOI, OLC_PPN]

There is no K10plus ID in the list of allowed identifier schemes regardless of any resource type.

The corresponding controller also does not have any specific logic for K10plus identifiers.So, adding the identifier to the enum above will not be enough.

It would require a major update of the back-end to enable ingesting resource via K10Plus ids. DOI's should, however, work and trigger a CrossRef lookup.