Project

General

Profile

Actions

Bug #14652

closed

Problem when removing 'unsigned' attributes from db-fields

Added by Rupert Germann over 19 years ago. Updated over 18 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2005-04-07
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
3.8.0beta2
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

I discovered a strange behaviour in TYPO3 3.8.0dev (latest from cvs).
as I was working on class.t3lib_install.php I found the new option to remove
all "unsigned" attributes that kasper implemented yesterday.

changelog says:"... However if somebody wants to actually do the conversion
they can switch an internal variable in the t3lib_install class to TRUE and
the install tool WILL convert...

I set this to TRUE and converted the tables. After this the doktype of all
syfolders was set to [ INVALID VALUE (127) ]. Changing it back to sysfolder
results in an error message: '... Probably value mismatch with fieldtype.'

the field is defined in stddb/tables.sql as:
doktype tinyint(3) DEFAULT '0' NOT NULL,

the value 255 does not fit here.

After I changed it back (inserted the 'unsigned' attribute again), it was
possible to change the doktype of sysfolders back to "sysfolder" (255).

I'm pretty shure that this is not the only field where this change makes problems
(issue imported from #M955)

Actions #1

Updated by Ingmar Schlecht over 19 years ago

Thanks Rupi!
I just sent Kasper a pointer to this bug description.

Actions #2

Updated by Ingmar Schlecht over 19 years ago

Kasper will revert the whole UNSIGNED change he did, so everything will be like before again.

Thanks very much for testing this new feature and spotting the problem!

Actions #3

Updated by Peter Niederlag over 19 years ago

I think there is just one small leftover form reverting the changes:

class.t3lib_install.php, line 425

} elseif (strcmp($FDcomp[$table][$theKey][$fieldN], $fieldC) && strcmp($FDcomp[$table][$theKey][$fieldN], $fieldC)){

duplicate strcmp should be removed?

Actions #4

Updated by Michael Stucki over 19 years ago

Status:
- The unsigned thing has been reverted by Kasper.
- Peter is right with his statement - I will fix that one.

Actions #5

Updated by Rupert Germann over 19 years ago

the uplicate strcmp is fixed in cvs.

Actions

Also available in: Atom PDF