zdavatz / oddb.org

Open Drug Database for Switzerland
https://ch.oddb.org
GNU General Public License v3.0
10 stars 8 forks source link

Fetching new_atc_codes from who_cc fails #122

Closed ngiger closed 3 years ago

ngiger commented 3 years ago

Running test/test_plugin/who.rb has two failures, which stem from the fact that the URL http://www.whocc.no/atc/lists_of_new_atc_ddds_and_altera/new_atc/?order_by=1&d=DESC is no longer valid.

Looks for me that we might:

  1. Change URL form http to https
  2. Use ddd/lists_of_temporary_atc_ddds_and_alterations/atc_codes/?order_by=0&d=DESC
zdavatz commented 3 years ago

New URLs seem to be:

https://www.whocc.no/atc_ddd_alterations__cumulative/ddd_alterations/ https://www.whocc.no/atc_ddd_alterations__cumulative/atc_alterations/

ngiger commented 3 years ago

Bist Du sicher, dass wir beide Seiten besuchen müssen und wird die Anpassungen von 2005 bis 2020 durchsuchen müssen. Meiner Meinung nach wär der Link https://www.whocc.no/ddd/lists_of_temporary_atc_ddds_and_alterations/atc_codes/?order_by=1&d=DESC mit der Liste des aktuellesten Jahres gut genug.

zdavatz commented 3 years ago

Ja, Entschuldigung, dieser Link genügt: https://www.whocc.no/ddd/lists_of_temporary_atc_ddds_and_alterations/atc_codes/?order_by=1&d=DESC

ngiger commented 3 years ago

Aufwand war 1h. jobs/import_whocc lief durch. Die folgenden Fehler sind soweit ich weiss normal:

Could not create: :!A16AX14!ddd,O., reason: :!A16AX14!ddd,O. -> OddbPrevalence::atc_class(A16AX14) returned nil Could not create: :!R03DX09!ddd,P., reason: :!R03DX09!ddd,P. -> OddbPrevalence::atc_class(R03DX09) returned nil

zdavatz commented 3 years ago

Warum kann das nicht gespeichert werden:

latest-screenshot

Das müsste eigentlich klappen.

zdavatz commented 3 years ago

Das gleiche gilt für

latest-screenshot

zdavatz commented 3 years ago

diese neuen ATC-Codes müssten eigentlich kreiert und gespeichert werden, damit man Ihnen neue Medikamente zuweisen kann. Der ATC-Code in der Fachinfo ist bereits korrekt.

ngiger commented 3 years ago

Das hast auf ch.oddb.org getestet? Das würde heissen, dass das auch schon unter Ruby 2.7 nicht lief.

zdavatz commented 3 years ago

Screenshots sind von ch.oddb.org, von heute.

ngiger commented 3 years ago

Okay. Ich lassen den Import nochmals laufen und analysiere das Problem mit pry.

ngiger commented 3 years ago

Gemäss who A16AX14 wurde A16AX14 erst im Dezember 20 geändert. Dito für WHO A16AX14 Sehe im pry, dass zuerst eine DDD '0' erstellt werden soll, obwohl die dazugehörende ATC noch nicht existiert.

ngiger commented 3 years ago

Aufwand: 22.1.21 1.5h + 0.5h Nachdenken nach dem Nachtessen

zdavatz commented 3 years ago

See #128