OPD UX Changes

[adeolabadmus]

Fixes #996

Changes proposed in this pull request:

• Changed Edit report to View
• Added Print Button to visit page

cc @HospitalRun/core-maintainers

[jkleinsc]

@adeolabadmus I commented below on how the print functionality can be done in a cleaner way.

As far as the update button is concerned, the abstract-edit-controller has a computed property. showUpdateButton that is used to determine if the update button should display. You can override that in your controller to not show the update button on an opd report when isNew is false. You still need to run the logic in abstract-edit-controller because it checks user permissions, so in your override you should do something like this:

showUpdateButton: computed('updateCapability', 'model.{isNew,reportType}', function() {
 let showUpdate = this._super();
 ...
additional logic to determine if update button should display.
}) 

[jkleinsc]

I'm not too keen on how this was implemented. I think a better approach would be when clicking on the Print button, you are directed to the report page and print automatically fires from that page. You can use the {{print-this}} component to automatically print the page on render, so if you pass in a queryParam, in the template you can add the following to auto print:

{{#if printQueryParam}} 
  {{print-this}}
{{/if}}

[jkleinsc]

There is a small change needed here before this can be merged.

[jkleinsc]

print should default to true; otherwise invoice printing won't work unless you change that template. I would prefer to change the default to true.