Bug 11433: (code cleanup) remove unused 'attachbasket' op value in basket.pl
authorMathieu Saby <mathieu.saby@univ-rennes2.fr>
Fri, 20 Dec 2013 22:21:32 +0000 (23:21 +0100)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 7 May 2014 15:19:21 +0000 (15:19 +0000)
commitf8f7382ef7267d53b61462773e300e517c48e9e8
tree20d243a822745df2c7c6d7423a142dbc8d6b5752
parent34e136a485c58c6911de59899f94a0543e483b0a
Bug 11433: (code cleanup) remove unused 'attachbasket' op value in basket.pl

This patch cleans code in basket.pl

In basket.pl, some code is supposed to be executed if
$op eq 'attachbasket'. But it is never the case
(grep attachbasket * -r), so this condition can be removed.

No functional change expected.

Regression test only :
* Make a complete acquisition process, from the creation of a basket
 to the closure of a basketgroup, and check everything is OK
* On a basket page, try to change the basketgroup it belongs to, and
  check everything is OK
* On a basketgroup page, try to edit the content of a basketgroup (put
  a new basket in it, change the deliverybranch...), and check everything
  is OK
* On a basketgroup page, try to reopen a closed basketgroup, and close an
  open basketgroup, and check everything is OK

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
acqui/basket.pl