Bug #43893: selected = 1 doesn't work in class.tslib_content_form.php - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #43893

closed

selected = 1 doesn't work in class.tslib_content_form.php

Added by Thorsten Menkenhagen almost 12 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Target version:

4.5.23

Start date:

2012-12-11

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.5

PHP Version:

Tags:

Complexity:

easy

Is Regression:

Sprint Focus:

Description

When I change

$selected = isset($singleKeyArray['selected.'])
? $this->cObj->stdWrap($singleKeyArray['selected'], $singleKeyArray['selected.'])
: $singleKeyArray['selected'];

in Line 107 to

$selected = isset($singleKey_valueArray['selected.'])
? $this->cObj->stdWrap($singleKey_valueArray['selected'], $singleKey_valueArray['selected.'])
: $singleKey_valueArray['selected'];

then everything is okay.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Peter Schuster almost 12 years ago

I can confirm this. The same problem exists in TYPO3 v4.7.
The suggested fix works for me, too.

Actions

Copy link

Updated by Tobias Liebig almost 12 years ago

Could you provide a more detailed description how to test/reproduce this?

I guess it also occurs in 6.0, even if the code looks completely different there

Actions

Copy link

Updated by Tobias Liebig almost 12 years ago

Test TypoScript:

page = PAGE

temp.mailform = FORM
temp.mailform {
  dataArray {
    20.label = Foobar
    20.type = test=select
    20.valueArray {
      10.label = Blue
      10.value = 1
      20.label = Red
      20.value = 2
      20.selected = 1
    }
  }
  recipient = info@example.com
  layout = <div class="some-class">###LABEL### ###FIELD###</div>
}

page.10 < temp.mailform

Actions

Copy link

Updated by Tobias Liebig almost 12 years ago

Complexity set to easy

I can confirm this still happens in 6.0, although the TYPO3\CMS\Frontend\ContentObject\FormContentObject seems not to be used at all if the new EXT:form is installed.

However, the suggested fix makes sense and is pretty easy to implement.

Actions

Copy link