Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

renamed "grocery_CRUD_Model" to "Grocery_crud_model" #336

Merged
merged 1 commit into from
Nov 4, 2015

Conversation

P8H
Copy link
Contributor

@P8H P8H commented Nov 3, 2015

renamed the model for fixing #335.
Used name convention from: http://www.codeigniter.com/user_guide/general/models.html
"Where Model_name is the name of your class. Class names must have the first letter capitalized with the rest of the name lowercase."

Tested with CodeIgniter-3.0.3

scoumbourdis added a commit that referenced this pull request Nov 4, 2015
renamed "grocery_CRUD_Model" to "Grocery_crud_model"
@scoumbourdis scoumbourdis merged commit 4c290da into scoumbourdis:master Nov 4, 2015
@scoumbourdis
Copy link
Owner

Thanks for the pull request. I didn't have this issue before but your fix seems correct eitherway :)

scoumbourdis added a commit that referenced this pull request Nov 4, 2015
Fixing bug: #335 - Unable to locate the model you have specified: grocery_crud_model (pull request: #336 from @P8H)
luiscoms pushed a commit to luiscoms/grocery-crud that referenced this pull request Jan 14, 2016
Fixing bug: scoumbourdis#335 - Unable to locate the model you have specified: grocery_crud_model (pull request: scoumbourdis#336 from @P8H)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants