Jonathan Druart
37370c415a
On bug 24386 we prevent double form submission using the our own preventDoubleFormSubmit JS function. The problem is that we are checking some conditions, and prevent the form submission if something is not filled (for instance no checkbox checked, or no fund selected). Technically it means that: - click the submit button - it submits the form - we disable the submit button - we prevent the form to be submitted before something is wrong At this stage the button is disabled and the form cannot be longer be submitted. This patch replaces the "on submit" event of the form with the "on click" event of the submit button. Which means we are going to: - click the submit button - we prevent the form to be submitted before something is wrong => The button will only be disabled if the form is really submitted Test plan: - stage a marc record - acquisitions: have a basket - click on "Add to basket" - "From a staged file" - Don't tick the record - Save - You should see an expected error message - Tick the record - Save - You should see an expected error message - Choose a fund - Click the "Save" button as many times as you can, to try double submit it. => The order is saved Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> |
||
|---|---|---|
| .. | ||
| pages | ||
| acq.js | ||
| acquisitions-menu.js | ||
| additem.js | ||
| addorderiso2709.js | ||
| admin-menu.js | ||
| ajax.js | ||
| audio_alerts.js | ||
| auth-finder-search.js | ||
| automatic_item_modification_by_age.js | ||
| background-job-progressbar.js | ||
| basket.js | ||
| basketgroup.js | ||
| biblio_framework.js | ||
| cart.js | ||
| catalog.js | ||
| cataloging.js | ||
| cataloging_additem.js | ||
| categories.js | ||
| charts.js | ||
| checkouts.js | ||
| circ-patron-search-results.js | ||
| commons.js | ||
| datatables.js | ||
| file-upload.js | ||
| funds_sorts.js | ||
| holds.js | ||
| ill-availability-partner.js | ||
| ill-availability.js | ||
| ill-list-table.js | ||
| item_search_fields.js | ||
| letter.js | ||
| localcovers.js | ||
| mana.js | ||
| marc_modification_templates.js | ||
| marc_subfields_structure.js | ||
| members-menu.js | ||
| members.js | ||
| merge-record.js | ||
| messaging-preference-form.js | ||
| offlinecirc.js | ||
| onboarding.js | ||
| rotating-collections.js | ||
| serials-toolbar.js | ||
| showpredictionpattern.js | ||
| sms_providers.js | ||
| staff-global.js | ||
| subscription-add.js | ||
| suggestions.js | ||
| table_filters.js | ||
| tools-menu.js | ||
| viewlog.js | ||
| xmlControlfield.js | ||
| z3950_search.js | ||