Project

General

Profile

Actions
Copy link

Bug #70630

closed

Story #69617: FormEngine bugs

Can't select parent Category for Categories without parent

Added by Sascha Löffler over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Categorization API
Target version:
7 LTS
Start date:
2015-10-12
Due date:
% Done:

100%

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

Description

We found another problem with Category selection in Typo3 7.5.

the Parent can't be selected anymore. Pretty sure it worked in 7.4 as most of them where created there.

See attached Gif-Animation of the issue.
The shown category has no parent yet selected. If i look into the HTML all entries in the category-tree have the attribute unselectable="on" which is probably used by some javascript.

Files

typo3_7.5_bug.gif (762 KB) typo3_7.5_bug.gif Sascha Löffler, 2015-10-12 17:41

Related issues 1 (0 open — 1 closed)

Has duplicate TYPO3 Core - Bug #70306: Sys Category Selection does not work when adding new Sys CategoryClosed2015-10-02

Actions
Actions
Copy link
 #1

Updated by Witali Rott over 8 years ago

I can comfirm this

Actions
Copy link
 #2

Updated by Witali Rott over 8 years ago

  • % Done changed from 0 to 100

There was an error with the parsing of the elements, the default value zero was count as an selected node.

https://github.com/hlop/TYPO3.CMS/commit/816f3c5609d3ce815301066bf7319aa25957e2d0

Actions
Copy link
 #3

Updated by Sascha Löffler over 8 years ago

Thanks. Can confirm that this fixes it.

Actions
Copy link
 #4

Updated by Christian Fries over 8 years ago

Thanks Witali, your patch fixes the issue.

Actions
Copy link
 #5

Updated by Henri Nathanson over 8 years ago

https://review.typo3.org/#/c/43531/ that probably caused it.

Actions
Copy link
 #6

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

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

Actions
Copy link
 #7

Updated by Andreas Allacher over 8 years ago

Please also check
https://review.typo3.org/#/c/43807/

And
https://forge.typo3.org/issues/70372

And yes it was caused by the review above but that was still a correct bugfix. Just added another bug which still should be handled differently I think.
I am still waiting for feedback from Christian on how to be solve this because checking renderType is not enough if one considers custom renderTypes

Actions
Copy link
 #8

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #9

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #10

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #11

Updated by Gerrit Code Review over 8 years ago

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

Actions
Copy link
 #12

Updated by Christian Kuhn over 8 years ago

  • Parent task set to #69617
Actions
Copy link
 #13

Updated by Morton Jonuschat over 8 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #14

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF