by Danny Wahl.
I'm rewriting the renderer for login_info() for a theme but there's one conditional that doesn't make sense to me (I mean I think I understand what it's doing - but I don't know why we're doing that...)
if (is_mnet_remote_user($USER) and $idprovider = $DB->get_record('mnet_host', array('id'=>$USER->mnethostid))) { ... }What I don't understand is the second part. Shouldn't $idprovider ALWAYS be instantiated? It might be the DB record, it might be an error, it might be NULL - but it will always exist, so what's the point, and what's it doing?