The URLCleanup function used to take a Knockout observable errorObservable as a parameter and had a default to always yield the empty string when the parameter was null. This has changed to an errorCallback function, but there was no default for it. Therefore, the edit URL page that didn’t provide a callback caused a type error, and the cleanup code didn’t run.
This commit reinstates the previous behaviour: If no callback is given, it defaults to a function always returning the empty string.
The
URLCleanupfunction used to take a Knockout observableerrorObservableas a parameter and had a default to always yield the empty string when the parameter was null. This has changed to anerrorCallbackfunction, but there was no default for it. Therefore, the edit URL page that didn’t provide a callback caused a type error, and the cleanup code didn’t run.This commit reinstates the previous behaviour: If no callback is given, it defaults to a function always returning the empty string.