Bug 12274 - Closed invoices not properly handled on invoice details page
Summary: Closed invoices not properly handled on invoice details page
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: master
Hardware: All All
: P3 critical (vote)
Assignee: Jacek Ablewicz
QA Contact: Testopia
URL:
Keywords:
Depends on: 10613
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-17 08:27 UTC by Jacek Ablewicz
Modified: 2019-06-27 09:24 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 12274 - Closed invoices not properly handled on invoice details page (1.56 KB, patch)
2014-05-17 09:12 UTC, Jacek Ablewicz
Details | Diff | Splinter Review
Bug 12274 - Closed invoices not properly handled on invoice details page (1.62 KB, patch)
2014-05-20 11:42 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 12274: Invoice detail: billingdate is badly formated (1.34 KB, patch)
2014-05-20 11:43 UTC, Jonathan Druart
Details | Diff | Splinter Review
[PASSED QA] Bug 12274 - Closed invoices not properly handled on invoice details page (1.70 KB, patch)
2014-05-21 10:31 UTC, Katrin Fischer
Details | Diff | Splinter Review
[PASSED QA] Bug 12274: Invoice detail: billingdate is badly formated (1.46 KB, patch)
2014-05-21 10:32 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jacek Ablewicz 2014-05-17 08:27:05 UTC
Symptoms:
- closed invoices are displayed as "Open" on details page (acqui/invoice.pl),
- it's not possible to reopen the closed invoice using "Edit" button.

(see also bug 3050 comment #11)
Comment 1 Jacek Ablewicz 2014-05-17 09:08:53 UTC
Ouch; it's not an "Edit" button, should be "Save" button instead in comment #1..
Comment 2 Jacek Ablewicz 2014-05-17 09:12:01 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-05-20 11:42:58 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-05-20 11:43:07 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2014-05-20 11:44:17 UTC
And I found another bug introduced by bug 10613.

This is critical for the release, I raise the severity.
Comment 6 Katrin Fischer 2014-05-21 10:31:35 UTC
Created attachment 28395 [details] [review]
[PASSED QA] Bug 12274 - Closed invoices not properly handled on invoice details page

Trivial fix for small regression (closed invoices are displayed as
"Open" on details page, and it's not possible to reopen the closed
invoice using "Save" button) introduced by bug 10613.

Test plan:

1) Create and close some invoices
2) Note that closed invoices are erroneously displayed as "Open"
on individual invoice[s] details page
3) Apply patch
4) Check previously closed invoices; their status on details page
should now be properly displayed as "Closed on ..." (and an option
for reopening would reappear as well)
5) Ensure that "Close" / "Reopen" checkboxes followed by "Save" button
do work as expected for individual open / closed invoices respectivelly.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described.
Comment 7 Katrin Fischer 2014-05-21 10:32:13 UTC
Created attachment 28396 [details] [review]
[PASSED QA] Bug 12274: Invoice detail: billingdate is badly formated

Bug 10613 sent the billingdate as a string. The template waits a DateTime
object.

To reproduce:
1/ Go on a invoice detail page
2/ Select a billing date
3/ Boom without the patch
[Tue May 20 13:39:18 2014] invoice.pl: Template process failed: undef
error - The 'day' parameter ("2014") to DateTime::new did not pass the
'an integer which is a possible valid day of month' callback.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Not all dates will make it go 'boom' but 31/07/2014 did.
Comment 8 Galen Charlton 2014-05-21 19:23:32 UTC
Pushed to master.  Thanks, Jacek and Jonathan!