Bug #69836

Selected node count detection in select tree is invalid

Added by Alexander Stehlik over 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
FormEngine aka TCEforms
Target version:
-
Start date:
2015-09-15
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

The selected node count will always be at least one, even if no value is selected.

Associated revisions

Revision 2f4303a3 (diff)
Added by Alexander Stehlik over 4 years ago

[BUGFIX] Fix selected node count in select tree

If only one item can be selected in a select tree field (maxitems=1)
and the first selected value is empty is will be cleared from the
selected node array.

Resolves: #69836
Releases: master
Change-Id: Ic3b71a39f73638a7d8c46c7939d17260356437dd
Reviewed-on: http://review.typo3.org/43342
Reviewed-by: Oliver Thiele <>
Tested-by: Oliver Thiele <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision bbed735b (diff)
Added by Andreas Allacher over 4 years ago

[BUGFIX] need to parse field value for maxitems = 1 with MM table

If a select field has an MM table like sys_category_record_mm
it is necessary to parse the field value even if maxitems = 1
Furthermore, this really fixes #69836 as that was actually
only a workaround.

Resolves: #70081
Relates: #69836
Releases: master

Change-Id: I13c286048d666fa196d5dc83a72a3612090c4e25
Reviewed-on: http://review.typo3.org/43531
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Morton Jonuschat <>
Tested-by: Morton Jonuschat <>

Revision dc506d5a (diff)
Added by Morton Jonuschat over 4 years ago

[BUGFIX] FormEngine: Pass correct selected node count to ExtJS Tree

The selected node count for the ExtJS Tree wasn't calculated using the
raw database field value instead of the parsed tree configuration. In
case of empty database values this resulted in a wrong count of selected
nodes, disabling the selection of nodes in the case of maxitems=1 in the
tree configuration.

Resolves: #70306
Resolves: #70630
Related: #69836
Releases: master
Change-Id: Ib04207c60b7a3473eaac619b6f2f4091aa5b938f
Reviewed-on: https://review.typo3.org/44356
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from New 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/43342

#2 Updated by Alexander Stehlik over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#3 Updated by Anja Leichsenring over 4 years ago

  • Sprint Focus deleted (On Location Sprint)

#4 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF