search

CMPUT301W15T12 / C301Project

Apache License 2.0
3 stars 2 forks source link

**IMPORTATNT, MUST READ** Using controllers from front-end code #56

Closed ViIIager closed 9 years ago

ViIIager commented 9 years ago

Hi guys. it seems to me some of us are misunderstanding the need for controllers.

I have noticed that some of you in the front-end code writing have code similar to this:

UserlistController.getUserList().addUser(); << THIS IS ONLY FOR ME TO DO. Direct access to UserList. defeating the purpose of controllers

what you should be doing is this:

UserListController.addUser(). << this uses the controller to interface with the UserList Model and returns you the data you need. please fix this as soon as possible.

please make sure this does not happen again.

vanbelle commented 9 years ago

Sure, I only left it like that so we could have something to run in the meantime.

On Mar 14, 2015, at 12:57 PM, Omar Zioueche notifications@github.com wrote:

Hi guys. it seems to me some of us are misunderstanding the need for controllers.

I have noticed that some of you in the front-end code writing have code similar to this:

UserlistController.getUserList().addUser(); << THIS IS ONLY FOR ME TO DO. Direct access to UserList. defeating the purpose of controllers

what you should be doing is this:

UserListController.addUser(). << this uses the controller to interface with the UserList Model and returns you the data you need. please fix this as soon as possible.

@vanbelle, i noticed this in your code as well. can you please fix it asap so I can test my code and make sure it works fully? thanks

— Reply to this email directly or view it on GitHub.