Changeset 7329 for main

Timestamp:

11 Dec 2011, 13:18:40 (13 years ago)

Author:

Henrik Bettermann

Message:

Also protect students section against unintentional deletions with new jsaction.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/students

Files:

• browser.py (modified) (10 diffs)
• browser_templates/containermanagepage.pt (modified) (1 diff)

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

@@ -35 +35 @@
     ContactAdminForm)
 from waeup.sirp.browser.breadcrumbs import Breadcrumb
-from waeup.sirp.browser.resources import datepicker, datatable, tabs
+from waeup.sirp.browser.resources import datepicker, datatable, tabs, warning
 from waeup.sirp.browser.viewlets import (
     ManageActionButton, AddActionButton)
+from waeup.sirp.browser.layout import jsaction, JSAction
 from waeup.sirp.interfaces import (
     ISIRPObject, IUserAccount, IExtFileStore, IPasswordValidator, IContactForm)
@@ -270 +271 @@
         datatable.need()
         toggleall.need()
+        warning.need()
         form = self.request.form
         self.hitlist = []
@@ -721 +723 @@
         super(StudyCourseManageFormPage, self).update()
         tabs.need()
+        warning.need()
         datatable.need()
         return 
@@ -749 +752 @@
         return
 
-    @grok.action('Remove selected levels')
+    @jsaction('Remove selected levels')
     def delStudyLevels(self, **data):
         form = self.request.form
@@ -869 +872 @@
         super(StudyLevelManageFormPage, self).update()
         tabs.need()
+        warning.need()
         datatable.need()
         return
@@ -889 +893 @@
         self.redirect(self.url(self.context, '@@add'))
 
-    @grok.action('Remove selected tickets')
+    @jsaction('Remove selected tickets')
     def delCourseTicket(self, **data):
         form = self.request.form
@@ -1052 +1056 @@
         super(PaymentsManageFormPage, self).update()
         datatable.need()
-        return
-
-    @grok.action('Remove selected tickets')
+        warning.need()
+        return
+
+    @jsaction('Remove selected tickets')
     def delPaymentTicket(self, **data):
         form = self.request.form
@@ -1314 +1319 @@
         super(AccommodationManageFormPage, self).update()
         datatable.need()
-        return
-
-    @grok.action('Remove selected')
+        warning.need()
+        return
+
+    @jsaction('Remove selected')
     def delBedTickets(self, **data):
         if getattr(self.request.principal, 'user_type', None) == 'student':
@@ -2010 +2016 @@
     #    tabs.need()
         datatable.need()
+        warning.need()
         return
 
@@ -2031 +2038 @@
         self.redirect(self.url(self.context, 'ctadd'))
 
-    @grok.action('Remove selected tickets')
+    @jsaction('Remove selected tickets')
     def delCourseTicket(self, **data):
         form = self.request.form

main/waeup.sirp/trunk/src/waeup/sirp/students/browser_templates/containermanagepage.pt

@@ -49 +49 @@
     
     <div class="actionButtons">
-        <input type="submit" name="remove" value="Remove selected" />
+        <input type="submit" name="remove" value="Remove selected"
+               onclick="return confirmPost('Are you sure?')"/>
     </div>