[Koha-bugs] [Bug 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Fridolin Somers  changed:

   What|Removed |Added

 Status|Needs Signoff   |Failed QA

--- Comment #18 from Fridolin Somers  ---
+use C4::Letters;
+use C4::Acquisition;


Needs to be :
+ use C4::Letters qw( GetPreparedLetter EnqueueLetter );

C4::Acquisition seems not used anymore

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #17 from Fridolin Somers  ---
+my $basket = $schema->resultset('Aqbookseller')->find(
+{ id => $basketno },
+{ result_class => 'DBIx::Class::ResultClass::HashRefInflator' })

We can now use Koha::Acquisition::Bookseller no ?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #16 from Alex Arnaud  ---
Created attachment 112981
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112981=edit
Bug 20125: Make Koha::Acquisition::Orders->filter_by_lates usable in command
line scripts

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #15 from Alex Arnaud  ---
Created attachment 112977
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=112977=edit
Bug 20125: Take into account new table aqorders_claims

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Amit Gupta  changed:

   What|Removed |Added

 CC||amit.gupta@informaticsgloba
   ||l.com,
   ||amitddng...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Fridolin SOMERS  changed:

   What|Removed |Added

 CC||fridolin.som...@biblibre.co
   ||m

--- Comment #14 from Fridolin SOMERS  ---
It may be missing "=cut" in misc/cronjobs/ordersClaim.pl

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

M. Tompsett  changed:

   What|Removed |Added

 CC||mtomp...@hotmail.com

--- Comment #13 from M. Tompsett  ---
perlcritic -1 ... WOW! It's so clean compared to most Koha things.

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #74837|0   |1
is obsolete||

--- Comment #12 from Alex Arnaud  ---
Created attachment 74839
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74839=edit
Bug 20125: QA followup   - fix a QA script warning   - rename script
description   - add letter-code parameter to ordersClaim script   - use, in
order, library email or admin email as from address

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #74767|0   |1
is obsolete||

--- Comment #11 from Alex Arnaud  ---
Created attachment 74838
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74838=edit
Bug 20125: Add a script to claim late orders (ordersClaim.pl)

Test plan:

 - Put some orders late from different baskets by changing the closedate
   column in aqbasket table,

Testing delay parameters:
 - launch "perl misc/cronjobs/ordersClaim.pl" with the following
   parameters:
   - delay: Up to the number of days from the basket's closedate,
   - claimed-for: no matter for the first claim,
 - Check that there is one entry in message_queue *for each basket with
   late order(s)*
 - Check these late orders in database:
   - claims_count column should be set to 1,
   - claimed_date should be today

Testing claimed-for parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first test,
   - claimed-for: 3 (if set claimed_date 2 days back),
   - max-claims: at least 2
 - check that there is no new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   claimed-for to 1
 - There should be new entry in message_queue
 - claims_count column should be set to 2
 - claimed_date should be today

Testing max-claims parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first and second test,
   - claimed-for: 1 (if set claimed_date 2 days back),
   - max-claims: 1 and 2 (test the two options),
 - There should be new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   max-claims to 3
 - There should be new entry in message_queue
 - claims_count column should be set to 3
 - claimed_date should be today

Signed-off-by: Séverine QUEUNE 

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #10 from Alex Arnaud  ---
Created attachment 74837
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74837=edit
Bug 20125: QA followup   - fix a QA script warning   - rename script
description   - add letter-code parameter to ordersClaim script   - use, in
order, library email or admin email as from address

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Katrin Fischer  changed:

   What|Removed |Added

 Status|Signed Off  |Failed 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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #9 from Katrin Fischer  ---
Comment on attachment 74767
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74767
Bug 20125 - Add a script to claim late orders (ordersClaim.pl)

Review of attachment 74767:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html=20125=74767)
-

Hi Alex, 

a few small things pointed out by QA script:

 FAIL   misc/cronjobs/ordersClaim.pl
   OK critic
   OK forbidden patterns
   OK git manipulation
   FAIL   pod

 in file misc/cronjobs/ordersClaim.pl
*** ERROR: 
=over on line 54 without closing =back
 at line EOF
   OK spelling
   OK valid


Processing additional checks

* Commit title does not start with 'Bug X: ' - 18e290c

And some questions and notes from first review:

::: misc/cronjobs/ordersClaim.pl
@@ +18,5 @@
> +# along with Koha; if not, see .
> +
> +=head1 NAME
> +
> +ordersClaim.pl - cron script that put late orders in message queue grouped 
> by basket.

... that puts
Or maybe:
con script to automatically generate late order claims in the message queue
grouped by basket.

@@ +129,5 @@
> +next unless @$orders;
> +
> +my $letter = GetPreparedLetter(
> +module => 'claimacquisition',
> +letter_code => 'ACQCLAIM',

The letter code for claiming acquisitions can be different from the sample
provided by Koha. Koha only checks for the module and allows you to have
multiple messages (for example in different languages). Can we make this a
script option to be more flexible? Should at lest be documented.

@@ +139,5 @@
> +},
> +repeat => $orders,
> +) or next;
> +
> +my $admin_address = C4::Context->preference('KohaAdminEmailAddress');

Maybe use the library from the basket if set first and only fall back to
KohaAdminEmailAddress?

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Séverine Queune  changed:

   What|Removed |Added

 CC||severine.que...@bulac.fr

--- Comment #8 from Séverine Queune  ---
Works as described, thanks for this very clear test plan !

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Séverine Queune  changed:

   What|Removed |Added

  Attachment #71274|0   |1
