EDIT

Copied from #8595

• Evtl. will man auf aggr. Description von höheren Taxa nicht die Specimen als Quellen anzeigen, sondern die niedrigeren aggr. Description.

AM: Generally all descriptive data should be shown, not only structured descriptions. E.g. distribution data is still missing. => #8687

TODOs from Code review:¶

1.

DONE

description.inc#1179 :

$enclosing_tag = cdm_feature_block_element_tag_name($feature_block_settings); 

is unused, this breaks the feature block setting as_list (old bug)

2. Don't use <i> and <br> etc to apply styling, all styling must be applied through css

DONE cdm-dataportal|bea70813 & cdm-dataportal|753a8083

1. pages.inc: render_cdm_description_link() ...
2. pages.inc: $render_array[] = markup_to_render_array("Aggregation source from ".$description->created.'<br><br>');
3. pages.inc: render_cdm_description() - $detail_html = (empty($description_string)?$link_text:$description_string).$iconlink."<br><br>";
4. pages.inc: render_description_string()

              $descriptionString .= '<i>' . $element->feature->representation_L10n . '</i>: ' . $state_data . "; ";
            }
            break;
          case 'QuantitativeData':
            $descriptionString .= '<i>' . $element->feature->representation_L10n . '</i>: ' . compose_quant_stats($element) . "; ";

1. pages.inc: compose_cdm_description()

if (isset($description->describedSpecimenOrObservation)) {
    $render_array[] = markup_to_render_array("<b>Specimen</b><br>");
    $render_array[] = markup_to_render_array(render_cdm_specimen_link($description->describedSpecimenOrObservation));
  }
  if (isset($description->taxon)) {
    $render_array[] = markup_to_render_array("<b>Taxon</b><br>");
    $render_array[] = markup_to_render_array(render_cdm_taxon($description->taxon));
  }

SampleSize:

diese Sache mit der SampleSize, die überall auftaucht, ist das schnell zu fixen? Das ist ziemlich irritierend.

• warum werden die nur bei einem Teil der Characters in Klammern (wie es sein sollte), bei den anderen aber als "92 Sample Size" ausgegeben?
• wir sollten in der Tabelle im Header hinter "States" erklärend anfügen: "(bracketed numbers denote sample size)" oder so, ähnlich bei der Description ...

===> #8766

Name der Description

Da war ich ungenau. Wenn das Taxon einen TaxonName hat, sollte der TaxonName.titleCache genommen werden, sonst wird’s zu lang. Nur wenn taxon.getName() == null dann taxon.titleCache nehmen. Wäre jedenfalls mein Vorschlag und machen wir an vielen andere Stellen so. Auf der Seite kommt dann ja auch nochmal der Link, wenn man es genau haben will.

===> #8779

Hallo,

hier wie besprochen noch ein paar Anmerkungen zur Portaldarstellung
zu http://test.e-taxonomy.eu/dataportal/preview/greece_bupleurum/cdm_dataportal/description/ff646ffe-0e6a-4edf-8aff-d3a5e0fdab11

1. wir sollten in der Tabelle im Header hinter "States" erklärend anfügen: "(bracketed numbers denote sample size)" oder so, ähnlich bei der Description ... DONE: Note appears as caption below the table and feature block, the text is aligned right.
2. bei numerischen Characters wird immer noch am Ende die Specimen-Zahl ausgegeben als e.g. "10 SampleSize", auch hier sollte stehen "[10]" - DONE
3. Bei "stem width" wird als Minimum "0" angegeben: wie kommt das? Dateneingabe- oder -ausgabe-Fehler? REJETCED: this is a problem in the aggregation algorithm not in the dataportal.
   • es fehlt hier die Maßeinheit. Dateneingabe- oder -ausgabe-Fehler? - DONE: Problems solved by cdm-dataportal|ad7fa57c but nothing visible yet due to missing data.
   • abgesehen davon, dass die Mittelwertangabe zwischen Min und Max unüblich ist, sind 4 Nachkommastellen mindestens 2 zuviel. Das lässt sich doch sicher angeben? ==> #8771
   • Eine bessere Darstellung wäre ", mean = xx hinter Min+Max . ==> #8786
4. Presence-Problematik: Hatten wir nicht mal eine Regel aufgestellt, wann die Angabe von Presence entfallen soll? Ganz schlimm: "Entire Plant present (28)"; siehe auch, im Gegensatz: "adventitious root presence present (2)"; spine presence present (2), absent(1)" ===> #8759
5. Die seq. Referenzen bei den Taxanahmen sollten in solchen Zusammenhängen unbedingt entfallen, das ist nur krude und völlig überflüssig, da es ja eine Link aufs Taxon gibt; eigentlich sind sogar die Taxonautoren entbehrlich. ==> #8779

Schöne Grüße,
Norbert