DDC-180: [patch] The SmallIntType does not extend the default Type class #222

Closed
opened 2026-01-22 12:31:22 +01:00 by admin · 5 comments
Owner

Originally created by @doctrinebot on GitHub (Nov 27, 2009).

Jira issue originally created by user marijn:

The schematool has a check to see if a field is a string value, this check throws a small error because it passes an object into strtolower instead of a string value. The attached patch calls the method Type->getName() to check if the field is a string.

Originally created by @doctrinebot on GitHub (Nov 27, 2009). Jira issue originally created by user marijn: The schematool has a check to see if a field is a `string` value, this check throws a small error because it passes an object into `strtolower` instead of a `string` value. The attached patch calls the method `Type->getName()` to check if the field is a string.
admin added the Bug label 2026-01-22 12:31:22 +01:00
admin closed this issue 2026-01-22 12:31:22 +01:00
Author
Owner

@doctrinebot commented on GitHub (Nov 27, 2009):

Comment created by @beberlei:

this shouldnt throw an error, all types have a **toString() method.

@doctrinebot commented on GitHub (Nov 27, 2009): Comment created by @beberlei: this shouldnt throw an error, all types have a **toString() method.
Author
Owner

@doctrinebot commented on GitHub (Nov 27, 2009):

Comment created by marijn:

You are right, the issue originated from the SmallIntType class which does not extend the Type class. See the attached patch file.

@doctrinebot commented on GitHub (Nov 27, 2009): Comment created by marijn: You are right, the issue originated from the `SmallIntType` class which does not extend the `Type` class. See the attached patch file.
Author
Owner

@doctrinebot commented on GitHub (Dec 3, 2009):

Comment created by @guilhermeblanco:

Issue fixed!

Thanks for the patch!

@doctrinebot commented on GitHub (Dec 3, 2009): Comment created by @guilhermeblanco: Issue fixed! Thanks for the patch!
Author
Owner

@doctrinebot commented on GitHub (Dec 3, 2009):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Dec 3, 2009): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Dec 13, 2015):

Imported 1 attachments from Jira into https://gist.github.com/71f33ff9d5bf6966067a

@doctrinebot commented on GitHub (Dec 13, 2015): Imported 1 attachments from Jira into https://gist.github.com/71f33ff9d5bf6966067a - [10191_strtolower_bug_doctrine.diff](https://gist.github.com/71f33ff9d5bf6966067a#file-10191_strtolower_bug_doctrine-diff)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#222