Project

General

Profile

bug #8739

Updated by Andreas Kohlbecker over 4 years ago

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. 

 ---  

 ## TODOs from Code review: 


 **1.** 

 **DONE** 

 description.inc#1179 :  

 ~~~php 
 $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 

 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()  

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

 5. pages.inc: compose_cdm_description() 

 ~~~php 
 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:** 

 moved to #8766 

 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 ... 


 **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. 

 ---- 

 Hallo, 

 hier wie besprochen noch ein paar Anmerkungen zur Portaldarstellung 
 zu http://test.e-taxonomy.eu/dataportal/preview/campanulaceae/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 ... 
 2. bei numerischen Characters wird immer noch am Ende die Specimen-Zahl ausgegeben als e.g. "10 SampleSize", auch hier sollte stehen "(10)" 
 3. Bei "stem width" wird als Minimum "0" angegeben: wie kommt das? Dateneingabe- oder -ausgabe-Fehler? 
     * es fehlt hier die Maßeinheit. Dateneingabe- oder -ausgabe-Fehler? 
     * 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 . 
 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)" 
 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. 



 Schöne Grüße, 
 Norbert 

Back