Author: agrundman Date: Wed Aug 31 07:09:18 2011 New Revision: 9498 URL: http://svn.slimdevices.com/jive?rev=9498&view=rev Log: r79403@andy-imac (orig r9496): mherger | 2011-08-31 03:35:21 -0400 Bug: n/a Description: in an attempt to escape the URL input screen people often accidentally invalidate the default url. If the new value is a sub-string of the default value, replace it with the default value. r79404@andy-imac (orig r9497): ayoung | 2011-08-31 09:15:07 -0400 bug 17518: Touch and Radio lost mySB.com connexion since 7.6.1 Ignore undefined player name.
Modified: 7.6/branches/usb-audio/ (props changed) 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/applets/ImageViewer/ImageSourceHttp.lua 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua Propchange: 7.6/branches/usb-audio/ ------------------------------------------------------------------------------ --- svk:merge (original) +++ svk:merge Wed Aug 31 07:09:18 2011 @@ -15,7 +15,7 @@ bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.4/trunk:8423 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/private-branches/7.5.3-rc:9282 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/trunk:9408 -bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.6/trunk:9493 +bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.6/trunk:9497 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/7.0:2013 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/SN:1083 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/scrolling:1378 Modified: 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/applets/ImageViewer/ImageSourceHttp.lua URL: http://svn.slimdevices.com/jive/7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/applets/ImageViewer/ImageSourceHttp.lua?rev=9498&r1=9497&r2=9498&view=diff ============================================================================== --- 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/applets/ImageViewer/ImageSourceHttp.lua (original) +++ 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/applets/ImageViewer/ImageSourceHttp.lua Wed Aug 31 07:09:18 2011 @@ -40,6 +40,8 @@ function __init(self, applet) log:info("initialize ImageSourceHttp") obj = oo.rawnew(self, ImageSource(applet)) + + obj:_fixImageListURL() obj.imgFiles = {} obj:readImageList() @@ -149,6 +151,8 @@ log:debug("Input " .. value) self.applet:getSettings()["http.path"] = value self.applet:storeSettings() + + self:_fixImageListURL(); window:playSound("WINDOWSHOW") window:hide(Window.transitionPushLeft) @@ -166,6 +170,21 @@ return window end + +-- in an attempt to escape the URL input screen people often accidentally invalidate the default url +-- if the new value is a sub-string of the default value, revert it +function _fixImageListURL(self) + local urlString = self.applet:getSettings()["http.path"] + local defaultUrl = self.applet:getDefaultSettings()["http.path"] + + if (urlString != defaultUrl and string.find(defaultUrl, urlString, 1, true)) then + log:warn("Invalid URL: " .. urlString) + log:warn("Replacing with default value") + self.applet:getSettings()["http.path"] = defaultUrl + self.applet:storeSettings() + end +end + --[[ =head1 LICENSE Modified: 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua URL: http://svn.slimdevices.com/jive/7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua?rev=9498&r1=9497&r2=9498&view=diff ============================================================================== --- 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua (original) +++ 7.6/branches/usb-audio/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua Wed Aug 31 07:09:18 2011 @@ -463,8 +463,8 @@ self.jnt:notify('playerNeedsUpgrade', self, self:isNeedsUpgrade(), self:isUpgrading()) end - -- Check if the player name has changed - if oldInfo.name ~= self.info.name then + -- Check if the player name has changed & is defined + if self.info.name and oldInfo.name ~= self.info.name then self.jnt:notify('playerNewName', self, self.info.name) end _______________________________________________ Jive-checkins mailing list Jive-checkins@lists.slimdevices.com http://lists.slimdevices.com/mailman/listinfo/jive-checkins