[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Julian Maurice julian.maur...@biblibre.com changed: What|Removed |Added CC||julian.maur...@biblibre.com --- Comment #27 from Julian Maurice julian.maur...@biblibre.com --- I just tried to apply both patches on up-to-date 3.8.x branch in this order proposed patch to match 3.8.x Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl and had no conflicts. Furthermore, it works :) Can you give it a try again Chris ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Chris Cormack ch...@bigballofwax.co.nz changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable --- Comment #28 from Chris Cormack ch...@bigballofwax.co.nz --- Worked this time, pushed to 3.8.x will be in 3.8.8 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382
--- Comment #26 from Chris Cormack ch...@bigballofwax.co.nz ---
Yep,
You can see in the patch this bit
-my $datereceived = ($input-param('op') eq 'new') ?
C4::Dates-new($input-param('datereceived'))
-: C4::Dates-new($input-param('datereceived'), 'iso')
;
+my $datereceived = ($input-param('op') eq ('new' or search)) ?
C4::Dates-new($input-param('datereceived'))
+: C4::Dates-new($input-param('datereceived'), 'iso');
This is the bit that is conflicting, I can fix the conflict if you are sure the
search bit is to do with this change, not something that has already gone into
master?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #25 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Hi Chris, this is amazing, I have retried and for me it's ok on a branch 3.8.x (the last one is : commit e7b91d6a823401f73d4ef69ae80e045031995dfc Author: Fridolyn SOMERS fridolyn.som...@biblibre.com Date: Wed Sep 19 11:45:19 2012 +0200 Bug 8791: OPAC UNIMARC XSLT details: subject with subfield 3) And Fridolyn Somers has tried too, it's ok for him.. Have you take the right one ? = proposed patch to match 3.8.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #23 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Created attachment 12913 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12913action=edit proposed patch to match 3.8.x Hello Chris, done, I hope this one is ok. For the follow up I had no conflicts so it must be ok. thx -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382
--- Comment #24 from Chris Cormack ch...@bigballofwax.co.nz ---
(In reply to comment #23)
Created attachment 12913 [details]
proposed patch to match 3.8.x
Hello Chris,
done, I hope this one is ok.
For the follow up I had no conflicts so it must be ok.
thx
Unfortunately still has a conflict.
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
Auto-merging acqui/parcel.pl
CONFLICT (content): Merge conflict in acqui/parcel.pl
Auto-merging C4/Acquisition.pm
Looks like something to do search has snuck in
my $datereceived = ($input-param('op') eq ('new' or search))
Should that be in 3.8.x?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #22 from Chris Cormack ch...@bigballofwax.co.nz --- Original patch doesn't apply to 3.8.x, so the follow up doesn't either. Please resubmit both for 3.8.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Paul Poulain paul.poul...@biblibre.com changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #21 from Paul Poulain paul.poul...@biblibre.com --- follow-up pushed -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Mason James m...@kohaaloha.com changed: What|Removed |Added Status|Signed Off |Passed QA CC||m...@kohaaloha.com QA Contact||m...@kohaaloha.com --- Comment #20 from Mason James m...@kohaaloha.com --- (In reply to comment #16) Created attachment 12272 [details] Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl In order to be valid XHTML ampersands must be encoded (amp;). Signed-off-by: Jonathan Druart jonathan.dru...@biblibre.com passing QA, trival patch $ koha-qa.pl -c 1 testing 1 commit(s) (applied to commit 0a35b26) * 74d0ff6 Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt * koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt OK -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply CC||k...@bywatersolutions.com --- Comment #18 from Kyle M Hall k...@bywatersolutions.com --- Applying: Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl Bug 8382: Followup: FIX wrong comparaison Apply? [yn] y Applying: Bug 8382: Followup: FIX wrong comparaison error: patch failed: acqui/parcel.pl:79 error: acqui/parcel.pl: patch does not apply Patch failed at 0001 Bug 8382: Followup: FIX wrong comparaison -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Attachment #12273|0 |1 is obsolete|| --- Comment #19 from Jonathan Druart jonathan.dru...@biblibre.com --- Comment on attachment 12273 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12273 Bug 8382: Followup: FIX wrong comparaison This patch seems useless now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382
--- Comment #15 from Jonathan Druart jonathan.dru...@biblibre.com ---
Christophe: A followup is mandatory !
This line does not do what we want:
+my $datereceived = ($input-param('op') eq ('new' or search)) ?
C4::Dates-new($input-param('datereceived'))
+: C4::Dates-new($input-param('datereceived'), 'iso');
$s eq ('foo' or 'bar') does not return true if $s = 'bar'
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Attachment #11653|0 |1 is obsolete|| Attachment #11982|0 |1 is obsolete|| --- Comment #16 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 12272 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12272action=edit Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl In order to be valid XHTML ampersands must be encoded (amp;). Signed-off-by: Jonathan Druart jonathan.dru...@biblibre.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #17 from Jonathan Druart jonathan.dru...@biblibre.com --- Created attachment 12273 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12273action=edit Bug 8382: Followup: FIX wrong comparaison -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|Pushed to Master|ASSIGNED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Owen Leonard oleon...@myacpl.org changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Paul Poulain paul.poul...@biblibre.com changed: What|Removed |Added Status|Passed QA |Pushed to Master Version|master |rel_3_8 Severity|major |normal --- Comment #12 from Paul Poulain paul.poul...@biblibre.com --- Patch pushed to master I've tested it, it's OK, patch pushed. But i'm a little bit uncomfortable with this patch. I can't say why, so I've pushed it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #13 from Owen Leonard oleon...@myacpl.org --- Created attachment 11982 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11982action=edit Bug 8382 [Follow-up] It is not possible to cancel an order when receiving on a filtered page in parcel.pl In order to be valid XHTML ampersands must be encoded (amp;). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Chris Cormack ch...@bigballofwax.co.nz changed: What|Removed |Added CC||ch...@bigballofwax.co.nz --- Comment #14 from Chris Cormack ch...@bigballofwax.co.nz --- Ill wait until the follow up is pushed before deciding whether to push this to 3.8.x or not -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Paul Poulain paul.poul...@biblibre.com changed: What|Removed |Added CC||paul.poul...@biblibre.com Severity|blocker |major --- Comment #11 from Paul Poulain paul.poul...@biblibre.com --- This is not a blocker, as we can deal with recieving without filtering (even if it's a major problem I agree). Due to the number of passed QA bugs, i'll deal with this one later -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Ian Walls koha.sek...@gmail.com changed: What|Removed |Added Status|Signed Off |Passed QA CC||koha.sek...@gmail.com --- Comment #10 from Ian Walls koha.sek...@gmail.com --- Reimplements search as server-side, rather than client side with Javascript and JSON, so two large chunks of code are removed. Changes to GetPendingOrders function are clear and safe; new parameters are added at the end, so existing function calls do not need to be changed, and while $booksellerid is renamed to $supplierid internally, it's kept consistent within the function. POD for GetPendingOrders is updated, but the new parameters are not sufficiently described. Please provide more details on what these parameters actually do (a followup patch would be appropriate for this). Furthermore, unit tests should be expanded to cover the addition of the new filter options. A followup can be provided for this, as well. I'm still defrosting after a long absence, so all my tests are not yet working for other reasons, but to the best of my knowledge, this patch is safe and can pass QA. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #8 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Hello wajasu, sorry if I was not clear for the test, your snapshots are useful to explain: 1° without patch: if you look at the first snapshot you see a column without title on the right of 'Order cost' column, this column have Receive/Cancel, two explicit links. But if you click on Cancel, this link doesn't cancel the order. That is the bug. 2° with the patch: we can see on your second snapshot we have now two columns, the second one offers the same choice than the normal page without filter, I mean 'Delete order' and depending on the situation of the biblio record, 'Delete order and catalog record' or 'Can't delete...' in our case. And of course if you click on the link 'Delete...' it does its job, no more bug. I hope I am clear, so the first check of my patch is when you look at your snapshots (the right column(s)) changes, and the second is to try to cancel an order without my patch (it doesn't cancel) and with my patch (cancel successful). So thx for re testing wajasu. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 wajasu matted-34...@mypacks.net changed: What|Removed |Added Attachment #11142|0 |1 is obsolete|| --- Comment #9 from wajasu matted-34...@mypacks.net --- Created attachment 11653 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11653action=edit [SIGNED-OFF] 0001-bug-8382-It-is-not-possible-to-cancel-an-.patch I think I understand. In aquisitions, when recieving orders, one may want to cancel and order, or specifically, an order line. After finding the vendor for the order, and clicking receive shipment button, and keying in the vendor's invoice number and Next button, you see the order lines. If you use the javascipt filter widget at the left of the page, and put the order line number in it and clock Filter button, a NEW column containing Receive/Cancel links shows up. (see PREPATCH attachment image) The bug is when you click Cancel the order is not cancelled. The patch writer, reworked this so now with the new patch, after using a filter, no Receive/Cancel links appear anymore, and you can just click Delete Order to cancel the order entry. I used the patch and the resulting screen showed the cancelled order entry in a section below. So it seems to work fine. Note: I had to add funds, budgets, vendors, order books, a new staff user (since sysadmin get warning it must define a budget), plus figure out how to get into this workflow point. I hope the workflow gets improved in the future. wajasu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 wajasu matted-34...@mypacks.net changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #6 from wajasu matted-34...@mypacks.net --- Created attachment 11630 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11630action=edit PREPATCH acquisitions_recieveorder_with filter_withReceiveCancel.png I attempted a test, but to clarify where this applies, I attached this PREPATCH image. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #7 from wajasu matted-34...@mypacks.net --- Created attachment 11631 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11631action=edit POSTPATCH acquisitions_receiveorder_with filter_withoutReceiveCancel.png After applying the patch, I could not see the Receive/Cancel link to try to Cancel. I wasn't sure if this is what you wanted, so I attached a POSTPATH image of what I see. Maybe a clearer test plan, or a further fix is needed. wajasu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 wajasu matted-34...@mypacks.net changed: What|Removed |Added Status|Needs Signoff |Failed QA CC||matted-34...@mypacks.net -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Attachment #10702|0 |1 is obsolete|| --- Comment #5 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Created attachment 11142 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=11142action=edit last patch right Jonathan. So I think it's ok now. thx -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Jonathan Druart jonathan.dru...@biblibre.com changed: What|Removed |Added Status|Needs Signoff |Failed QA CC||jonathan.dru...@biblibre.co ||m --- Comment #4 from Jonathan Druart jonathan.dru...@biblibre.com --- QA Comments: You can't modify the prototype of a routine without modifying all the occurrences ! grep is your friend: acqui/uncertainprice.pl:my $pendingorders = GetPendingOrders($booksellerid,0,$owner,$basketno); + Modify POD In the majority of cases, it is preferable to add the new parameters at the end of the parameter's list. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Status|NEW |ASSIGNED -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Priority|P5 - low|P1 - high -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |christophe.croullebois@bibl |ity.org |ibre.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 --- Comment #1 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Created attachment 10701 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10701action=edit proposed patch thx for testing -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 8382] It is not possible to cancel an order when receiving on a filtered page in parcel.pl
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8382 Christophe Croullebois christophe.croulleb...@biblibre.com changed: What|Removed |Added Attachment #10701|0 |1 is obsolete|| --- Comment #3 from Christophe Croullebois christophe.croulleb...@biblibre.com --- Created attachment 10702 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=10702action=edit proposed patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
