Bug 24010: Number of issues to display to staff accepts non-integer values
authorOwen Leonard <oleonard@myacpl.org>
Fri, 13 May 2022 15:14:26 +0000 (15:14 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 5 Jul 2022 13:57:16 +0000 (10:57 -0300)
commit57221e083bb329f3c428fd20e7e7f65a6ab10186
tree68316e8a109384f1ea9ca0d06da76837c05bf445
parentb48ac1ef741d8e1c02290dd9d2809713f3bba23b
Bug 24010: Number of issues to display to staff accepts non-integer values

This patch modifies the subscription entry form so that it will perform
a check on the staffdisplaycount and opacdisplaycount fields before
proceding to the second step. It verifies that the values are numeric.

The changes are made in the style of the existing form validation, which
should be rewritten to either use the validation plugin or to peform
checks in a way that all checks are run before warning the user.
However, this smaller change will work in the meantime.

To test, apply the patch and go to Serials -> New subscription.

- Fill out the form with at least the required fields, but put something
  other than a number if the "Number of issues to display to staff" and
  "Number of issues to display to the public" with non-numeric characters.
- When you click the "Next" button you should get an error message,
  "Number of issues to display to staff must be a number."
- Correct the issues to display to staff field and submit again.
- You should get a different error message, "Number of issues to display
  to the public must be a number."
- Correct this field and you should be able to proceed to the next step.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt
koha-tmpl/intranet-tmpl/prog/js/subscription-add.js