Incorrect SI Brochure docnumber about bipm-si-brochure HOT 6 CLOSED

Looks like a bug with document id generation.

from bipm-si-brochure.

@opoudjis says it’s his bug in the collections code.

from bipm-si-brochure.

... I did?

Let me try to replicate locally...

from bipm-si-brochure.

Hm. sib-a4-en has :docnumber: Brochure, and it's had it since d936eb7 , two years ago; before that, it was blank. The other documents also have :docnumber: Brochure, and always did...

No open branch has a different docnumber.

I have refactored the generation of document identifiers, and it was two weeks ago, but I'm at a loss to explain how SI was being added as a prefix by code. Investigating further.

from bipm-si-brochure.

I cannot replicate, and older versions I have lying around do not display that behaviour.

So si-brochure-fr.rxl, generated May 2023, has:

<docidentifier type="BIPM">BIPM Brochure</docidentifier>
<docidentifier type="ISBN">978-92-822-2272-0</docidentifier>
<docnumber>Brochure</docnumber>

sib-a4-fr.xml just generated has

<docidentifier type="BIPM" language="fr">BIPM Brochure Annexe 4</docidentifier>
<docidentifier type="BIPM" language="en">BIPM Brochure Appendix 4</docidentifier>
<docnumber>Brochure</docnumber>

@andrew2net I hate to say it, but I don't see how <docnumber>SI Brochure</docnumber> could ever have been generated. /bibdata/docnumber just replicates the value given in :docnumber:, and I have no evidence that value in the source files was ever anything but Brochure.

I am seeing that your code expects to see SI Brochure there, I just can't reproduce what has happened to put that "SI" in there. And I notice that other documents put the SI explicitly in the attribute; e.g. bipm-si-brochure/sources/mep-ampere-definition-en.adoc explicitly has:

:docnumber: SI MEP A1

and has had it since Oct 2020: 3397df7

I don't know what's happened, I don't know what's changed, and I don't know why I'm getting different results to you Andrej, but my recommendation is that we simply change the :docnumber: attribute for those documents from Brochure to SI Brochure.

from bipm-si-brochure.

I've reviewed the past month of commits to metanorma-bipm, and can't find anything that might have injected an SI prefix to docnumber or docidentifier. Sorry, I can't work this out, and if something was injecting SI before a supplied docnumber attribute, then honestly, it should not have been...

from bipm-si-brochure.