Does that make sense

Sent from my iPhone

> On Nov 15, 2013, at 9:10 PM, Jeff Carouth <notifications@github.com> wrote:
> 
> In site/src/MentorApp/PartnershipManager.php:
> 
> > +     */
> > +    public function __construct(\PDO $db)
> > +    {
> > +        $this->db = $db;
> > +    }
> > +
> > +    /**
> > +     * Create a partnership between a mentor and an apprentice
> > +     *
> > +     * @param \MentorApp\User $mentor mentoring user instance
> > +     * @param \MentorApp\User $apprentice apprentice user instance
> > +     * @return boolean
> > +     */
> > +    public function create(User $mentor, User $apprentice)
> > +    {
> > +        $id = '';
> Is the ID supposed to be actually generated? I don't think this is going to work as you expect given that id is set to the primary key of the table and afaict $id is only ever ''.
> 
> —
> Reply to this email directly or view it on GitHub.