added manager classes and serialize functions #37
Conversation
| @@ -2,6 +2,15 @@ | |||
| from django.db import models | |||
|
|
|||
| # Create your models here. | |||
| class ArticleManager(models.Manager): | |||
| def serialize(self, account, filter=None): | |||
There was a problem hiding this comment.
lol don't copy paste code my man! This isn't an account object we are receiving, so why call it account? We also don't use the filter object so why pass it in? Half of the reason you write code is so someone can read it and understand what is going on.!
There was a problem hiding this comment.
If you look at my code in account.py carefully, you will notice that the data is being retrieved from the variable being passed into the function. Not the model objects.
if i were to do:
data = { "email": Account.email }
the code would break because the Account object has no attribute called email, only the account variable that we pass into the function which is an instance of an Account object and not the class itself.
fixed objects definitions and serialize method definitions
Added manager classes with serialize functions to the models that did not have them