Changeset 596 for WAeUP_SRP/trunk/skins/waeup_student/searchStudents.py

Timestamp:

29 Sep 2006, 14:14:40 (18 years ago)

Author:

joachim

Message:

student search should work now as follows:

if All States is selected a search string must be specified.
if another review state is specified, the searchstring can be empty.
if it is not empty, both are ANDed.

File:

• WAeUP_SRP/trunk/skins/waeup_student/searchStudents.py (modified) (3 diffs)

WAeUP_SRP/trunk/skins/waeup_student/searchStudents.py

@@ -54 +54 @@
 state = ds.get('review_state')
 term = ds.get('search_string')
-if state == " ----- ":
-    state = ''
 err = False
 if not term and what in ('student_edit','jamb_id','name','matric_no'):
-    psm = "You must specify a search string"
+    psm = "You must specify a search string when searching in 'All States'"
     err = True
 elif '*' in term:
-    psm = "You cannot use '*'"
+    psm = "you cannot use the '*' alone"
     err = True
 if err: 
@@ -70 +68 @@
                              is_manager = is_manager,
                              )
-onlyreview = state and what == "review_state"
+with_review = state != "all"
 items = []
 res = []
 portal_type_query = {'query':['Student','StudentApplication','StudentPersonal']}
 st_queries = ('jamb_id','matric_no','name')
+onlyreview = with_review and not what
 if onlyreview:
     res = catalog(portal_type=portal_type_query,
                   review_state=state)
 elif what == "student_id":
-    res = catalog(portal_type='Student',
-                  id = term.strip())
+    if with_review:
+        res = catalog(portal_type='Student',
+                      id = term.strip(),
+                      review_state=state)
+    else:
+        res = catalog(portal_type='Student',
+                      id = term.strip())
 elif what in st_queries: 
     if what == "jamb_id":
@@ -91 +95 @@
         pt = portal_type_query
         st = "%s*" % term.strip()
-    try:
-        res = catalog(portal_type=pt,SearchableText=st,)
-    except:
-        return context.students_manager_view(rendered = rend,
-                             psm = 'Searchstring "%s" not allowed' % term,
-                             #psm = "%s, %s" % (psm,ds),
-                             students = items,
-                             is_manager = is_manager,
-                             )
+    if with_review:
+        try:
+            res = catalog(portal_type=pt,SearchableText=st,
+                          review_state=state)
+        except:
+            return context.students_manager_view(rendered = rend,
+                                 psm = 'Searchstring "%s" not allowed' % term,
+                                 #psm = "%s, %s" % (psm,ds),
+                                 students = items,
+                                 is_manager = is_manager,
+                                 )
+    else:
+        try:
+            res = catalog(portal_type=pt,SearchableText=st,)
+        except:
+            return context.students_manager_view(rendered = rend,
+                                 psm = 'Searchstring "%s" not allowed' % term,
+                                 #psm = "%s, %s" % (psm,ds),
+                                 students = items,
+                                 is_manager = is_manager,
+                                 )
             
 if res: