Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Klargjør om alle _links-oppføringer skal ha «templated»-feltet #16

Closed
petterreinholdtsen opened this issue Oct 26, 2018 · 2 comments
Labels
kommentar innspill for å gjøre spesifikasjonen bedre trenger klargjøring eksisterende tekst ikke er klar og entydig nok

Comments

@petterreinholdtsen
Copy link
Collaborator

Dette er en kopi av mangelmelding sendt inn til Arkivverket, se også https://github.com/petterreinholdtsen/noark5-tester/blob/master/mangelmelding/sendt/2017-03-31-templated-links.md .


       Prosjekt  NOARK 5 Tjenestegresesnitt
       Kategori  Versjon 1.0 beta
    Alvorlighet  kommentar
   Meldingstype  trenger klargjøring
Brukerreferanse  pere@hungry.com
    Dokumentdel  6.1.1.2
     Sidenummer  13
    Linjenummer  25
Innsendingsdato  2017-03-31

Beskrivelse

Del 6.1.1.2 (Finne objekter (Read)) på side 13 omtaler feltet «templated» som kan brukes i _links-lister for ressurser som støtter oData-filter. Men det står ikke om feltet er påkrevd eller valgfritt for de ressursene som ikke støtter slike filter? Kan klienter forvente at feltet alltid er tilstede i korrekte API-implementasjoner eller ikke?

Feltet er så vidt jeg kan se alltid satt for elementer i _links-lister i testinstansen tilgjengelig fra http://n5test.kxml.no/api/. Men feltet eksisterer ikke i eksempelresponsen vist frem på [http://rel.kxml.no/noark5/v4/api/arkivstruktur/mappe/] http://rel.kxml.no/noark5/v4/api/arkivstruktur/mappe/).

I eksempelresponsen finnes det derimot et felt som heter «templatedSpecified». Mens hvis en besøker http://rel.kxml.no/noark5/v4/api/arkivstruktur/ så eksisterer både «templated» og «templatedSpecified». Er sistnevnte en skrivefeil for «templated»? Jeg finner ikke «templatedSpecified» omtalt i spesifikasjonen.

For å spare båndbredde under overføring ved bruk av API-et foreslår jeg at det klargjøres at feltet «templated» er valgfritt og kun må eksistere når det settes til «true», og kan antas å være «false» hvis det ikke eksisterer.

Hvis feltet ikke skal være valgfritt bør det nevnes i spesifikasjonen at feltet alltid skal være med i API-responsen.

Ønsket endring

Foreslås at det legges inn ny setning på side 13 under 6.1.1.2 (Finne objekter (Read)).

Etter setningen «De ressurser som støtter filter skal annonserer dette under _links med «templated=true» og parametre som kan brukes til dette i «href» » føyes det til følgende setning:

«Feltet «templated» er valgfritt og antas å være «false» hvis det ikke finnes.»

Respons

Henvendelsen er gitt saksnummer 2017/5964 hos Arkivverket.

@torkjetil
Copy link

det er valgfritt i utgangspunktet om templated skal være med og er bare krav om en skal annonsere at filter støttes. Kan det tolkes annerledes fra opprinnelig tekst? Tanken var at konformitetsnivåene skulle stramme inn igjen hvor og hvilke filter som må støttes for de enkelte ressursene

@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Oct 29, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kommentar innspill for å gjøre spesifikasjonen bedre trenger klargjøring eksisterende tekst ikke er klar og entydig nok
Projects
None yet
Development

No branches or pull requests

2 participants