Display "Requested procedure" on all filtered html pages

Issue #544 resolved
David Platten created an issue

This is only displayed on the CT and radiographic pages at the moment. I would find it useful on the fluoroscopy page, and figure we may as well add it to the mammo one too.

Comments (20)

  1. Ed McDonagh

    I'd like to broaden this out to ensure consistency across modalities based on changes made for issue #507.

    Each modality should display in the filtering list the following:

    • Study description
    • Procedure
    • Requested Procedure

    Each of these should be searchable, along with acquisition protocol.

    I'd appreciate anyone's thoughts on whether having four items in the same cell (including accession number) is too much. You can see the effect in the mammo tab on testing or dev demo sites.

    @tcdewit, @luuko, @dplatten, anyone else!

  2. Ed McDonagh

    Named summary list filter urls so that they can be used in testing. Added tests for RF - currently failing for acquisition filter as study is listed more than once. Refs #544

    → <<cset cc2de56e8508>>

  3. David Platten reporter

    Having four items in the same cell and being consistent for all modalities is fine with me.

  4. David Platten reporter

    studyhist is set to 1 in urls attached to histogram data bars of plots of DLP and CTDIvol per study description. The code that puts them there is in ctfiltered.html on lines 567 and 666.

    I think that it is set to ensure that when a user clicks on a histogram link from one of the above-mentioned plots that the resulting data is filtered by the appropriate study description.

  5. Log in to comment