is obsolete||

--- Comment #7 from Séverine Queune  ---
Created attachment 74767
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=74767=edit
Bug 20125 - Add a script to claim late orders (ordersClaim.pl)

Test plan:

 - Put some orders late from different baskets by changing the closedate
   column in aqbasket table,

Testing delay parameters:
 - launch "perl misc/cronjobs/ordersClaim.pl" with the following
   parameters:
   - delay: Up to the number of days from the basket's closedate,
   - claimed-for: no matter for the first claim,
 - Check that there is one entry in message_queue *for each basket with
   late order(s)*
 - Check these late orders in database:
   - claims_count column should be set to 1,
   - claimed_date should be today

Testing claimed-for parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first test,
   - claimed-for: 3 (if set claimed_date 2 days back),
   - max-claims: at least 2
 - check that there is no new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   claimed-for to 1
 - There should be new entry in message_queue
 - claims_count column should be set to 2
 - claimed_date should be today

Testing max-claims parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first and second test,
   - claimed-for: 1 (if set claimed_date 2 days back),
   - max-claims: 1 and 2 (test the two options),
 - There should be new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   max-claims to 3
 - There should be new entry in message_queue
 - claims_count column should be set to 3
 - claimed_date should be today

Signed-off-by: Séverine QUEUNE 

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Séverine Queune  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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Fridolin SOMERS  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=20119

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #6 from Katrin Fischer  ---
Ah, so it's optional? Great :)

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #5 from Alex Arnaud  ---
(In reply to Katrin Fischer from comment #2)
> Just wondering: we configure a delay value for each vendor, can this be used
> in the script? I think it would make sense and have the delay value as a
> possible overwrite if you haven't used the field maybe?

The delay paramter of the script is set to the vendor's delivery time by
default (if not used in the command)

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  changed:

   What|Removed |Added

  Attachment #71139|0   |1
is obsolete||

--- Comment #4 from Alex Arnaud  ---
Created attachment 71274
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71274=edit
Bug 20125 - Add a script to claim late orders (ordersClaim.pl)

Test plan:

 - Put some orders late from different baskets by changing the closedate
   column in aqbasket table,

Testing delay parameters:
 - launch "perl misc/cronjobs/ordersClaim.pl" with the following
   parameters:
   - delay: Up to the number of days from the basket's closedate,
   - claimed-for: no matter for the first claim,
 - Check that there is one entry in message_queue *for each basket with
   late order(s)*
 - Check these late orders in database:
   - claims_count column should be set to 1,
   - claimed_date should be today

Testing claimed-for parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first test,
   - claimed-for: 3 (if set claimed_date 2 days back),
   - max-claims: at least 2
 - check that there is no new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   claimed-for to 1
 - There should be new entry in message_queue
 - claims_count column should be set to 2
 - claimed_date should be today

Testing max-claims parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first and second test,
   - claimed-for: 1 (if set claimed_date 2 days back),
   - max-claims: 1 and 2 (test the two options),
 - There should be new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   max-claims to 3
 - There should be new entry in message_queue
 - claims_count column should be set to 3
 - claimed_date should be today

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #3 from Katrin Fischer  ---
It's called grace period actually in the interface.

The thought is that you can treat vendors differently. Your local bookstore
will be usually faster than something you ordered from outside the country.

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Katrin Fischer  changed:

   What|Removed |Added

 CC||katrin.fisc...@bsz-bw.de

--- Comment #2 from Katrin Fischer  ---
Just wondering: we configure a delay value for each vendor, can this be used in
the script? I think it would make sense and have the delay value as a possible
overwrite if you haven't used the field maybe?

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

--- Comment #1 from Alex Arnaud  ---
Created attachment 71139
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71139=edit
Bug 20125 - Add a script to claim late orders (ordersClaim.pl)

Test plan:

 - Put some orders late from different baskets by changing the closedate
   column in aqbasket table,

Testing delay parameters:
 - launch "perl misc/cronjobs/ordersClaim.pl" with the following
   parameters:
   - delay: Up to the number of days from the basket's closedate,
   - claimed-for: no matter for the first claim,
 - Check that there is one entry in message_queue *for each basket with
   late order(s)*
 - Check these late orders in database:
   - claims_count column should be set to 1,
   - claimed_date should be today

Testing claimed-for parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first test,
   - claimed-for: 3 (if set claimed_date 2 days back),
   - max-claims: at least 2
 - check that there is no new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   claimed-for to 1
 - There should be new entry in message_queue
 - claims_count column should be set to 2
 - claimed_date should be today

Testing max-claims parameter:
 - change the claimed_date column in database for all orders (i.e 2 days
   back),
 - launch "perl misc/cronjobs/ordersClaim.pl" again with the following
   parameters:
   - delay: Same than for the first and second test,
   - claimed-for: 1 (if set claimed_date 2 days back),
   - max-claims: 1 and 2 (test the two options),
 - There should be new entry in message_queue
 - launch "perl misc/cronjobs/ordersClaim.pl" again by moving only
   max-claims to 3
 - There should be new entry in message_queue
 - claims_count column should be set to 3
 - claimed_date should be today

-- 
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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  changed:

   What|Removed |Added

 Status|NEW |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 20125] Add a cron script to claim late orders

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20125

Alex Arnaud  changed:

   What|Removed |Added

 QA Contact|testo...@bugs.koha-communit |alex.arn...@biblibre.com
   |y.org   |
   Assignee|koha-b...@lists.koha-commun |alex.arn...@biblibre.com
   |ity.org |

-- 
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/