Bug #56524
element browser is shown instead of vidi - TYPO3 6.2
100%
Description
i'm on typo3 6.2 beta5, vidi 0.3.0-dev and media 3.0.0-dev
the media extension is working as expected but
when i add an image to a page and click on "add image" the standard element browser opens up instead of the media (vidi) file browser.
did i miss something or isn't media (vidi) ready for typo3 6.2 ?
thx,
martin
Associated revisions
[BUGFIX] Element browser is shown instead of Media File Picker
Change-Id: I526dba39cb0d267313bbad5c3b7d215b3cfefcc9
Releases: 3.0.0
Resolves: #56524
Reviewed-on: http://review.typo3.org/31776
Reviewed-by: Fabien Udriot <fabien.udriot@ecodev.ch>
Tested-by: Fabien Udriot <fabien.udriot@ecodev.ch>
History
#1
Updated by Lorenz Ulrich over 5 years ago
Updated by - Status changed from New to Needs Feedback
Vidi now has an own item in the RTE which can be enabled in the Extension Configuration of "Media". If you have a custom RTE configuration that is loaded after Media, you need to add it manually to your configuration.
Media adds the following configuration:
RTE.default.showButtons := addToList(linkcreator,imageeditor)
// Toolbar order
// Must be completely reset
RTE.default.toolbarOrder = formatblock, blockstyle, textstyle, linebreak, bold, italic, underline, strikethrough, bar, textcolor, bgcolor, bar, orderedlist, unorderedlist, bar, left, center, right, justifyfull, copy, cut, paste, bar, undo, redo, bar, findreplace, removeformat, bar, link, unlink, linkcreator, bar, imageeditor, bar, table, bar, line, bar, insertparagraphbefore, insertparagraphafter, bar, chMode, showhelp, about, linebreak, tableproperties, rowproperties, rowinsertabove, rowinsertunder, rowdelete, rowsplit, columninsertbefore, columninsertafter, columndelete, columnsplit, cellproperties, cellinsertbefore, cellinsertafter, celldelete, cellsplit, cellmerge
RTE.default.RTEHeightOverride = 700
RTE.default.RTEWidthOverride = 700
(see ext_tables.php)
#2
Updated by Lorenz Ulrich over 5 years ago
- Target version deleted (
3.0.0)
#3
Updated by Philipp Wintermantel over 5 years ago
Updated by Lorenz, i think Martin is referring to (TCA-based) File Fields not RTE Images. I can reproduce this bug in a project. The culprit here is XCLASSing.
// Override classes for the Object Manager
$GLOBALS['TYPO3_CONF_VARS']['SYS']['Objects']['TYPO3\CMS\Backend\Form\FormEngine'] = array(
'className' => 'TYPO3\CMS\Media\Override\Backend\Form\FormEngine'
In my case i also have FLUX installed after media. FLUX also XCLASSES the FormEngine. So, there you go...
As a dirty hack to get stuff done i extended the classes like so "class Tx_Flux_Override_Backend_Form_FormEngine extends \TYPO3\CMS\Media\Override\Backend\Form\FormEngine". That seems to work for now. Not sure what can be done to resolve this properly.
#4
Updated by Loïc Stéphan over 5 years ago
Updated by I have similar problem on TYPO3 6.2.2 and media 3.0.0-dev.
flux is not installed, so only media XCLASS FormEngine.
What I do :
- Edit content and clic on "Add image" button --> Media browser is openned (user_VidiSysFileM1)
- Reload page and edit same content --> standard element browser is openned
- Edit another content --> standard element browser is openned
After a certain time (cache ?), Media browser works again.
Thanks
#5
Updated by Alexander Hense over 5 years ago
Updated by I have been experiencing the same behaviour as Loïc.
After some debugging I seem to have found the actual problem.
The FormEngine of media adds a requireJS module. This module is then loaded additionally to the original TYPO3 FormEngine module. Both modules only define jquery as their dependency. This leads to them being executed in a non deterministic order.
In the case that the original FormEngine is executed after the media FormEngine, the standard element browser is shown and vice versa.
Adding 'TYPO3/CMS/Backend/FormEngine' to the dependencies in Resources/Public/JavaScript/FormEngine.js works for me:
define('TYPO3/CMS/Media/FormEngine', ['jquery', 'TYPO3/CMS/Backend/FormEngine'], function ($) {
#6
Updated by Fabien Udriot over 5 years ago
Updated by Make sense to add this.
#7
Updated by Gerrit Code Review over 5 years ago
Updated by - Status changed from Needs Feedback to Under Review
Patch set 1 for branch master of project TYPO3CMS/Extensions/media has been pushed to the review server.
It is available at http://review.typo3.org/31776
#8
Updated by Fabien Udriot over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 75533e0dc7ba68796711d750b38f1102bfcb53d9.