Sync function should take empty string and convert to nil

In previous versions we converted a sync function supplied as an empty string to nil. Setting this to nil means the default will be used in other places.

This empty string conversion is done inside applySyncFunction / UpdateSyncFun. 

We should maintain this behaviour in 3.0 although perhaps we should do this higher up in validate where we are currently blocking invalid javascript / or empty strings.