@owend That's a good idea, but the `data` property of `WallNotificationEntity` is just an instance of `stdClass` so there's a possibility that the `url` property might not be defined (although I don't know why), in which case you'd get an error when trying to pass it to `fixNotificationURL()`. I'm assuming that's why the empty check was there originally.