Bug 33040 - Add "Date published (text)" to serials tab on record view (detail.pl)
Summary: Add "Date published (text)" to serials tab on record view (detail.pl)
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Kyle M Hall
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-22 15:54 UTC by Kyle M Hall
Modified: 2023-12-28 20:42 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.05,22.05.13,21.11.21


Attachments
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl) (2.80 KB, patch)
2023-02-22 16:10 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl) (2.88 KB, patch)
2023-02-22 17:16 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl) (2.92 KB, patch)
2023-04-03 23:39 UTC, Victor Grousset/tuxayo
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 2023-02-22 15:54:50 UTC
Some libraries would like to see the "Date published (text)" in the serials tab on the "Normal" record view ( aka catalogue/detail.pl ).
Comment 1 Kyle M Hall 2023-02-22 16:10:31 UTC
Created attachment 147172 [details] [review]
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl)

Some libraries would like to see the "Date published (text)" in the serials tab on the "Normal" record view ( aka catalogue/detail.pl ).

Test Plan:
1) View the serials tab on detail.pl for a subscription where the latest
   serial has a "Date published (text)" field populated.
2) Apply this patch
3) Restart all the thigns!
4) Browse to detail.pl for that record
5) View the Subscriptions tab
6) Note the "Date published (text)" column exists now!
Comment 2 Matt Blenkinsop 2023-02-22 17:16:06 UTC
Created attachment 147177 [details] [review]
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl)

Some libraries would like to see the "Date published (text)" in the serials tab on the "Normal" record view ( aka catalogue/detail.pl ).

Test Plan:
1) View the serials tab on detail.pl for a subscription where the latest
   serial has a "Date published (text)" field populated.
2) Apply this patch
3) Restart all the thigns!
4) Browse to detail.pl for that record
5) View the Subscriptions tab
6) Note the "Date published (text)" column exists now!

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Comment 3 Victor Grousset/tuxayo 2023-04-03 23:39:34 UTC
Created attachment 149111 [details] [review]
Bug 33040: Add "Date published (text)" to serials tab on record view (detail.pl)

Some libraries would like to see the "Date published (text)" in the serials tab on the "Normal" record view ( aka catalogue/detail.pl ).

Test Plan:
1) View the serials tab on detail.pl for a subscription where the latest
   serial has a "Date published (text)" field populated.
2) Apply this patch
3) Restart all the thigns!
4) Browse to detail.pl for that record
5) View the Subscriptions tab
6) Note the "Date published (text)" column exists now!

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Comment 4 Victor Grousset/tuxayo 2023-04-03 23:41:25 UTC
Works, QA script happy, code looks good, makes sense, passing QA :)

Thanks Aleisha for the functional relevance confirmation.
Comment 5 Tomás Cohen Arazi 2023-04-06 12:52:31 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 6 Jacob O'Mara 2023-04-16 20:37:34 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.
Comment 7 Lucas Gass 2023-05-05 21:40:54 UTC
Backported to 22.05.x for upcoming 22.05.13
Comment 8 Arthur Suzuki 2023-05-23 09:09:35 UTC
minor, won't backport
Comment 9 Arthur Suzuki 2023-05-23 09:25:04 UTC
actually minors should also be backported for oldstable : try and backport regardless of severity.
Applied to 21.11 for 21.11.21