[BUGFIX] Remove SelectImage.initEventListener 10/42410/4
authorManuel Selbach <manuel_selbach@yahoo.de>
Sat, 8 Aug 2015 14:02:35 +0000 (16:02 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Sun, 9 Aug 2015 21:30:24 +0000 (23:30 +0200)
This removes the calls of SelectImage.initEventListener
and deprecate the method.

Resolves: #68860
Releases: master
Change-Id: I3c0df4b4d15fb5da5dcb8e008e63496417c875ef
Reviewed-on: http://review.typo3.org/42410
Reviewed-by: Daniel Maier <dani-maier@gmx.de>
Tested-by: Daniel Maier <dani-maier@gmx.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/core/Documentation/Changelog/master/Deprecation-68860-DeprecateSelectImageinitEventHandler.rst [new file with mode: 0644]
typo3/sysext/rtehtmlarea/Classes/SelectImage.php
typo3/sysext/rtehtmlarea/Resources/Public/JavaScript/Modules/SelectImage.js

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Deprecation-68860-DeprecateSelectImageinitEventHandler.rst b/typo3/sysext/core/Documentation/Changelog/master/Deprecation-68860-DeprecateSelectImageinitEventHandler.rst
new file mode 100644 (file)
index 0000000..3c2f940
--- /dev/null
@@ -0,0 +1,29 @@
+============================================================
+Deprecation: #68860 - Deprecate SelectImage.initEventHandler
+============================================================
+
+Description
+===========
+
+Removes the calls of SelectImage.initEventHandler method.
+Was limited for UserAgent WebKit and the provided implementation
+of require was wrong, so that the EventListener was not
+registered at all. Nevertheless the functionality of
+drag and drop is not broken without the initEventHandler.
+
+Impact
+======
+
+Throws console log with deprecation message
+
+
+Affected Installations
+======================
+
+All where SelectImage.initEventHandler method is called
+
+
+Migration
+=========
+
+Remove call of SelectImage.initEventHandler method
index 12f3583..cdbd412 100644 (file)
@@ -141,7 +141,6 @@ class SelectImage extends ElementBrowser {
                parent::initDocumentTemplate();
 
                $this->doc->bodyTagId = 'typo3-browse-links-php';
-               $this->doc->bodyTagAdditions = $this->getBodyTagAdditions();
                $this->getPageRenderer()->loadRequireJsModule('TYPO3/CMS/Backend/LegacyTree', 'function(Tree) {
                        Tree.ajaxID = "SC_alt_file_navframe::expandCollapse";
                }');
@@ -160,8 +159,10 @@ class SelectImage extends ElementBrowser {
         * Provide the additional parameters to be included in the template body tag
         *
         * @return string the body tag additions
+        * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         */
        public function getBodyTagAdditions() {
+               GeneralUtility::logDeprecatedFunction();
                return 'onload="SelectImage.initEventListeners();"';
        }
 
index 7f22751..f557c43 100644 (file)
@@ -35,13 +35,12 @@ define('TYPO3/CMS/Rtehtmlarea/Modules/SelectImage', function () {
                 * Initialize an event handler for dropping an image in WebKit browsers
                 *
                 * @return void
+         * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
                 */
                initEventListeners: function() {
-                       require(['TYPO3/CMS/Rtehtmlarea/HTMLArea/UserAgent/UserAgent'], function (UserAgent, Event) {
-                               if (UserAgent.isWebKit) {
-                                       Event.one(window.document.body, 'dragend.TYPO3Image', function (event) { plugin.onDrop(event); });
-                               }
-                       });
+                       if (typeof console !== 'undefined') {
+                               console.log('SelectImage.initEventListeners() is deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8');
+                       }
                },
 
                /**