Bug #40498
embedded jquery
| Status: | Resolved | Start date: | 2012-09-01 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Kai Vogel | % Done: | 100% |
|
| Category: | Extension Manager | Spent time: | - | |
| Target version: | - | |||
| TYPO3 Version: | 6.0 | Is Regression: | ||
| PHP Version: | Sprint Focus: | |||
| Complexity: |
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
History
#1 Updated by Christian Welzel almost 3 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 almost 3 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 almost 3 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 almost 3 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 almost 3 years ago
- Status changed from New to Accepted
- Assigned To set to Kai Vogel
#6 Updated by Gerrit Code Review almost 3 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 almost 3 years ago
- Status changed from Accepted to Under Review
#8 Updated by Stefan Neufeind almost 3 years ago
15500 was merged. This is just not reflected in forge unfortunately.
#9 Updated by Christian Kuhn almost 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Resolved: Patch is merged.
#10 Updated by Oliver Hader over 2 years ago
- Target version set to 2222
#11 Updated by Oliver Hader over 2 years ago
- Project changed from Extension Manager to Core
#12 Updated by Oliver Hader over 2 years ago
- Category set to Extension Manager
#13 Updated by Oliver Hader over 2 years ago
- Target version deleted (
2222)