Bug 37546: Error message output improvements

As suggested by David, I've re-arranged the message output so we pass
'message' first, followed by the error reference inside brackets,
followed by "for SERVER" and finally "result No."

This follows the Z39.50 suggestions more closely whilst still including
our level of detail for Server + Result number.

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Martin Renvoize 2025-01-13 07:50:00 +00:00 committed by Katrin Fischer
parent 03afbb442c
commit d32a1b7063
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834

View file

@ -123,7 +123,7 @@
[% IF ( errcon.error == '10000' ) %]<li>Connection failed to [% errcon.server | html %]</li>
[% ELSIF ( errcon.error == '10007' ) %]<li>Connection timeout to [% errcon.server | html %]</li>
[% ELSIF errcon.error.match( '^XSLTH' ) %]<li>[% errcon.server | html %] record [% errcon.seq | html %]: [% PROCESS 'xslt-handler.inc' code=errcon.error %]</li>
[% ELSE %]<li>[% errcon.server | html %] record [% errcon.seq | html %]: [% errcon.error | html %] - [% errcon.error_msg | html %]</li>
[% ELSE %]<li>[% errcon.error_msg | html %] (ref. [% errcon.error | html %]) for [% errcon.server | html %] result [% errcon.seq | html %]</li>
[% END %]
[% END %]
</ul>