I'm talking about the test at L352, where the value of the attribute is tested for null.  We fall back to (next).getID() in this case, which seems a little sketchy to me.