professor/cmusv — pull request #123 by sionc (2012-11-20 23:39:00+00:00)
Title: [Defect] Procrastinators ID01

Todd,

Defect
If statement does not check for array length

Description
If statement in self.email_professors_about_added_and_dropped_students does not check whether the array has any elements. It only checks for the truthy value of an array which always evaluates to true and the else block is never reached. So, if the course has no assigned faculty, chris.ziese@sv.cmu.edu will not get notified.

Resolution
The if statement now checks whether the course has any assigned faculty before
determining the email recipients

Thanks.
Team Procrastinators