Without have read the full PR - why is it checking for `not value` instead of for `doc`?