The advanced editor has a hardcoded use of 'zed' for the type of server, which implies Z39.50 and not SRU.
Created attachment 188596 [details] [review] Bug 41136: Allow advanced editor to search SRU servers This patch makes the advanced editor be able to search on SRU servers also, instead of only Z39.50. To test: 0. On a fresh KTD 1. Enable and open the cataloguing advanced editor 2. Click on 'Advanced >>' on the left column => FAIL: Only non-SRU targets are displayed 3. Apply this patch and reload 4. Repeat 2 => SUCCESS: SRU server displayed! => SUCCESS: No authority server is displayed due to the change i.e. behavior remains unchanged besides SRU support 5. Choose the sample SRU server and make a search => SUCCESS: Things work 6. Sign off :-D Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
This patch makes the advanced editor be able to search on SRU servers also, instead of only Z39.50. To test: 0. On a fresh KTD 1. Enable and open the cataloguing advanced editor 2. Click on 'Advanced >>' on the left column => FAIL: Only non-SRU targets are displayed 3. Apply this patch and reload 4. Repeat 2 => SUCCESS: SRU server displayed! => SUCCESS: No authority server is displayed due to the change i.e. behavior remains unchanged besides SRU support 5. Choose the sample SRU server and make a search => SUCCESS: Things work 6. Sign off :-D Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Created attachment 188607 [details] [review] Bug 41136: Allow advanced editor to search SRU servers This patch makes the advanced editor be able to search on SRU servers also, instead of only Z39.50. To test: 0. On a fresh KTD 1. Enable and open the cataloguing advanced editor 2. Click on 'Advanced >>' on the left column => FAIL: Only non-SRU targets are displayed 3. Apply this patch and reload 4. Repeat 2 => SUCCESS: SRU server displayed! => SUCCESS: No authority server is displayed due to the change i.e. behavior remains unchanged besides SRU support 5. Choose the sample SRU server and make a search => SUCCESS: Things work 6. Sign off :-D Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Created attachment 188608 [details] [review] Bug 41136: Allow advanced editor to search SRU servers
This patch makes the advanced editor be able to search on SRU servers also, instead of only Z39.50. To test: 0. On a fresh KTD 1. Enable and open the cataloguing advanced editor 2. Click on 'Advanced >>' on the left column => FAIL: Only non-SRU targets are displayed 3. Apply this patch and reload 4. Repeat 2 => SUCCESS: SRU server displayed! => SUCCESS: No authority server is displayed due to the change i.e. behavior remains unchanged besides SRU support 5. Choose the sample SRU server and make a search => SUCCESS: Things work 6. Sign off :-D Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>