Bug 13166 - add all restrictions purge to cleanup_database.pl
authorFridolin Somers <fridolin.somers@biblibre.com>
Thu, 30 Oct 2014 10:59:28 +0000 (11:59 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Thu, 9 Apr 2015 14:59:41 +0000 (11:59 -0300)
commit16828155809fb8d977b04ac2a89df8a2ba239518
tree7472b59aaf98408d18c4a2c50d2e908ad1e192ba
parent7a94c50fa6b51e25586d3a02a72cb17987649e7a
Bug 13166 - add all restrictions purge to cleanup_database.pl

Bug 12760 adds the ability to purge expired restrictions older than some days.

But if you want to purge all expired restrictions, using "--restrictions 0" does not work, it's like "--restrictions" so it uses default purge days.

This patch adds a new option "--all-restrictions" to purge all expired restrictions.

Test plan :
- Select a borrower
- Create a restriction with expiration date in the futur
- Create a restriction expired since 1 day
- Create a restriction expired since 10 days
- run without argument "misc/cronjobs/cleanup_database.pl"
=> You see help text for --all-restrictions option
- run "misc/cronjobs/cleanup_database.pl -v --restrictions --all-restrictions"
=> You get the message : You can not specify both --restrictions and --all-restrictions
- run "misc/cronjobs/cleanup_database.pl -v --restrictions 30"
=> no restriction is removed
- run "misc/cronjobs/cleanup_database.pl -v --restrictions 9"
=> restriction expired since 10 days is removed
- run "misc/cronjobs/cleanup_database.pl -v --all-restrictions"
=> restriction expired since 1 day is removed

Signed-off-by: Larry Baerveldt <larry@bywatersolutions.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
misc/cronjobs/cleanup_database.pl