session_id is not a great name either, since the sessions table has an id field and a session_id field, and this is neither.  I think my inclination is to be explicit and call it session_user_id.  I would say we should do it as a simple accessor and just assign it as an id every time.  so self.session, self.session_user, and self.session_id could all go away, right?