BSONObj::getObjectID is declared as:

bool getObjectID(BSONElement& e);

but it does not change the BSONObj, so just like the rest of the field getters, it should be const:

bool getObjectID(BSONElement& e) const;

That const_cast in my code is looking ugly