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

David Nind <da...@davidnind.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #179300|0                           |1
        is obsolete|                            |

--- Comment #11 from David Nind <da...@davidnind.com> ---
Created attachment 182280
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182280&action=edit
Bug 25318: (follow-up) Make searchType be defined in merge.tt

With authorities_js.inc doing "var searchType = '[% marclist | html %]';" in
merge.tt where marclist isn't defined we got "var searchType = '';". This
patch just adds 'const searchType = ""' directly to merge.tt to avoid a
JavaScript console error.

Test plan:
1. Open the devtools console
2. Do an authority search that returns at least two authorities
3. Select one, then another for merge
4. When the merge.pl page loads, observe an "Uncaught ReferenceError" in
   the console
5. Apply followup, repeat steps 1-3, without an "Uncaught ReferenceError"

Signed-off-by: David Nind <da...@davidnind.com>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to