Bug #40498

embedded jquery

Added by Christian Welzel over 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Extension Manager
Target version:
-
Start date:
2012-09-01
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

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.


Related issues

Related to TYPO3 Core - Task #42197: Use central jQueryClosedStefan Neufeind2012-10-20

Actions
#1

Updated by Christian Welzel over 8 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.

#2

Updated by Kai Vogel over 8 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/

#3

Updated by Helmut Hummel over 8 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.

#4

Updated by Christian Welzel over 8 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.

#5

Updated by Kai Vogel over 8 years ago

  • Status changed from New to Accepted
  • Assignee set to Kai Vogel
#6

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15500

#7

Updated by Kai Vogel over 8 years ago

  • Status changed from Accepted to Under Review
#8

Updated by Stefan Neufeind over 8 years ago

15500 was merged. This is just not reflected in forge unfortunately.

#9

Updated by Christian Kuhn over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Resolved: Patch is merged.

#10

Updated by Oliver Hader almost 8 years ago

  • Target version set to 2222
#11

Updated by Oliver Hader almost 8 years ago

  • Project changed from Extension Manager to TYPO3 Core
#12

Updated by Oliver Hader almost 8 years ago

  • Category set to Extension Manager
#13

Updated by Oliver Hader almost 8 years ago

  • Target version deleted (2222)
#14

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF