Changeset 6966 for main/waeup.sirp/trunk

Timestamp:

29 Oct 2011, 17:26:32 (13 years ago)

Author:

Henrik Bettermann

Message:

Catch KeyError?.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/hostels/browser.py (modified) (3 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/hostels/browser.py

@@ -116 +116 @@
 
     # It's quite dangerous to remove entire hostels with its content (beds).
-    # Thus, this removed should be combined with an archiving function.
+    # Thus, this remove method should be combined with an archiving function.
     @grok.action('Remove selected')
     def delHostels(self, **data):
@@ -123 +123 @@
             deleted = []
             child_id = form['val_id']
-            child_id = [child_id]
+            if not isinstance(child_id, list):
+                child_id = [child_id]
             for id in child_id:
                 deleted.append(id)
@@ -152 +153 @@
         self.applyData(hostel, **data)
         hostel.hostel_id = data['hostel_name'].lower().replace(' ','_')
-        self.context.addHostel(hostel)
+        try:
+            self.context.addHostel(hostel)
+        except KeyError:
+            self.flash('The hostel already exists.')
+            return
         self.flash('Hostel created.')
         write_log_message(self, 'added: % s' % data['hostel_name'])