Inflection of ID causes issues in Rails apps

qbwc / qbxml

QBXML Parser and Validation Tool

MIT License

27 stars 41 forks source link

Inflection of ID causes issues in Rails apps #2

Closed bcardarella closed 9 years ago

bcardarella commented 11 years ago

You should avoid inflecting common words like 'id' because String#camelize rely upon the inflection. We have several migrations that have 'Id' in the name of the migration. This inflection breaks those migrations now.

skalee commented 9 years ago

This is still broken on master.

Given example, I have Rails migration in db/migrate/TIMESTAMP_add_user_id_to_invoices.rb. After adding qbxml to Gemfile, this migration fails. In order to get it working, I need to rename migration class defined in this file from AddUserIdToInvoices to AddUserIDToInvoices (ID in upper case).

If, for any reason, this is not going to be fixed soon, I suggest adding post install message to gemspec.