search

JMU-CIME / CPR-Music-Backend

MIT License
4 stars 28 forks source link

`TODO` in B/E > `musics/models.py:Part.for_activity` #52

Open hcientist opened 7 months ago

hcientist commented 7 months ago

From a glance, the todo seems to be resolved. It was more of a question than a true task to complete, that question being whether a check for null is necessary; the check is then provided. I wouldn't imagine checking could hurt anything here, even if it is not strictly necessary.

The segment

hcientist commented 7 months ago

There is at least an else in there, but does it work? i.e. what if indeed the if fails, will the else currently succeed?