Bug #41344

Bug parsing ext_tables.sql

Added by Kay Strobach over 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extension Manager
Target version:
-
Start date:
2013-06-12
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Hello Guys,

in theory these two statements should be seen as equal (in sql)

CREATE TABLE tx_coreupdate_cache (
    id INT(11) INSIGNED NOT NULL AUTO_INCREMENT
)

based on example from: http://wiki.typo3.org/Caching_framework#Cache_registration_and_usage_for_extensions_supporting_TYPO3_4.3_and_above

This code causes constant warnings of not uptodate database for this extension, if i change the case like below everything works fine.
This is reproduceable in 4.5 to 4.7 :D

CREATE TABLE tx_coreupdate_cache (
    id int(11) unsigned NOT NULL auto_increment,
)

Solution should be to either convert all reserved words to lower / to uppercase and then do the comparision, or change the examples in the wiki :D

Regards
Kay


Subtasks

Bug #49072: Fix missing single quotes in SchemaMigratorTestClosed2013-06-12

Actions

Related issues

Related to CoreUpdate - Bug #41341: Database always not uptodateResolvedKay Strobach2012-09-25

Actions
#1

Updated by Gerrit Code Review about 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/19630

#2

Updated by Gerrit Code Review about 8 years ago

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

#3

Updated by Gerrit Code Review about 8 years ago

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

#4

Updated by Gerrit Code Review about 8 years ago

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

#5

Updated by Gerrit Code Review about 8 years ago

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

#6

Updated by Gerrit Code Review about 8 years ago

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

#7

Updated by Gerrit Code Review about 8 years ago

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

#8

Updated by Gerrit Code Review about 8 years ago

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

#9

Updated by Gerrit Code Review about 8 years ago

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

#10

Updated by Gerrit Code Review about 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/21353

#11

Updated by Anonymous about 8 years ago

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

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/21354

#13

Updated by Gerrit Code Review about 8 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/21355

#14

Updated by Gerrit Code Review about 8 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/21354

#15

Updated by Gerrit Code Review about 8 years ago

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

#16

Updated by Gerrit Code Review about 8 years ago

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

#17

Updated by Anonymous almost 8 years ago

  • Status changed from Under Review to Resolved
#18

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF