Release - OzWeather - Australian Weather Addon using BOM data inc. animated radar support

Code:

15:23:32 T:8564   ERROR: Error Type: <type 'exceptions.IndexError'>

15:23:32 T:8564   ERROR: Error Contents: list index out of range

15:23:32 T:8564   ERROR: Traceback (most recent call last):

                                              File "C:\Users\USERNAME\AppData\Roaming\XBMC\addons\weather.ozweather\default.py", line 540, in <module>

                                                locationName = str.split(locationName[0], ' Weather')

                                            IndexError: list index out of range

15:23:33 T:3980   ERROR: Control 1 in window 10099 has been asked to focus, but it can't

Code:

--- default.py.dist     2012-02-27 15:49:06.000000000 +1100

+++ default.py  2012-02-27 15:44:40.000000000 +1100

@@ -521,7 +521,7 @@

         text = keyboard.getText()



         #need to submit the postcode to the weatherzone search

-        searchURL = 'http://weatherzone.com.au/search/'

+        searchURL = 'http://www.weatherzone.com.au/search/'

         user_agent = 'Mozilla/4.0 (compatible; MSIE 5.5; Windows NT)'

         host = 'www.weatherzone.com.au'

         headers = { 'User-Agent' : user_agent, 'Host' : host }

@@ -530,9 +530,11 @@

         req = urllib2.Request(searchURL, data, headers)

         response = urllib2.urlopen(req)

         resultPage = str(response.read())

+#        log("OzWeather: got response", resultPage);

         #was there only one match?  If so it returns the page for that match so we need to check the URL

         responseurl = response.geturl()

-        if responseurl != 'http://weatherzone.com.au/search/':

+#        log("OzWeather: got responseurl", responseurl);

+        if responseurl != 'http://www.weatherzone.com.au/search/':

             #we were redirected to an actual result page

             locationName = common.parseDOM(resultPage, "h1", attrs = { "class": "unenclosed" })

             locationName = str.split(locationName[0], ' Weather')

@@ -540,17 +542,24 @@

             locationids = [responseurl]

         else:

             #we got back a page to choose a more specific location

-            skimmed = common.parseDOM(resultPage, "ul", attrs = { "class": "typ2" })

+            middle = common.parseDOM(resultPage, "div", attrs = { "id": "structure_middle" })

+#            log("OzWeather: got middle", middle);

+            skimmed = common.parseDOM(middle, "ul", attrs = { "class": "typ2" })

+#            skimmed = common.parseDOM(resultPage, "ul", attrs = { "class": "typ2" })

+#            log("OzWeather: got skimmed", skimmed);

             #ok now get two lists - one of the friendly names

             #and a matchin one of the URLs to store

             locations = common.parseDOM(skimmed[0], "a")

+#            log("OzWeather: got locations", locations);

+#            log("OzWeather: got locations length", len(locations));

             templocs = common.parseDOM(skimmed[0], "a", ret="href")

+#            log("OzWeather: got templocs", templocs);

             #build the full urls

             locationids = []

             for count, loc in enumerate(templocs):

                 locationids.append(WeatherZoneURL + loc)

             #if we did not get enough data back there are no locations with this postcode

-            if len(skimmed)<=1:

+            if len(locations)<=1:

                 locations = []

                 locationids = []