Bug 29811

Summary: misc/export_records.pl add possibility to export with timestamp option on authority record type
Product: Koha Reporter: Thibaud Guillot <thibaud.guillot>
Component: ToolsAssignee: Thibaud Guillot <thibaud.guillot>
Status: Pushed to oldstable --- QA Contact: Nick Clemens (kidclamp) <nick>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, martin.renvoize, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.06,22.11.12
Attachments: BUG 29811 : add timestamp option on authority record type
Bug 29811: add timestamp option on authority record type
Bug 29811: add timestamp option on authority record type

Description Thibaud Guillot 2022-01-06 13:46:28 UTC
misc/export_records.pl got many options, include bibs export based on a certain timestamp but it doesn't exist for authority record type.
It can be interresting to add this option for authority according to the timestamp field (modification_time) already present in the database.
Comment 1 Thibaud Guillot 2022-01-06 14:08:30 UTC
Created attachment 129088 [details] [review]
BUG 29811 : add timestamp option on authority record type

According to the timestamp option for bibs record type I added the
timestamp option for authority records.
Timestamp is already present in database on field "modification_time"

Test Plan :
1 - Be sure to have authority record type for easiest test create one
2 - Execute script export_records.pl in your koha/misc directory and
choose a date (example yesterday if you just created an authority
right now).(see export_records.pl -h for help)
3 - Timestamp option has no effect on authority record type
4 - Execute script again but choose the date of tomorrow for example
5 - Same result
6 - Apply this patch
7 - Play again steps 2 and 4
8 - On step 2 you will see only your authorities created today (because
script show you authority has changes since the date you choose in
option) and for step 4 you must see an empty file.
Comment 2 Chris Cormack 2023-10-08 07:15:59 UTC
Created attachment 156704 [details] [review]
Bug 29811: add timestamp option on authority record type

According to the timestamp option for bibs record type I added the
timestamp option for authority records.
Timestamp is already present in database on field "modification_time"

Test Plan :
1 - Be sure to have authority record type for easiest test create one
2 - Execute script export_records.pl in your koha/misc directory and
choose a date (example yesterday if you just created an authority
right now).(see export_records.pl -h for help)
3 - Timestamp option has no effect on authority record type
4 - Execute script again but choose the date of tomorrow for example
5 - Same result
6 - Apply this patch
7 - Play again steps 2 and 4
8 - On step 2 you will see only your authorities created today (because
script show you authority has changes since the date you choose in
option) and for step 4 you must see an empty file.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Comment 3 Nick Clemens (kidclamp) 2023-10-12 15:43:41 UTC
Created attachment 156984 [details] [review]
Bug 29811: add timestamp option on authority record type

According to the timestamp option for bibs record type I added the
timestamp option for authority records.
Timestamp is already present in database on field "modification_time"

Test Plan :
1 - Be sure to have authority record type for easiest test create one
2 - Execute script export_records.pl in your koha/misc directory and
choose a date (example yesterday if you just created an authority
right now).(see export_records.pl -h for help)
3 - Timestamp option has no effect on authority record type
4 - Execute script again but choose the date of tomorrow for example
5 - Same result
6 - Apply this patch
7 - Play again steps 2 and 4
8 - On step 2 you will see only your authorities created today (because
script show you authority has changes since the date you choose in
option) and for step 4 you must see an empty file.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 4 Tomás Cohen Arazi 2023-10-31 14:06:31 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 5 Fridolin Somers 2023-11-08 08:21:41 UTC
Small useful enhancement, I choose to backport.

Pushed to 23.05.x for 23.05.06
Comment 6 Matt Blenkinsop 2023-11-13 15:29:06 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x