Add patient ID to default filter display

[john loveland]

Sites without a RIS will not have accession numbers. In this scenario it is then useful to display patient ID instead.

[Ed McDonagh]

I would suggest the best way to proceed with this @JLMPO might be as follow:

• Fork the project
• Clone your fork to your computer
• Make your changes - see below for suggestions, committing regularly to your fork
• If you want us to look at what you have done, just add our names to a comment (use the @ format, so @edmcdonagh or @dplatten)
• When you are happy, create a pull request

There are guides on the bitbucket website to take you though all of this, including using git (the version control system we use), pulling, committing, pushing, pull requests etc.

In terms of the changes you might like to make, consider the following:

• In the database there should probably be a field id_displayed in the PatientIDSettings class in remapp/models.py. For completeness, there should probably be a field name_displayed too. This will require a database migration before you can make use of it - python manage.py makemigrations remapp python manage.py migrate remapp
• The additional field(s) will need to be added to PatientIDSettingsUpdate class in remapp/views.py
• The template remapp/templates/remapp/patientidsettings_form.html will need to be updated with the new fields so you can set them in the interface
• We need to pass the PatientIDSettings through to the template from remapp/views.py rf_summary_list_filter - something like adding the following at the start of the function,

    from remapp.models import PatientIDSettings

and

    if PatientIDSettings.id_displayed:
        admin['display_id'] = True
    if PatientIDSettings.name_displayed:
        admin['display_name'] = True

before the return section.

• You can then add the patient ID and/or name fields to the templates for the summary list filtered view and the detail view; remapp/templates/remapp/rffiltered.html and rfdetail.html respectively for the fluoroscopy data. They should be in an if statement something along the lines of

{% if admin.display_id %}
{{ db link to id }}
{% endif %}

[Ed McDonagh]

P.S. OpenREM developers can make use of a complementary licence for PyCharm for developing OpenREM - let me know if you are interested after you have made a little headway