Forums

DB errors updating add ons for 5.7

Permalink
Hi, I've started to try to update some add ons to work with 5.7, on a couple I am getting an error when installing

Unknown column type "" requested. Any Doctrine type that you use has to be registered with \Doctrine\DBAL\Types\Type::addType(). You can get a list of all the known types with \Doctrine\DBAL\Types\Type::getTypesMap(). If this error occurs during database introspection then you might have forgot to register all database types for a Doctrine Type. Use AbstractPlatform#registerDoctrineTypeMapping() or have your custom types implement Type#getMappedDatabaseTypes(). If the type name is empty you might have a problem with the cache or forgot some mapping information.

I don't see anything in the db.xml file that looks wrong, they work with other versions of C5. Anyone know what's going on with this?

thanks

Thanks
pvernaglia
 
andrew replied on at Permalink Reply
andrew
It's gotta be an error in either the package's db.xml or your package's blocks' db.xml files. We have a compatibility layer that maps the old style XML types (the "type" attribute for each column) for the new types for the new database abstraction layer. But I only did mappings for ones that occurred in the core db.xml file (like type="X", "X2", etc...). It's possible that there's one that ADODB supported that we didn't use in the core that we don't have any handling for.

I would look through those db.xml files and see if any type="" declarations match a value that isn't handled here:

https://github.com/concrete5/concrete5-5.7.0/blob/master/web/concret...

Looking at that code it probably will break on any lowercase options too, which I would guess is probably the problem. We should probably make that code use strcasecmp instead.
pvernaglia replied on at Permalink Reply
pvernaglia
I had some lower case and was using 'float' not 'F' - thanks
pvernaglia replied on at Permalink Reply
pvernaglia
Hey Andrew, where did that file move to? Thanks
pilipala replied on at Permalink Reply
pilipala
Hi

I had the same error. Does anyone know where the link has moved to?

Thanks,

Jennifer
pvernaglia replied on at Permalink Reply
pvernaglia
Make sure you have all capital letters for the field types. that fixed it for me
andrew replied on at Permalink Reply
andrew
https://github.com/concrete5/concrete5-5.7.0/blob/master/web/concrete/src/Database/Schema/Parser/Axmls.php#L146