Changeset 2720 for WAeUP_SRP/base/skins

Timestamp:

20 Nov 2007, 21:49:53 (17 years ago)

Author:

Henrik Bettermann

Message:

catch 'no workflow provides ...' traceback

File:

• WAeUP_SRP/base/skins/waeup_student/deactivate_students.py (modified) (1 diff)

WAeUP_SRP/base/skins/waeup_student/deactivate_students.py

@@ -45 +45 @@
         for student_id in ids:
             student = getattr(students_folder,student_id,None)
+            student_state = context.portal_workflow.getInfoFor(student,'review_state',None)
             if student is not None:
                 for object in student.objectValues():
                     if context.portal_workflow.getInfoFor(object,'review_state',None) in ('open','created'):
                         context.portal_workflow.doActionFor(object,'close')
-                context.portal_workflow.doActionFor(student,'deactivate')
-                logger.info("%s deactivated student  %s" % (member,student_id))
+                try:
+                    context.portal_workflow.doActionFor(student,'deactivate')
+                except:
+                    logger.info("%s failed to deactivate student %s in state %s" % (member,student_id,student_state))
+                    deleted_list.append(student_id + ' (failed)')
+                    continue
+                logger.info("%s deactivated student %s" % (member,student_id))
                 deleted_list.append(student_id)
             else:
                 logger.info("Student object %s not found" % (student_id))
                 continue
-        psm = "%s deactivated" % ' '.join(deleted_list)
+        psm = "Deactivated: %s" % ' '.join(deleted_list)
 
 if student_ids: