[TASK] Make JavaScript Icon API a singleton

The JavaScript Icon API is now a singleton. This ensures the code is only
executed once and especially the cache is shared between all requests.

Resolves: #72126
Releases: master
Change-Id: Iafd61b90d2414f655e6500bc564b90490afc58b0
Reviewed-on: https://review.typo3.org/45196
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>

diff --git a/typo3/sysext/backend/Resources/Public/JavaScript/Icons.js b/typo3/sysext/backend/Resources/Public/JavaScript/Icons.js
index bc92203..4ba9789 100644 (file)
--- a/typo3/sysext/backend/Resources/Public/JavaScript/Icons.js
+++ b/typo3/sysext/backend/Resources/Public/JavaScript/Icons.js
@@ -18,6 +18,28 @@
 define(['jquery'], function($) {
        'use strict';
 
 define(['jquery'], function($) {
        'use strict';
 

+       try {
+               // fetch from opening window
+               if (window.opener && window.opener.TYPO3 && window.opener.TYPO3.Icons) {
+                       return window.opener.TYPO3.Icons;
+               }
+
+               // fetch from parent
+               if (parent && parent.window.TYPO3 && parent.window.TYPO3.Icons) {
+                       return parent.window.TYPO3.Icons;
+               }
+
+               // fetch object from outer frame
+               if (top && top.TYPO3.Icons) {
+                       return top.TYPO3.Icons;
+               }
+       } catch (e) {
+               // This only happens if the opener, parent or top is some other url (eg a local file)
+               // which loaded the current window. Then the browser's cross domain policy jumps in
+               // and raises an exception.
+               // For this case we are safe and we can create our global object below.
+       }
+

        /**
         *
         * @type {{cache: {}, sizes: {small: string, default: string, large: string, overlay: string}, states: {default: string, disabled: string}}}
        /**
         *
         * @type {{cache: {}, sizes: {small: string, default: string, large: string, overlay: string}, states: {default: string, disabled: string}}}


@@ -124,5 +146,8 @@ define(['jquery'], function($) {
                Icons.cache[cacheIdentifier] = markup;
        };
 
                Icons.cache[cacheIdentifier] = markup;
        };
 

+       // attach to global frame
+       TYPO3.Icons = Icons;
+

        return Icons;
 });
        return Icons;
 });