Bug #17961

Missing TCA[table][columns] leads to PHP warning

Added by Franz Holzinger over 11 years ago. Updated 8 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend User Interface
Target version:
-
Start date:
2007-12-31
Due date:
% Done:

100%

TYPO3 Version:
4.2
PHP Version:
5.1
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

If a table does not have the TCA set, then the list module shows an error message.

Invalid argument supplied for foreach() in /var/www/html/typo3-svn/TYPO3core/trunk/typo3/class.db_list.inc on line 690

A TYPO3 warning should be displayed instead of this. This shows the name of the table and helps to save much time to find out the reason for this warning message.

(issue imported from #M7070)

notcacolumnserror.diff View (2.17 KB) Administrator Admin, 2007-12-31 14:09

7070-label.diff View (1.98 KB) Administrator Admin, 2007-12-31 15:24


Related issues

Related to TYPO3 Core - Bug #22152: 4.4: PHP warning "Invalid argument supplied for foreach()" in list module Closed 2010-02-22

Associated revisions

Revision cc2e767d (diff)
Added by Stefan Neufeind over 7 years ago

[FEATURE] Add error message for missing TCA

Adds an error message in case the TCA of a table is missing
column definitions. The error message adds a hint that helps
the developer to find the affected table.
In addition it changes the table header to show the table name
as it is in the database if there is no title defined in TCA.

Change-Id: I568c63d40de11385da0bf848904b860c4269fac4
Resolves: #17961
Releases: 4.7
Reviewed-on: http://review.typo3.org/6883
Reviewed-by: Stefan Neufeind
Reviewed-by: Oliver Klee
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1 Updated by Franz Holzinger over 11 years ago

There is also a problem with list views, if not title and label are set in TCA. Please implement this patch. It helps to save a lot of time with extension developing.

#2 Updated by Andreas Wolf over 7 years ago

  • Category set to Backend User Interface
  • Assignee set to Andreas Wolf
  • Target version changed from 0 to 4.6.0
  • Complexity set to easy

#3 Updated by Chris topher over 7 years ago

  • Target version changed from 4.6.0 to 4.6.1

#4 Updated by Mr. Jenkins over 7 years ago

  • Status changed from New to Under Review

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

#5 Updated by Mr. Jenkins over 7 years ago

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

#6 Updated by Mr. Jenkins over 7 years ago

Patch set 2 of change I568c63d40de11385da0bf848904b860c4269fac4 has been pushed to the review server.
It is available at http://review.typo3.org/6883

#7 Updated by Chris topher over 7 years ago

  • Target version changed from 4.6.1 to 4.6.2

#8 Updated by Mr. Jenkins over 7 years ago

Patch set 3 of change I568c63d40de11385da0bf848904b860c4269fac4 has been pushed to the review server.
It is available at http://review.typo3.org/6883

#9 Updated by Gerrit Code Review over 7 years ago

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

#10 Updated by Gerrit Code Review over 7 years ago

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

#11 Updated by Gerrit Code Review over 7 years ago

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

#12 Updated by Gerrit Code Review over 7 years ago

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

#13 Updated by Stefan Neufeind over 7 years ago

Issues 17961 and 22152 touch (and fix) the same thing - doing it slightly different, but the same thing. Please merge :-)

#14 Updated by Xavier Perseguers over 7 years ago

  • Target version deleted (4.6.2)

#15 Updated by Stefan Neufeind over 7 years ago

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

#16 Updated by Benni Mack 8 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF