I think $object->getPHID() is potentially null here (i.e., this code can run during creation of a new object, before the object is saved). We'll still get the right result since it's impossible to mention a nonexistent object, but adding an explicit if ($object->getPHID()) { ... } check might make this more clear some day down the line.