Bug #40498
closed
embedded jquery
100%
Description
The new extension manager embeds a copy of jquery 1.7.1 in typo3/sysext/extensionmanager/Resources/Public/JavaScript/jquery.js which is also bundled with a plugin "jquery.dataTables.min.js".
jquery 1.8 is added to the core in typo3/contrib/jquery.
The new EM must use this copy instead of its own and also move the embedded plugin to a proper location under typo3/contrib.
Updated by Christian Welzel over 11 years ago
There are many other embedded code copies in typo3/sysext/extensionmanager/Resources/Public/Contrib/ and
typo3/sysext/extensionmanager/Resources/Public/JavaScript/.
Please move them to a proper location under typo3/contrib/ too.
Updated by Kai Vogel over 11 years ago
Note: The DataTables plugin will be available in version 1.9.4 in the contrib directory of jquery if this patch will be merged into trunk: https://review.typo3.org/#/c/14683/
Updated by Helmut Hummel over 11 years ago
Christian Welzel wrote:
There are many other embedded code copies in typo3/sysext/extensionmanager/Resources/Public/Contrib/ and
typo3/sysext/extensionmanager/Resources/Public/JavaScript/.
Please move them to a proper location under typo3/contrib/ too.
This would mean, that we officially support this from now on and encourage extension developers to use these. This is not true.
We agreed on an exception for the extension manager, but besides that, we most likely will remove jQuery from typo3/contrib again.
Updated by Christian Welzel over 11 years ago
Then please move the datatable plugin out of the jquery.js so i can properly package TYPO3 for debian.
debian reqiures me to replace the embedded copy of jquery which is not possible when jquery and the plugin are merged.
Updated by Kai Vogel over 11 years ago
- Status changed from New to Accepted
- Assignee set to Kai Vogel
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15500
Updated by Kai Vogel over 11 years ago
- Status changed from Accepted to Under Review
Updated by Stefan Neufeind over 11 years ago
15500 was merged. This is just not reflected in forge unfortunately.
Updated by Christian Kuhn over 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Resolved: Patch is merged.
Updated by Oliver Hader about 11 years ago
- Project changed from 2269 to TYPO3 Core
Updated by