Bug #19496

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

Added by Oliver Hader almost 7 years ago. Updated about 5 years ago.

Status:Closed Start date:2008-10-22
Priority:Must have Due date:
Assigned To:Oliver Hader % Done:

0%

Category:- Spent time: -
Target version:-
TYPO3 Version:4.2 Is Regression:
PHP Version:5.2 Sprint Focus:
Complexity:

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.9 kB) Administrator Admin, 2008-10-27 15:00

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

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

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


Related issues

related to Core - Bug #18855: Wrong JavaScript inclusion in t3lib_TCEforms Closed 2008-05-26
related to Core - Bug #19049: drag & drop of section elements and IRRE records does not... Closed 2008-07-02

History

#1 Updated by Peter Niederlag almost 7 years ago

I'll do this tomorrow

#2 Updated by Oliver Hader almost 7 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 7 years ago

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

#4 Updated by Steffen Kamper almost 7 years ago

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

#5 Updated by Oliver Hader almost 7 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 7 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 7 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 7 years ago

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

Also available in: Atom PDF