Not sure why i can't find anyone else posting about this - but the table names should be lower case.

Reason #1 would be since this is sort of std for Drupal (although not likely written down anywhere).

but more importantly:

Reason #2 - Windows (by default) converts table names on import to lowercase. And, it also converts table names in sql queries to lower case as well - so the double conversion works out fine - BUT, if someone develops on Windows and then exports to a linux server then the table names will now be lower case but the sql queries will still be mixed case.

See this article for more info: http://dev.mysql.com/doc/refman/5.0/en/identifier-case-sensitivity.html

but simplest is just use lowercase.

Peter Lindstrom
LiquidCMS - Content Management Solution Experts

Comments

daniel.hunt’s picture

daniel.hunt commented
Assigned: Unassigned » daniel.hunt
Status: Active » Postponed (maintainer needs more info)

All of the Unreal.IE servers run Linux, and I develop on both Windows and Linux. I run both a WAMP and LAMP test environment, and I've never run into this issue.

You should look into using the lower_case_table_names system variable for MySQL Server if you have the option, but I will bow to the standards in this case and also change the code to use lower-case table names from now on.

This may well cause issues for others, so I will also include a db update script to force all existing installs to rename their UIEForum tables to lowercase.

Again, this hasn't been an issue for me, or for anyone else, since day 1, so I'm interested to find out why it is for you.

Daniel

liquidcms’s picture

liquidcms commented

Not so much an issue as a suggestion. By default MySQL on windows isn't set up to adhere to table name case. So if exported from windows by default it will create lower case names which when imported into linux system will no longer work.

Yes, i now have the mysql system var set - but many wouldn't. Why know one has noticed this is beyond me.

Also, it doesn't follow drupal standards.

daniel.hunt’s picture

daniel.hunt commented
Status: Postponed (maintainer needs more info) » Fixed

Well I've sorted the bugs out on Unreal.IE already, and I've committed an updated version of the module to CVS - so whats there now should force everything to lowercase.

Thanks :)
Daniel

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.