@dglance :  This is okay to keep in the model for now since it's not breaking anything, but you / your team should consider using django's native support for ManyToMany fields to do this type of Instructor <=> Course mapping.  In fact I think it's compatible with the code as you have it now, just need to add a ManyToMany field in class Instructor, then queries you make in additional_page.views and preview_login, etc. can be much more compactly stated.