Bug 25535

Summary: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date
Product: Koha Reporter: Kyle M Hall <kyle>
Component: Architecture, internals, and plumbingAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Tomás Cohen Arazi <tomascohen>
Severity: normal    
Priority: P5 - low CC: 1joynelson, martin.renvoize, tomascohen, victor
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:
20.05.00, 19.11.06, 19.05.12
Attachments: Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date
Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date
Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date

Description Kyle M Hall 2020-05-18 17:09:53 UTC

    
Comment 1 Kyle M Hall 2020-05-18 18:15:06 UTC
Created attachment 105037 [details] [review]
Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date
Comment 2 Victor Grousset/tuxayo 2020-05-18 21:05:01 UTC
Would the test plan be to run this?

prove t/db_dependent/api/v1/holds.t
Comment 3 Victor Grousset/tuxayo 2020-05-18 21:08:55 UTC
Created attachment 105047 [details] [review]
Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date
Comment 4 Victor Grousset/tuxayo 2020-05-18 21:09:20 UTC
Assuming yes.
Comment 5 Martin Renvoize 2020-05-19 07:02:38 UTC
Created attachment 105057 [details] [review]
Bug 25535: Hold API mapping maps cancellationdate to cancelation_date, but it should be cancellation_date

Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize 2020-05-19 07:30:40 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 7 Katrin Fischer 2020-05-19 08:51:19 UTC
It looks like cancelation date is AE? At least my online dictionary claims so.
Comment 8 Kyle M Hall 2020-05-19 11:07:55 UTC
(In reply to Katrin Fischer from comment #7)
> It looks like cancelation date is AE? At least my online dictionary claims
> so.

I've never seen that spelling in the wild. From Wiktionary: Despite the widely used form of canceled vs. cancelled in American English, the word cancellation is still much more widely used than the variant cancelation. The usage of two ls is a remnant of British English, remaining even after American English instated spelling changes.
Comment 9 Katrin Fischer 2020-05-19 21:33:03 UTC
Really interesting, I'll be sticking with ll myself then too :)
Comment 10 Tomás Cohen Arazi 2020-05-19 21:42:28 UTC
(In reply to Katrin Fischer from comment #9)
> Really interesting, I'll be sticking with ll myself then too :)

I really didn't notice, we all voted the naming on the 27 December 2017 Dev meeting LOL :-D
Comment 11 Joy Nelson 2020-05-20 22:45:12 UTC
pushed to 19.11.x for 19.11.06
Comment 12 Victor Grousset/tuxayo 2020-06-08 22:47:19 UTC
Backported to 19.05.x branch for 19.05.12