RE: Error in commit
It seems to me that the adherence to the coding guidelines has become very lax recently. When I was admin I tried to be rigorous about this.
For instance it has always been the guideline the SQL keywords should be upper case in order to make the SQL more readable. In a commit last night we have two sql statements one after the other that read:
alter table stockcatproperties drop foreign key stockcatproperties_ibfk_3;
ALTER TABLE `emailsettings` CHANGE `username` `username` VARCHAR( 50 ) CHARACTER SET utf8 COLLATE utf8_general_ci NULL DEFAULT NULL;
as you can see the first uses lower case for SQL keywords, and the second uses upper case.
This is something I have always felt strongly about, and it would be a shame if now others are running the project these standards are being dropped. There have been many such examples recently.
Tim
|