Bug 21389: Correct Javascript error on article requests page
authorFridolin Somers <fridolin.somers@biblibre.com>
Fri, 21 Sep 2018 09:17:25 +0000 (11:17 +0200)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 15 Oct 2018 12:47:05 +0000 (13:47 +0100)
commitb0f3bc8c134c195e3d943a0f7f9d5ff6a064da79
tree236c483e505c8e62c1a1e03258ceeea9fc9f37da
parentef93ed60acf10d47a92669490eff1acda3abc135
Bug 21389: Correct Javascript error on article requests page

Bug 21076 fixed a Javascript error on article requests page when patron is found.

There is also an error when patron is not found, with message :
  Patron not found
  No patron with this name, please, try another

This patch fixes by replacing TT condition by a JS condition testing the existance of the patron input.

Test plan:
1 - Do not apply patch
2 - Enable article requests
3 - Search on staff client and click 'Request article'
4 - Search for a patron that does not exist
5 - You get a yellow message "Patron not found"
6 - View error in console
7 - Apply patch
8 - Reload
9 - Not error is gone
10 - Search for a patron that does exist
11 - Check there is no error in console
12 - Check that autocompletion works

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit b27db0fbfc0b715df1b63c57228a9945d4834f29)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/circ/request-article.tt