]> git.koha-community.org Git - koha.git/commit
Bug 15936 - Revise layout and behavior of SMS cellular providers management
authorOwen Leonard <oleonard@myacpl.org>
Sat, 20 Feb 2016 01:24:46 +0000 (20:24 -0500)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Tue, 8 Mar 2016 21:56:44 +0000 (21:56 +0000)
commit859e3ce41b69692471e9a1e37d646f9ed0bfe6dd
tree7ed5d2a856943242000c307ff20319e6ed41d0c3
parent357ddab7c6519fdbb57e448c28170c267035a130
 Bug 15936 - Revise layout and behavior of SMS cellular providers management

This patch changes the behavior of the SMS cellular providers management
page so that sections of the page are shown or hidden based on what the
current task is. Other changes include:

- Moving JavaScript to a separate file
- Adding a toolbar with a "New SMS provider" button.
- Adding the "generic" catalog search header form to the top of the
  page.
- Adding "Required" classes to required fields so that the staff
  client's built-in JS validation library can be used.

To test, the  SMSSendDriver system preference must be set to "Email."
Apply the patch and go to Administration -> SMS cellular providers.

- Confirm that the add/edit form is hidden initially. A toolbar with a
  "New SMS provider" button should appear with existing SMS cellular
  providers in a table below. If there are no existing providers a
  message dialog should say so.
- Confirm that the "New SMS provider" button works:
  - The table should be hidden and an empty "add" form displayed.
  - Confirm that an empty form cannot be submitted.
  - Confirm that submitting valid data works.
  - Confirm that clicking the "Cancel" button hides the form and
    redisplays the table.
- Test the "edit" button for an existing provider:
  - Confirm that the edit form is displayed and populated with the
    correct data.
  - Confirm that edits are saved correctly.
- When viewing the table of existing providers, confirm that clicking
  the "Delete" button triggers a confirmation dialog. Test both
  cancelling and confirming that dialog.
- Ponder whether all this is an improvement or not.

Signed-off-by: Nicole C Engard <nengard@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/js/sms_providers.js [new file with mode: 0644]
koha-tmpl/intranet-tmpl/prog/en/modules/admin/sms_providers.tt