Bug 19986

Summary:	'Server name' doesn't appear as required when creating new z39.50/sru server
Product:	Koha	Reporter:	Aleisha Amohia <aleisha>
Component:	Z39.50 / SRU / OpenSearch Servers	Assignee:	Grace <gracesmythh>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	gracesmythh, jonathan.druart, julian.maurice, m.de.rooy, nick
Version:	Main	Keywords:	Academy
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Attachments:	Bug 19986: 'Server name' doesn't appear as required Bug 19986: 'Server name' doesn't appear as required Bug 19986: 'Server name' doesn't appear as required

Description Aleisha Amohia 2018-01-16 23:34:36 UTC

The server name field is a required field, but it is not red and does not say required like the other required fields.

To reproduce:
1) Go to Admin -> z39.50/SRU servers
2) Make a new z39.50 server
3) Fill in the required fields (hostname, port, database)
4) Save the server
5) Notice the form does not submit, tells you that server name is required.

The behaviour is the same for making a new SRU server too, so also needs to be fixed there.

Comment 1 Grace 2018-01-17 20:54:24 UTC

Created attachment 70646 [details] [review]
Bug 19986: 'Server name' doesn't appear as required

Test plan:
[1.] Go to Admin -> z39.50/SRU servers
[2.] Make a new z39.50 server
[3.] Fill in the required field (hostname, port, database)
[4.] Save the server
[5.] Notice that the form does not submit, tells you that Sever name is
     required
[6.] Repeat steps 2 - 5 but this time make a new SRU server
[6.] Upload the patch and repeat steps 1 - 3 (make a new z39.50 server
     and a new SRRU server)
[7.] See that now 'Server name:' is red and on the right side of the box
     there is a required note
[8.] Save both servers and it should have created a new z39.50 server
and a new SRU server

https://bugs.koha-community.org/show_bug.cgi?id=19833

Comment 2 Grace 2018-01-17 20:55:29 UTC

(In reply to Grace from comment #1)
> Created attachment 70646 [details] [review] [review]
> Bug 19986: 'Server name' doesn't appear as required
> 
> Test plan:
> [1.] Go to Admin -> z39.50/SRU servers
> [2.] Make a new z39.50 server
> [3.] Fill in the required field (hostname, port, database)
> [4.] Save the server
> [5.] Notice that the form does not submit, tells you that Sever name is
>      required
> [6.] Repeat steps 2 - 5 but this time make a new SRU server
> [6.] Upload the patch and repeat steps 1 - 3 (make a new z39.50 server
>      and a new SRRU server)
> [7.] See that now 'Server name:' is red and on the right side of the box
>      there is a required note
> [8.] Save both servers and it should have created a new z39.50 server
> and a new SRU server
> 
> https://bugs.koha-community.org/show_bug.cgi?id=19833

P.S the link was added by accident and is not relevant to this bug :)

Comment 3 Roch D'Amour 2018-01-26 21:29:25 UTC

Created attachment 70985 [details] [review]
Bug 19986: 'Server name' doesn't appear as required

Test plan:
[1.] Go to Admin -> z39.50/SRU servers
[2.] Make a new z39.50 server
[3.] Fill in the required field (hostname, port, database)
[4.] Save the server
[5.] Notice that the form does not submit, tells you that Sever name is
     required
[6.] Repeat steps 2 - 5 but this time make a new SRU server
[6.] Upload the patch and repeat steps 1 - 3 (make a new z39.50 server
     and a new SRRU server)
[7.] See that now 'Server name:' is red and on the right side of the box
     there is a required note
[8.] Save both servers and it should have created a new z39.50 server
and a new SRU server

https://bugs.koha-community.org/show_bug.cgi?id=19833
Signed-off-by: Roch D'Amour <roch.damour@inlibro.com>

Comment 4 Julian Maurice 2018-01-29 16:32:09 UTC

Created attachment 71026 [details] [review]
Bug 19986: 'Server name' doesn't appear as required

Test plan:
[1.] Go to Admin -> z39.50/SRU servers
[2.] Make a new z39.50 server
[3.] Fill in the required field (hostname, port, database)
[4.] Save the server
[5.] Notice that the form does not submit, tells you that Sever name is
     required
[6.] Repeat steps 2 - 5 but this time make a new SRU server
[6.] Upload the patch and repeat steps 1 - 3 (make a new z39.50 server
     and a new SRRU server)
[7.] See that now 'Server name:' is red and on the right side of the box
     there is a required note
[8.] Save both servers and it should have created a new z39.50 server
and a new SRU server

Signed-off-by: Roch D'Amour <roch.damour@inlibro.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Comment 5 Jonathan Druart 2018-01-30 17:26:14 UTC

Pushed to master for 18.05, thanks to everybody involved!

Comment 6 Nick Clemens (kidclamp) 2018-02-12 20:57:43 UTC

Awesome work all!

This patch has been pushed to 17.11.x and will be in 17.11.03