Bug 18952

Summary: Show internal note in acquisitions details tab
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Josef Moravec <josef.moravec>
Severity: enhancement    
Priority: P5 - low CC: david, josef.moravec, jzairo, kelly, martin.renvoize, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This enhancement makes internal and vendor notes visible in the acquisitions details tab that is displayed for each bibliographic record when the `AcquisitionDetails` system preferences is enabled.
Version(s) released in:
Attachments: Bug 18952: Add internal note to acq details tab
Bug 18952: Add internal note to acq details tab
Bug 18952: Add internal note to acq details tab
Bug 18952: Add internal note to acq details tab
Bug 18952: Add internal note to acq details tab
Bug 18952: Add internal note to acq details tab

Description Katrin Fischer 2017-07-18 14:22:50 UTC
If the AcquisitionDetails system preference is used, there is an extra tab on the staff details page displaying information about the acquisition orders linked to this bibliographic record.

It would be nice if the internal note from the order would be displayed there as well.
Comment 1 Katrin Fischer 2019-02-04 20:38:13 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2019-02-04 20:56:43 UTC
Created attachment 84755 [details] [review]
Bug 18952: Add internal note to acq details tab

Any important notes about an order are usually added to the
internal note. This patch makes the note visible on the
acq detail tab in the staff client.

To test:
- Add a basket
- Order something
- Add an internal note for the order
- Go to the detail page of the ordered record
- Switch to the Acquisition details tab
- Verify that with the patch the internal note shows
- Verify that the new column can be toggled using the
  column visibility settings
Comment 3 Owen Leonard 2019-02-05 13:57:06 UTC
Works well but needs one correction:

missing_filter at line 773 (            <td>[% order.order_internalnote %]</td>)
Comment 4 Katrin Fischer 2019-02-05 13:58:43 UTC
(In reply to Owen Leonard from comment #3)
> Works well but needs one correction:
> 
> missing_filter at line 773 (            <td>[% order.order_internalnote
> %]</td>)

you are so right *hides*
Comment 5 Katrin Fischer 2019-02-06 06:21:10 UTC
Created attachment 84806 [details] [review]
Bug 18952: Add internal note to acq details tab

Any important notes about an order are usually added to the
internal note. This patch makes the note visible on the
acq detail tab in the staff client.

To test:
- Add a basket
- Order something
- Add an internal note for the order
- Go to the detail page of the ordered record
- Switch to the Acquisition details tab
- Verify that with the patch the internal note shows
- Verify that the new column can be toggled using the
  column visibility settings
Comment 6 Katrin Fischer 2019-02-06 06:21:56 UTC
Since the patch and the change are so smal, I've amended the patch with the filter.
Comment 7 Maryse Simard 2019-02-08 20:27:37 UTC
Created attachment 84901 [details] [review]
Bug 18952: Add internal note to acq details tab

Any important notes about an order are usually added to the
internal note. This patch makes the note visible on the
acq detail tab in the staff client.

To test:
- Add a basket
- Order something
- Add an internal note for the order
- Go to the detail page of the ordered record
- Switch to the Acquisition details tab
- Verify that with the patch the internal note shows
- Verify that the new column can be toggled using the
  column visibility settings

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Comment 8 Katrin Fischer 2019-02-09 09:00:37 UTC
Thx Maryse!
Comment 9 Pierre-Marc Thibault 2019-02-15 20:51:29 UTC
Created attachment 85195 [details] [review]
Bug 18952: Add internal note to acq details tab

Any important notes about an order are usually added to the
internal note. This patch makes the note visible on the
acq detail tab in the staff client.

To test:
- Add a basket
- Order something
- Add an internal note for the order
- Go to the detail page of the ordered record
- Switch to the Acquisition details tab
- Verify that with the patch the internal note shows
- Verify that the new column can be toggled using the
  column visibility settings

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>
Comment 10 Josef Moravec 2019-02-28 08:46:39 UTC
Created attachment 85825 [details] [review]
Bug 18952: Add internal note to acq details tab

Any important notes about an order are usually added to the
internal note. This patch makes the note visible on the
acq detail tab in the staff client.

To test:
- Add a basket
- Order something
- Add an internal note for the order
- Go to the detail page of the ordered record
- Switch to the Acquisition details tab
- Verify that with the patch the internal note shows
- Verify that the new column can be toggled using the
  column visibility settings

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Signed-off-by: Pierre-Marc Thibault <pierre-marc.thibault@inLibro.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 11 Nick Clemens 2019-03-15 19:56:55 UTC
Awesome work all!

Pushed to master for 19.05
Comment 12 Martin Renvoize 2019-03-19 10:57:59 UTC
Enhancement, will not be backported to 18.11.x series.