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
Assignee: Kyle M Hall (khall)
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-22 15:54 UTC by Kyle M Hall (khall)
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
Circulation function:

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 (khall) 		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
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 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 (khall) 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 (tcohen) 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 (lukeg) 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