ahmader / node-zoho

Zoho API access for NodeJS
21 stars 15 forks source link

index.coffee assumes all module files are lowercase; make it so #20

Closed phillipleslie closed 7 years ago

phillipleslie commented 7 years ago

Over in https://github.com/ahmader/node-zoho/blob/master/lib/products/crm/index.coffee#L11 there is an assumption that all modules in /lib/products/crm/ will have lowercase filenames. However, /lib/products/crm/Accounts.coffee breaks this assumption.

Perhaps on certain OSes (Windows?), it might succeed to require.resolve with a filename string whose capitalization differs from the underlying file system's filename capitalization. However, on Amazon Linux (and probably many other linux distros) the capitalization does matter.

This pull request fixes it, with no known downsides for anyone else.