loopback-example-access-control
loopback-example-access-control copied to clipboard
strongloop
Refactor `user` to another name
We get a lot of confusion from my original implementation of user extending builtin User. We should rename it to something like CustomUser or something to signify it's obviously not the same as builtin User.
Thanks for reporting that ;)
The official documentation tells us to use Customer or Client when overriding the User model.
Moreover, I am not conviced that mixing naming conventions in the example is a good practice.
https://docs.strongloop.com/display/public/LB/Managing+users
I think that using the same names as in the official documentation Customer, Client would be a good idea. I might highy help people understanding the examples.
Haha, actually user was the official name before and we added Customer/Client because we had to keep explaining it before, but never got around to updating the examples to match. ;) I think we'll go with Customer because Client will probably cause other issues down the line. Thanks for the feedback.
Ok I get it.
I don't mind using User twice. That's even what I would have done if
not told otherwise in the doc. What bother me more is that it's once
written in lowercase and the second time with an uppercase U
No, it has to be lowercase or it will conflict with the builtin User model. That or you can name it something other than "User".
the User vs user issue also breaks down if you use mysql as a backend, since it can't distinguish between tables by case.
Account seems to be distinguishable enough and independent from actual user purpose (Client, Customer etc)
