Changeset 3611

Timestamp:

14/03/08 16:42:24 (11 years ago)

Author:

cbyrom

Message:

Add redirect in controller for viewItems if not items are currently selected - to avoid page crash.

File:

• TI05-delivery/ows_framework/trunk/ows_server/ows_server/controllers/viewItems.py (modified) (1 diff)

TI05-delivery/ows_framework/trunk/ows_server/ows_server/controllers/viewItems.py

@@ -28 +28 @@
         #    selected items;
         # ii) or from the selections tab - which will already have set up the correct selected items
-        
-        # firstly check if page has been visited before; if not display tab
-        if (Utilities.isTabRequired(c.pageTabs, 'View')):
-            c.pageTabs.append(('View', h.url_for(controller='viewItems',action='index')))
             
         # if ENDPOINT specified, we've reached the page via the WMC icon
         if ('ENDPOINT' in self.inputs):
             self.addViewItem(self.inputs['ENDPOINT'])
-                                    
+        
+        # avoid page crashing if we come here without view items defined
+        if 'viewItems' not in session:
+            h.redirect_to(h.url_for(controller='discovery'))
+                                        
         session.save()
+        
+        # check if page has been visited before; if not display tab
+        if (Utilities.isTabRequired(c.pageTabs, 'View')):
+            c.pageTabs.append(('View', h.url_for(controller='viewItems',action='index')))
+
         return render('viewItems')