Project

General

Profile

Actions

Bug #29352

closed

t3lib_extMgm::addToAllTCAtypes broken

Added by Gone With the Wind about 13 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Backend API
Target version:
Start date:
2011-08-30
Due date:
% Done:

0%

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

Description

As confirmed by Jo Hasenau, t3lib_extMgm::addToAllTCAtypes cannot be used with an empty 3rd parameter:

t3lib_extMgm::addToAllTCAtypes("tt_content", 'subheader;;8', '', 'after:header');

As with Typo3 4.5 this causes the creation of a new tab "extended", whereas the "subheader" field should show up right underneath the header field.

At the moment only a construction like the following works:

t3lib_extMgm::addToAllTCAtypes("tt_content", 'subheader;;8', 'text,textpic', 'after:header');


Files

t3lib_div_extMgm.diff (3.55 KB) t3lib_div_extMgm.diff Jo Hasenau, 2011-08-30 19:48
Actions #1

Updated by Gone With the Wind about 13 years ago

  • Project changed from TYPO3 Core to 78
  • Category deleted (Backend User Interface)
  • Target version deleted (4.5.6)

Moved from TYPO3 v4 Core

Actions #2

Updated by Jo Hasenau about 13 years ago

  • Project changed from 78 to TYPO3 Core
Actions #3

Updated by Jo Hasenau about 13 years ago

  • Target version set to 4.5.6
Actions #4

Updated by Jo Hasenau about 13 years ago

As mentioned in the description I can confirm this behaviour. Currently I am investigating why this happens, and why nobody complained so far.
After all 4.5.x is out for quite a while now.

Actions #5

Updated by Jo Hasenau about 13 years ago

It seems that this fix http://forge.typo3.org/issues/25187 introduced at least one reason for the misbehaviour.

Although I have been able to fix it, still there are some other problems regarding strings containing more than the field name.

t3lib_extMgm::addToAllTCAtypes("tt_content", 'subheader', '', 'after:header');

works with the fixed version, while
t3lib_extMgm::addToAllTCAtypes("tt_content", 'subheader;;8', '', 'after:header');

does not.

Actions #6

Updated by Jo Hasenau about 13 years ago

Please test attached patch.

Should be working with both variants and even with more than one field at once.

Actions #7

Updated by Andreas Wolf about 13 years ago

  • Category set to Backend API
  • Complexity set to easy
Actions #9

Updated by Tolleiv Nietsch about 13 years ago

the patch looks good for me - any reason why this shouldn't be in Gerrit?

Actions #10

Updated by Chris topher about 13 years ago

  • Target version changed from 4.5.6 to 4.5.8
Actions #11

Updated by Mr. Jenkins almost 13 years ago

  • Status changed from New to Under Review

Patch set 1 of change I187c0ce388de7b878b2adf42d133a0758971f42c has been pushed to the review server.
It is available at http://review.typo3.org/7005

Actions #12

Updated by Gerrit Code Review almost 13 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7005

Actions #13

Updated by Ernesto Baschny almost 13 years ago

  • Target version changed from 4.5.8 to 4.5.12
Actions #14

Updated by Gone With the Wind over 12 years ago

Any progress here? The targeted version 4.5.12 obviousely hasn't been updated with this patch, neither has been 4.5.14.

Thanks for the info!

Actions #15

Updated by Gerrit Code Review over 12 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7005

Actions #16

Updated by Steffen Ritter over 11 years ago

  • Status changed from Under Review to Needs Feedback
  • Priority changed from Must have to Could have

No action for 1.5 years - the pushed patchset has been abandond

Actions #17

Updated by Alexander Opitz over 11 years ago

Hi,

as this issue is very old. Does the problem still exists within newer versions of TYPO3 CMS (4.5 or 6.1)?

Actions #18

Updated by Markus Klein almost 11 years ago

  • Status changed from Needs Feedback to Closed
  • Is Regression set to No

No feedback within the last 90 days => closing this ticket.

If you experience this issue again, please open a new ticket and add a relation to this ticket number.

Actions

Also available in: Atom PDF