Project

General

Profile

Actions

bug #10078

closed

Symbol and details view title of misapplied name type is not the same as shown in the type combo element

Added by Katja Luther almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Priority13
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

90%

Estimated time:
Severity:
normal
Found in Version:

Description

mail ERS:

Wenn man partial misapplication UND pro parte misapplication behalten will (den Unterschied habe ich persönlich nie verstanden), sollten zumindest die verwendeten Abkürzungen ausgetauscht werden:

Partial misaaplication: (part.)
Pro parte misapplication: (p.p.).

Im Moment ist es immer noch genau umgekehrt.

mail AM:
Auf was bezieht sich diese Aussage, auf Portal oder auf den TaxEditor?
Eigentlich müsste das schon korrigiert sein, aber vielleicht noch nicht überall.
Ggf. sollte man die Repräsentation schnell ändern können.
Hast du ein konkretes Beispiel?

mail ERS:
ich beziehe mich auf den TaxEditor, wo man im Details View zu einem misapplied name den misapplication type auswählen kann.
Wähle ich „is pro parte misapplied name for“, erscheint im Freitextbereich und dann auch im Portal „(part.)“, wähle ich dagegen „is partial misapplied name for“, erscheint ein (p.p.). Das sollte eben genau umgekehrt sein.

Beispiel: https://europlusmed.org/cdm_dataportal/taxon/63fead7a-662a-4082-9503-c6ea994c2049
-(p.p.) „Fritillaria nervosa“ sensu…

Hatten wir uns eigentlich darauf geeinigt, dass das (p.p.) dort an der richtigen Stelle steht? Wenn ja, sollte man das grafisch so lösen, dass mindestens noch ein Leerzeichen zwischen Strich und Klammer steht.

In der Literatur findet man aber gelegentlich auch eine andere Reihenfolge mit hintangestelltem p.p.: Hatten die Taxonomen sich dazu mal einheitlich geäußert, wie sie das haben wollten?

“Fritillaria nervosa” sensu Franco & Rocha Afonso, p.p.

mail AM:
danke für das Beispiel.
Das ist sehr seltsam. In der DB steht es auf jeden Fall richtig drin:

SELECT r.id, r.label, r.abbreviatedLabel, dtb.id, dtb.idInVocabulary, dtb.titleCache, dtb.symbol, dtb.symbol2
FROM Representation r INNER JOIN DefinedTermBase_Representation MN ON MN.representations_id = r.id INNER JOIN DefinedTermBase dtb ON dtb.id = MN.DefinedTermBase_id
WHERE r.label LIKE '%misapplied%'

Es könnte sein, dass der Formatierer in cdmlib das falsch macht. Wäre zwar seltsam (weil durch Tests abgedeckt) aber möglich.
Wenn nicht, müsstest du (Katja) mal im TaxEditor und Portalcode schauen, warum es in beiden falsch ankommt. Evtl. gibt es auch ein Ticket, ganz dunkel kann ich mich erinnern.

mail AM:

ich habe mir das gerade noch mal angeschaut und es ist leider kritischer.
Wenn man im Editor schaut sieht man, dass auch die Label im Details View nicht übereinstimmen:

Konkret ist es so, dass bei Auswahl von „partial misapplied name for“ eigentlich „pro parte“ ausgewählt wird. Ich habe das in der DB überprüft.
Katja, kannst du das bitte auch nochmal überprüfen und dann mit höchster Priorität fixen?
Wir müssen dann auch zum nächsten Release alle bestehenden Daten in allen DBs die pp/partial MAN benutzen überprüfen und ggf. nach Rücksprache fixen.
Gut, Eckhard, dass dir das aufgefallen ist.


Files

select_view.png (14.7 KB) select_view.png Katja Luther, 06/22/2022 11:03 AM
editor.jpg (50.5 KB) editor.jpg Katja Luther, 06/22/2022 11:04 AM
Actions #1

Updated by Katja Luther almost 2 years ago

Actions #2

Updated by Andreas Müller almost 2 years ago

I do not understand the title of this ticket. The problem to me seems that the selection in the dropdown is stored incorrectly and not the symbol or the title of the details view.

Actions #3

Updated by Katja Luther almost 2 years ago

Andreas Müller wrote in #note-2:

I do not understand the title of this ticket. The problem to me seems that the selection in the dropdown is stored incorrectly and not the symbol or the title of the details view.

The problem seems to be a wrong interpretation of the selection of the combo element.

Actions #4

Updated by Katja Luther almost 2 years ago

  • Subject changed from Symbol and details view title of misapplied name type is not correct to Symbol and details view title of misapplied name type is not the same as shown in the type combo element
Actions #5

Updated by Katja Luther almost 2 years ago

  • Status changed from New to Resolved
  • Assignee changed from Katja Luther to Andreas Müller

This should be solved, please review.

Actions #6

Updated by Andreas Müller almost 2 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 0 to 90

Seems to work correctly and code changes make sense. Can you still remove not needed commented code (and maybe also remove generics from new ArrayList();) and then we can close the ticket.

Actions #7

Updated by Andreas Müller almost 2 years ago

IMPORTANT:

Before closing the ticket we need to check existing data in production databases for incorrect relationship types.!!

Actions #8

Updated by Andreas Müller almost 2 years ago

  • Status changed from Feedback to Resolved
  • Assignee changed from Katja Luther to Andreas Müller
  • Priority changed from Highest to Priority13
  • % Done changed from 90 to 80

I give this to me for updating data after the release. Give back to KL for cleanup and close afterwards.

Actions #9

Updated by Andreas Müller almost 2 years ago

Non-default MAN:

SELECT tr.id relId, dtb.titleCache relType, tb1.titleCache MAN, tb2.titleCache accTaxon
FROM DefinedTermBase dtb INNER JOIN TaxonRelationship tr ON tr.type_id = dtb.id INNER JOIN TaxonBase tb1 ON tb1.id = tr.relatedFrom_id INNER JOIN TaxonBase tb2 ON tb2.id = tr.relatedTo_id
WHERE dtb.titleCache LIKE '%p%misappl%'
Actions #10

Updated by Andreas Müller almost 2 years ago

Instances with Non-default-MAN:

Nepenthaceae: 1 partial, ~10 pp => Mail sent to WGB
Caryo_spp: 1 partial => Mail sent to WGB and NaK
Cyprus: 1pp => Mail sent to RH
E+M: 1partial, ca. 45pp => Mail sent to ERS
E+M caucasus: 17 pp => nothing done as only distribution will be exported to E+M
Flora Cuba: 14pp => Mail sent do WGB
End

Actions #11

Updated by Andreas Müller almost 2 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 80 to 90

Bug is fixed. Notification sent to data managers.

Katja, can you still clean up the code a bit and then close the ticket?

Actions #12

Updated by Katja Luther almost 2 years ago

  • Status changed from Feedback to Closed

Code cleaning done.

Actions

Also available in: Atom PDF