Bug #19496

Flexform sections are not working anymore since script.aculo.us is not loaded

Added by Oliver Hader almost 11 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
-
Target version:
-
Start date:
2008-10-22
Due date:
% Done:

0%

TYPO3 Version:
4.2
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Due to a change of loading JavaScript resources between TYPO3 4.2.1 and 4.2.2 the script.aculo.us JavaScript framework now only gets loaded when it is required (before it was always loaded).

The solution is to explicitely load script.aculo.us when flexform sections are being rendered.

(issue imported from #M9623)

T3X_test_flexform_sections-0_0_0-z-200810271429.t3x (6.91 KB) Administrator Admin, 2008-10-27 15:00

0009623_v2.patch View (764 Bytes) Administrator Admin, 2008-10-27 15:07

patch-sections-8518.patch View (697 Bytes) Administrator Admin, 2008-10-28 11:46

0009623_v3.patch View (600 Bytes) Administrator Admin, 2008-10-28 13:58


Related issues

Related to TYPO3 Core - Bug #18855: Wrong JavaScript inclusion in t3lib_TCEforms Closed 2008-05-26
Related to TYPO3 Core - Bug #19049: drag & drop of section elements and IRRE records does not work if frame has to be scrolled Closed 2008-07-02

History

#1 Updated by Peter Niederlag almost 11 years ago

I'll do this tomorrow

#2 Updated by Oliver Hader almost 11 years ago

Please test the attached patch with the provided extension.
To do so, create a new content element of type "Insert plugin" and select "Test Flexform sections" as plugin name. Thanks to Steffen Kamper for writing this test!

#3 Updated by Oliver Hader almost 11 years ago

Version 2 of the patch does exactly the same but is IMO easier to read...

#4 Updated by Steffen Kamper almost 11 years ago

Thanks olly for the patch, now it's working perfect!

#5 Updated by Oliver Hader almost 11 years ago

Hi Peter, a patch for this issue is pending in the Core List. You can give your +1 there if it solves your problem as well.

#6 Updated by Peter Niederlag almost 11 years ago

Do 'containers' need scriptaculous as well? I have done a slightly diffrenet patch adding scriptaculous only to 'sections'.

#7 Updated by Peter Niederlag almost 11 years ago

0009623_v2.patch and patch-sections-8518.patch both work.

patch-sections-8518.patch will call loadJavascript only once though while 0009623_v2.patch will call it fo ALL child ellements. Do you think this is needed (for 'containers' maybe)?

#8 Updated by Oliver Hader almost 11 years ago

Committed to SVN:
  • TYPO3_4-2 (rev. 4358)
  • Trunk (rev. 4359)

Also available in: Atom PDF