Feature #25382

TCA Tree should respect $TCA['myTable']['ctrl']['requestUpdate'] and $TCA['myTable']['type']

Added by Marco Huber over 8 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2011-03-24
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The onChange-Javascript is missing. I will push a patch to gerrit as soon as possible.

(issue imported from #M18022)

issue-25382_ext-tcatree-z.t3x - Test extension (10.9 KB) Thorsten Kahler, 2011-07-06 19:44

Associated revisions

Revision eea2e586 (diff)
Added by Marco Huber over 8 years ago

[BUGFIX] TCA Tree should respect requestUpdate and type

TCA Tree should respect $TCA['myTable']['ctrl']['requestUpdate']
and $TCA['myTable']['type'], but the onChange Javascript is missing.

Change-Id: I46b9513413cbb61660609bbf73eb3551c5ce7a24
Resolves: #25382
Reviewed-on: http://review.typo3.org/1251
Reviewed-by: Stefan Neufeind
Reviewed-by: Fabien Udriot
Reviewed-by: Steffen Ritter
Tested-by: Thorsten Kahler
Tested-by: Steffen Ritter

Revision 8d4cb92a (diff)
Added by Marco Huber about 8 years ago

[BUGFIX] TCA Tree should respect requestUpdate and type

TCA Tree should respect $TCA['myTable']['ctrl']['requestUpdate']
and $TCA['myTable']['type'], but the onChange Javascript is missing.

Change-Id: If2c7997c9627249e3f643c9670a2a9d4ba94103a
Resolves: #25382
Reviewed-on: http://review.typo3.org/3402
Reviewed-by: Lars Trebing
Tested-by: Lars Trebing
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

History

#2 Updated by Fabien Udriot over 8 years ago

  • Target version deleted (0)

I am still a bit unsure how to test the patch properly and what to test.

Default TCA contains:

$TCA['tt_content']['ctrl'][requestUpdate]=list_type,rte_enabled

and whenever I click on the checkbox "Rich Text Editor" the Interface is correctly re-loaded in TYPO3 4.5.6alpha1... Maybe more info would be welcome about how to reproduce the bug.

#3 Updated by Marco Huber over 8 years ago

Hi Fabien,

Fabien Udriot wrote:

I am still a bit unsure how to test the patch properly and what to test.

The field must be of type "select" and renderMode "tree" ("TCA tree" ;-)). If such a field is listed as requestUpdate-field the needed Javascript is missing.

#4 Updated by Mr. Hudson over 8 years ago

Patch set 6 of change I46b9513413cbb61660609bbf73eb3551c5ce7a24 has been pushed to the review server.
It is available at http://review.typo3.org/1251

#5 Updated by Steffen Ritter over 8 years ago

  • Category set to Backend API
  • Status changed from New to Accepted

#6 Updated by Thorsten Kahler over 8 years ago

I attached a simple test extension, that add a table with two field (title and TCA tree field).

#7 Updated by Marco Huber over 8 years ago

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

#8 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF