Bug #69195: EXT:form - Error if old mailform option is called - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #69195

closed

EXT:form - Error if old mailform option is called

Added by Konstantin Salakhutdinov over 8 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Form Framework

Target version:

7.5

Start date:

2015-08-19

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

5.5

Tags:

Complexity:

Is Regression:

Sprint Focus:

Stabilization Sprint

Description

When using the old mailform option and having useDefaultContentObject set in typoscript breaks the website if ext:compatibility6 is not installed.

if ($typoScriptObjectName === 'FORM' && !empty($typoScript['useDefaultContentObject'])) {
$content = $contentObject->getContentObject($typoScriptObjectName)->render($typoScript); <----- breaks here
} elseif ($typoScriptObjectName === 'FORM') {

Actions

Copy link

Updated by Wouter Wolters over 8 years ago

Status changed from New to Needs Feedback

I'm sorry but with this description we can't find out what is going on.
Please give us more information about the form itself etc.

A white screen means a PHP error that should be logged.
Can you at least try to enable the development preset in the Install Tool and check the error logs again.

Actions

Copy link

Updated by Konstantin Salakhutdinov over 8 years ago

Sorry, initiated developer mode in install tool and see hundreds of Notices lines and a last line that is like this:

Fatal error: Call to a member function render() on a non-object in /home/k/konstantin/typo3_src-7.4.0/typo3/sysext/form/Classes/Controller/FormController.php on line 92

Actions

Copy link

Updated by Wouter Wolters over 8 years ago

Subject changed from Mail Form in Typo3 7.4 leads to white screen to Old mailform option called within ext:form
Description updated (diff)
Status changed from Needs Feedback to Accepted

You use the old school mailform option from TYPO3. This option has been moved to ext:compatibility6 and will be moved out of the core. This extension needs to be installed to make your website work again.

I will get in contact with a developer if we can make this work without a fatal error, but an nicer error message.

Actions

Copy link

Updated by Wouter Wolters over 8 years ago

Description updated (diff)

Actions

Copy link

Updated by Konstantin Salakhutdinov over 8 years ago

Ok, but if sysext form is not actual anymore, then why is it still included in Typo3 7.4 ? I think it is more handy than powermail for simple forms, because of easy to use interface.

Actions

Copy link

Updated by Wouter Wolters over 8 years ago

I think you got me wrong there. ext:form is just working, but your form still uses the earlier form solution of the core. Which is moved to ext:compatibility6.

Check if your TypoScript contains the option "useDefaultContentObject". This calls the old form solution which is located in the compat extension now.

Actions

Copy link

Updated by Konstantin Salakhutdinov over 8 years ago

Wouter Wolters wrote:

Check if your TypoScript contains the option "useDefaultContentObject". This calls the old form solution which is located in the compat extension now.

Wow. You are definitely right "useDefaultContentObject" is set to 1. I found it in Typo Script Object Browser.
But it is strange. To understand what's the problem I've installed fresh Typo3 7.4 distributive and used only that what it offers. Only extensions that were installed are:
- Builder
- Fluidconent
- Fluidpages
- vhs
- flux

So I can't imaging where did old mailform come from...

Actions

Copy link

Updated by Wouter Wolters over 8 years ago

Do you include css_styled_content version 6.2 typoscript?

Actions

Copy link

Updated by Konstantin Salakhutdinov over 8 years ago

Wouter Wolters wrote:

Do you include css_styled_content version 6.2 typoscript?

Yes. But i tried including usual csc and also fluidcontent_core - and still the problem seems to be in using the old type of mailform CE.

Actions

Copy link

#10

Updated by Gerrit Code Review over 8 years ago

Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43472

Actions

Copy link

#11

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43472

Actions

Copy link

#12

Updated by Ralf Zimmermann over 8 years ago

Subject changed from Old mailform option called within ext:form to EXT:form - Error if old mailform option is called

Actions

Copy link

#13

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43472

Actions

Copy link

#14