I'm not 100% sure about this. It does work, and it does not get run at all in any instance I could find, so I suspect it does not break any existing behaviour. Still, it might be more correct for the getElementValueInEntry() function to return the value instead.