Bug #34544: sys_action > create backend user > assigning a db mount leads to javascript error "TBE_EDITOR not defined" - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #34544

closed

sys_action > create backend user > assigning a db mount leads to javascript error "TBE_EDITOR not defined"

Added by Gwenhadu about 12 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Target version:

Start date:

2012-03-05

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.6

PHP Version:

5.3

Tags:

Complexity:

easy

Is Regression:

Sprint Focus:

Description

Unable to assign a db mount while executing the task.

Javascript is built through class.t3lib_tceforms.php (function dbFileCon - line 5586).

Hypothesis :
The code on line 5616 beginning by :
if (typeof TBE_EDITOR.clearBeforeSettingFormValueFromBrowseWin[fName] != "undefined") ...

should be included under the condition on line 5631:
if (!isMultiple) {...

Because ... let's have a look to the code beginning on line 2076, same class.

Files

34544.diff (1.68 KB) 34544.diff

Gwenhadu , 2012-03-08 09:37

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Christian Jul Jensen about 12 years ago

Could you maybe try out if your suggested solutions solves the problem and provide a patch for testing?

Actions

Copy link

Updated by Gwenhadu about 12 years ago

File 34544.diff 34544.diff added

Here is a patch for testing.

Actions

Copy link

Updated by Henrik Ziegenhain about 12 years ago

Applying the attached patch worked for me (v4.6.4)
Thank you, +1

Actions

Copy link

Updated by Christian Jul Jensen about 12 years ago

Complexity set to easy

Thanks for providing the patch and testing!

Would be great if one you submit for review:
Check out the guides here http://typo3.org/development/bug-fixing/resolving-bugs/

Actions

Copy link

Updated by Ralf Hettinger about 11 years ago

The issue is still there in TYPO3 6.0.4 where the buggy code has moved to typo3/sysext/backend/Classes/Form/FormEngine.php
+1 for the solution based on testing only, since it fixes the problem :)

Actions

Copy link

Updated by Gerrit Code Review about 11 years ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review almost 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20206

Actions

Copy link

Updated by Philipp Holdener over 10 years ago

Will that bug be fixed in next versions?
Need that function in a 4.7.x version.

Actions

Copy link

Updated by Christian Buelter over 10 years ago

Patch works in 6.1.5

Actions

Copy link

#10

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25500

Actions

Copy link

#11

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25501

Actions

Copy link

#12