wadqc / WAD_Interface

NVKF WAD webinterface
3 stars 1 forks source link

series date leeg, geen datum in dropdown #60

Closed JaapGroen closed 9 years ago

JaapGroen commented 9 years ago

Als een selector op serieniveau wordt gedefinieerd, maar de dicomtag series date is leeg, dan is het dropdownmenu bij het bekijken van de resultaten ook leeg. Beter zou het zijn om dan terug te vallen op study date oid.

Antwoord Tim: Om je vraag te beantwoorden: Dcm4chee slaat helaas geen seriestime op; in plaats daarvan wordt de content_time van de eerste instance gebruikt om te tonen in de dropdown. Blijkbaar wordt deze dus niet altijd gevuld. De query in show_results.php zal moeten worden aangepast om terug te vallen op studytime, mits content_time leeg is, dus bijv (ongetest):

$year_Stmt_series="SELECT $table_gewenste_processen.pk as 'pk', $table_gewenste_processen.status as 'status', coalesce($table_instance.content_datetime,$table_study.study_datetime) as 'date_time' from $table_gewenste_processen inner join $table_series on $table_gewenste_processen.series_fk=$table_series.pk, $table_instance, $table_study where $table_gewenste_processen.selector_fk=$selector_fk and $table_gewenste_processen.status in (%s) and $table_series.pk=$table_instance.series_fk and $table_instance.pk=(select min(pk) from $table_instance where series_fk=$table_series.pk) and $table_study.pk=$table_series.study_fk order by $table_instance.content_datetime desc";

tcdewit commented 9 years ago

Net getest en op een correcte sortering na, werkt de query. Hieronder een aangepaste query. Zou je deze svp kunnen testen en laten weten of dit voor jou ook het probleem oplost?

$year_Stmt_series="SELECT $table_gewenste_processen.pk as 'pk', $table_gewenste_processen.status as 'status', coalesce($table_instance.content_datetime,$table_study.study_datetime) as 'date_time' from $table_gewenste_processen inner join $table_series on $table_gewenste_processen.series_fk=$table_series.pk, $table_instance, $table_study where $table_gewenste_processen.selector_fk=$selector_fk and $table_gewenste_processen.status in (%s) and $table_series.pk=$table_instance.series_fk and $table_instance.pk=(select min(pk) from $table_instance where series_fk=$table_series.pk) and $table_study.pk=$table_series.study_fk order by $table_instance.content_datetime,$table_study.study_datetime desc";

Je kunt show_results.php vinden in: WAD-IQC/database/iqc/show_results.php

JaapGroen commented 9 years ago

Ja dat lost het op. De dropdown geeft nu de datetime ipv alleen date. Ook mooier wat mij betreft. Sortering is ook correct, nieuwste bovenaan.

tcdewit commented 9 years ago

Wat bedoel je precies met "alleen date"? Hij gaf voorheen toch een lege dropdown? Blijf het wel vreemd vinden dat dicom-tag "content time" (0008,0033) leeg is. Voor welke modaliteit en type dicom-file is dit het geval? Ik verwacht geen problemen met deze workaround, maar wil nog even van de andere developers horen of we deze fix moeten doorvoeren.

JaapGroen commented 9 years ago

Dicom savescreen van een GE SPECT-CT QC uitwerking. Tag 0008.0033 bestaat uberhaupt niet in die files. En Series date is ook leeg. Daar werd ik door getriggerd.

En je hebt gelijk wb de lege dropdown. Eerst leeg, nu datetime. En dat gedlt ook voor wel goede datasets, bij oude query stond alleen de datum, nu ook datetime.

wadqc commented 9 years ago

Getest, fix in commit d1591bd3e9fa1faa638afb17bdcb7438b23e0ab8