Use display name table to delete unwanted studies

[Ed McDonagh]

Studies that get on the system that are not of interest or not useful, such as imported studies or CR studies, could all be deleted along with their entry in the display name table.

[Ed McDonagh]

This is actaully the issue I should have been referencing with all the recent commits, rather than issue #479

[Ed McDonagh]

See also pull request #149

[Ed McDonagh]

Adding refs #318, #580 to changes

→ <<cset ade0149799e9>>

[David Platten]

Adjusted the layout of text and buttons on the study review page to suit my taste. References issue #318 and perhaps issue #580

→ <<cset 22ba36efa973>>

[David Platten]

@edmcdonagh, apologies - my recent commit has broken the buttons...

[Ed McDonagh]

I don't see how?

The other thing I didn't end up doing was to change the behaviour of the 'cancel' button - currently it implements the 'back' function, which isn't really what you want unless you happen to have reloaded the page pressing the delete studies button...

[Ed McDonagh]

I see - you deleted the named span elements that are replaced with the AJAX to insert the form.

[David Platten]

Fixing the buttons that I just broke. References issue #318 and perhaps issue #580

→ <<cset a4c394420ed7>>

[Ed McDonagh]

The reason I shifted it to the right was because I didn't want it to just be part of a block of text. It needs to stand out in some way and break the flow.

How about throwing it all in a panel: https://getbootstrap.com/docs/3.3/components/#panels-alternatives ?

And the spelling mistake @LuukO pointed out ("ths equipment" instead of "this equipment") remains.

What do you think? I'd have a go but that would mean tidying up my #114 code to commit it or stashing it to checkout this branch...

[Luuk]

I think a panel would be fine for me.

[David Platten]

I've put the text and buttons for the Study deletion section of the study review page in a panel. I've chosen an info panel class as the colour of the heading is similar to what is used on the DICOM networking page. References issue #318 and perhaps issue #580

→ <<cset 9472948ad62f>>

[Ed McDonagh]

That works for me. Thanks.

[Ed McDonagh]

Merged in issue580rfanddx (pull request #149)

Fixes #580, #318 RF/DX and deleting studies

Documentation is outstanding (see issue ref #583)

→ <<cset 2eb379749d97>>

[Ed McDonagh]

Three issues with this as it stands:

1. Deleting a lot of studies can take more than 30 seconds it seems, which might lead to breakages!
2. I was thinking there should be a button to re-evaluate with a different modality filter or none. But that function exists I guess by finding the same entry in the different tables of the display names page.
3. The documentation doesn't indicate this, and doesn't indicate that if you delete you will only delete those in the current filter (current modality).

[Ed McDonagh]

Clarified what happens when a study has more than one modality. Changed docs link to deletions page from display name page. Refs #318, #583

→ <<cset 4cfdb64f39d3>>

[Ed McDonagh]

Merged in issue583docs4displaynamesAndDeleting (pull request #170)

Issue583docs4displaynamesAndDeleting

Refs #318, #580, fixes #583

→ <<cset 77cabb887da8>>