Bug #45122

Addition extTables script might not exists

Added by Philipp Gampe over 8 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Miscellaneous
Target version:
Start date:
2013-02-03
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
5.4
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

E.g. on Travis.


Related issues

Related to TYPO3 Core - Bug #44331: Impossible to save the TCA configurationClosedPhilipp Gampe2013-01-05

Actions
Related to TYPO3 Core - Bug #46161: Empty extTables script wrongly detectedClosedFrancois Suter2013-03-10

Actions
Related to TYPO3 Core - Bug #47002: Warning in getCompressedTCarray due to missing extTables.phpClosedPhilipp Gampe2013-04-06

Actions
#1

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17994

#2

Updated by Gerrit Code Review over 8 years ago

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

#3

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/18001

#4

Updated by Philipp Gampe over 8 years ago

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

Updated by Francois Suter over 8 years ago

The added check is not enough. file_exists will respond positively on the "typo3conf" directory which happens if TYPO3_extTableDef_script is set to an empty string (this might happen when upgrading from an older version which "disabled" that feature by setting it to an empty string). The test should also perform "is_file".

How do we handle follow-ups? Should I open a separate issue?

#6

Updated by Philipp Gampe over 8 years ago

Francois Suter wrote:

How do we handle follow-ups? Should I open a separate issue?

That would be the easiest solution.

#7

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF