1) When reading the (encoded) initcmd string into Form_Textarea(), decode it. Previously the encoded value was displaying in the text area and when the user presses save, that encode string is itself encoded again in the config and thus stuffs things up. 2) When loading the page, only use set_gps_default() to set initcmd if initcmd is currently empty. Previously the initcmd value would be set to the default string for the GPS type every time the page loaded, which would override any change that the user had made. Note: After this is fixed, then I will also submit changes to make the Advanced button work in the new ways.
services_ntpd_gps fix handling of initcmd
1) When reading the (encoded) initcmd string into Form_Textarea(), decode it.
Previously the encoded value was displaying in the text area and when the user presses save, that encode string is itself encoded again in the config and thus stuffs things up.
2) When loading the page, only use set_gps_default() to set initcmd if initcmd is currently empty.
Previously the initcmd value would be set to the default string for the GPS type every time the page loaded, which would override any change that the user had made.
Note: After this is fixed, then I will also submit changes to make the Advanced button work in the new ways.