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

Marcel de Rooy <[email protected]> changed:

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

--- Comment #6 from Marcel de Rooy <[email protected]> ---
Created attachment 184976
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=184976&action=edit
Bug 40150: Prevent uncaught error on multiple attempts to 'define' on
'CustomElementsRegistry' in islands.ts

If the name is already in use, we just return.

It might make sense to have a console.debug() or similar to notify
developers that this is happening. Open for other opinions.

To test:
1) Apply the first patch which adds an additional hydrate call in
main-container.inc
2) Go to acqui/acqui-home.pl
3) Verify that you see this error in the dev tool's console tab:
Uncaught (in promise) NotSupportedError: Failed to execute 'define' on
'CustomElementRegistry': the name "acquisitions-menu" has already been used
with this registry
4) Apply the second patch
5) Run `yarn js:build` in ktd (assuming ktd).
6) Reload the page without cached assets (Ctrl/Cmd + Shift + R) and verify that
the uncaught error is gone.
7) Sign off.

There's an additional problem here I (and probably you) noticed. But that's
unrelated to this bug.

Signed-off-by: David Nind <[email protected]>

Signed-off-by: Marcel de Rooy <[email protected]>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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