Changeset 1515 for WAeUP_SRP/trunk/skins

Timestamp:

3 Mar 2007, 11:45:58 (18 years ago)

Author:

Henrik Bettermann

Message:

CouseAdviser? role modifications

Location:

WAeUP_SRP/trunk/skins

Files:

• waeup_custom/getCPSCandidateLocalRoles.py (modified) (1 diff)
• waeup_default/isCourseAdviser.py (modified) (1 diff)
• waeup_student/getStudyLevelInfo.py (modified) (1 diff)
• waeup_student/notify_change_courses.py (deleted)
• waeup_student/reject_courses.py (added)
• waeup_student/study_level_view.pt (modified) (7 diffs)

WAeUP_SRP/trunk/skins/waeup_custom/getCPSCandidateLocalRoles.py

@@ -24 +24 @@
 
 if member.id in ("admin","perry"):
-    return ['SectionReader','SectionManager','ClearanceOfficer', 'SectionOfficer']
-return ['SectionReader', 'ClearanceOfficer', 'SectionOfficer']
+    return ['SectionReader','SectionManager','ClearanceOfficer', 'SectionOfficer', 'CourseAdviser']
+return ['SectionReader', 'ClearanceOfficer', 'SectionOfficer', 'CourseAdviser']

WAeUP_SRP/trunk/skins/waeup_default/isCourseAdviser.py

@@ -16 +16 @@
 mtool = context.portal_membership
 roles = mtool.getAuthenticatedMember().getRolesInContext(context)
-return context.isSectionOfficer or "ClearanceOfficer" in roles
+return "CourseAdviser" in roles
+

WAeUP_SRP/trunk/skins/waeup_student/getStudyLevelInfo.py

@@ -52 +52 @@
 student_id = context.getStudentId()
 
+
+
 info = {}
-info['is_student'] = is_student = context.isStudent() 
-info['is_ca'] = is_ca = context.isCourseAdviser() 
+info['is_so'] = is_so = context.isSectionOfficer()
+info['is_student'] = is_student = context.isStudent()
+info['is_ca'] = is_ca = context.isCourseAdviser()
 info['student'] = student = context.students_catalog(id=student_id)[0]
 info['review_state'] = review_state = context.getStudentReviewState()
-info['view_only'] =  review_state != "school_fee_paid" 
-info['show_check_boxes'] =  (is_ca and review_state == 'courses_registered') or (is_student and context.getStudentReviewState() == "school_fee_paid") 
+info['view_only'] =  review_state != "school_fee_paid"
+info['show_check_boxes'] =  (is_ca and review_state in ('courses_registered',)) or\
+                            (is_student and context.getStudentReviewState() == "school_fee_paid") or\
+                            (is_so)
 info['choosen_ids'] = request.get('ids',[])
 info['status_info'] = ""
 if is_student:
-    if review_state == 'courses_registered': 
+    if review_state == 'courses_registered':
         info['status_info'] = "Request for Course Validation pending"
-    elif review_state == 'courses_validated': 
+    elif review_state == 'courses_validated':
         info['status_info'] = "Courses validated"
 elif is_ca:
-    if review_state == 'courses_registered': 
+    if review_state == 'courses_registered':
         info['status_info'] = "Please validate these Courses"
-    elif review_state == 'courses_validated': 
+    elif review_state == 'courses_validated':
         info['status_info'] = "Courses validated"
 info['doc'] = context.getContent()

WAeUP_SRP/trunk/skins/waeup_student/study_level_view.pt

@@ -26 +26 @@
 
       <h3> 
-        <span tal:condition="is_so">
+        <span tal:condition="python: is_so or is_ca">
           <span tal:content="info/student/name" />:
         </span>
-        <span tal:content="here/title_or_id" />
+        <span tal:condition="is_student">My </span>
+        
+        Course Registration Form for <span tal:content="here/title_or_id" />
       </h3>
-      <br />
+
     </metal:block>
     <metal:main fill-slot="main" tal:condition="python:1">
-      <h3 tal:content="status_info"></h3>
       <form action="." method="post" class="group">
         <span tal:condition="info/carry_overs" tal:omit-tag="">
-        <h3>Carryover Courses</h3>
+        <h4>Carryover Courses</h4>
         <br />
         <table tal:condition="python: 1" class="contentListing" width="100%" summary="content layout" id="folder_content">
@@ -58 +59 @@
           </tr>
         </table>
-        </span>
-        <h3>Session Courses</h3>
+        </span> 
+        <br />
+        <h4>Session Courses</h4>
         <br />
         <table class="contentListing" width="100%" summary="content layout" id="folder_content">      
@@ -65 +67 @@
               tal:attributes="class python:test(repeat['row'].even(), 'even ajaxtd', 'odd ajaxtd')">
             <td align="left" valign="middle" style="width: 5px;"
-                tal:condition="show_check_boxes">
+                      tal:condition="show_check_boxes">
               <input type="checkbox" name="ids:list" value="" id="" class="noborder"
                      tal:define="id row/id"
@@ -80 +82 @@
           </tr>
           <tr>
-            <td colspan="1"></td>
+            <td tal:condition="show_check_boxes" colspan="1"></td>
             <td colspan="1"></td>
             <th colspan="2">Total Credits:</th>
@@ -86 +88 @@
           </tr>
         </table>
-        <table tal:condition="not:view_only" width="100%" cellspacing="0" cellpadding="2"
+
+        
+      
+        
+        <table tal:condition="python: is_student and review_state == 'school_fee_paid'" width="100%" cellspacing="0" cellpadding="2"
                class="folderButtons">
           <tr>
             <td align="left" valign="top" rowspan="3"></td>
             <td align="left" valign="top">
-              <span tal:condition="python:1">
-                <input type="button" value="button_select_all" class="context"
-                       onclick="someJavaScriptFunctionThatWillBeReplaced"
-                       i18n:attributes="value"
-                       tal:attributes="onclick python:'toggleSelect(this, \'%s\', \'%s\')'
-                       % (cpsmcat('button_select_all'), cpsmcat('button_deselect_all'))"
-                       />
-                <input type="submit" name="course_result_delete:method" value="Delete checked"
+                <input type="submit" name="course_result_delete:method" value="Delete Selected"
                        class="destructive" i18n:attributes="value"
                        tal:attributes="onclick python:'return window.confirm(\'%s\')' %
-                       ('really delete checked courses', )" 
+                       ('Do you really want to delete selected courses?', )" 
                        />
-                <input tal:condition="python:1" type="submit" name="register_courses:method"
-                       class="context" value="Register" />
-                <input tal:condition="is_ca" type="submit" name="notify_change_courses:method"
-                       class="context" value="Reject Courses" />
-                <input tal:condition="python: 1"
+                <input 
                        type="submit" name="refresh_level:method"
-                       class="context" value="Refresh List" 
+                       class="context" value="Check for Courses" 
                        />
-              </span>
+                <br /><br />
+                <input type="submit" name="register_courses:method"
+                       class="context" value="Submit Course List" />
             </td>
           </tr>
         </table>
-        <table tal:condition="python:is_ca and show_check_boxes" width="100%" cellspacing="0" cellpadding="2"
+        <table tal:condition="python:is_so" width="100%" cellspacing="0" cellpadding="2"
                class="folderButtons">
           <tr>
             <td align="left" valign="top" rowspan="3"></td>
             <td align="left" valign="top">
-              <span tal:condition="python:1">
-                <input type="button" value="button_select_all" class="context"
+                <input type="button" value="Select All" class="context"
                        onclick="someJavaScriptFunctionThatWillBeReplaced"
                        i18n:attributes="value"
@@ -127 +123 @@
                        % (cpsmcat('button_select_all'), cpsmcat('button_deselect_all'))"
                        />
-                <input type="submit" name="course_result_delete:method" value="Delete checked"
+                <input type="submit" name="course_result_delete:method" value="Delete Selected"
                        class="destructive" i18n:attributes="value"
                        tal:attributes="onclick python:'return window.confirm(\'%s\')' %
                        ('really delete checked courses', )" 
                        />
-                <input tal:condition="python:1" type="submit" name="validate_courses:method"
-                       class="context" value="Validate" />
-                <input tal:condition="is_ca" type="submit" name="notify_change_courses:method"
-                       class="context" value="Reject Courses" />
-                <input tal:condition="python: 1"
+                <input 
                        type="submit" name="refresh_level:method"
-                       class="context" value="Refresh List" 
+                       class="context" value="Check for Courses" 
                        />
-              </span>
+                <input tal:condition="python: review_state == 'school_fee_paid'" type="submit" name="register_courses:method"
+                       class="context" value="Submit Course List" />  
+                <input tal:condition="python: review_state == 'courses_registered'" type="submit" name="validate_courses:method"
+                       class="context" value="Validate List" />
+                <input tal:condition="python: review_state in ('courses_registered', 'courses_validated',)" type="submit" name="reject_courses:method"
+                       class="context" value="Reject Course List" />
+
             </td>
           </tr>
         </table>
+        <table tal:condition="python:is_ca" width="100%" cellspacing="0" cellpadding="2"
+               class="folderButtons">
+          <tr>
+            <td align="left" valign="top" rowspan="3"></td>
+            <td align="left" valign="top">
+                <input tal:condition="python: review_state == 'school_fee_paid'"
+                       type="button" value="Select All" class="context"
+                       onclick="someJavaScriptFunctionThatWillBeReplaced"
+                       i18n:attributes="value"
+                       tal:attributes="onclick python:'toggleSelect(this, \'%s\', \'%s\')'
+                       % (cpsmcat('button_select_all'), cpsmcat('button_deselect_all'))"
+                       />
+                <input tal:condition="python: review_state == 'school_fee_paid'"
+                       type="submit" name="course_result_delete:method" value="Delete Selected"
+                       class="destructive" i18n:attributes="value"
+                       tal:attributes="onclick python:'return window.confirm(\'%s\')' %
+                       ('really delete checked courses', )" 
+                       />
+                <input tal:condition="python: review_state == 'school_fee_paid'"
+                       type="submit" name="refresh_level:method"
+                       class="context" value="Check for Courses" 
+                       />
+                <input tal:condition="python: review_state == 'school_fee_paid'" type="submit" name="register_courses:method"
+                       class="context" value="Submit Course List" />  
+                <input tal:condition="python: review_state == 'courses_registered'" type="submit" name="validate_courses:method"
+                       class="context" value="Validate List" />
+                <input tal:condition="python: review_state in ('courses_registered', 'courses_validated',)" type="submit" name="reject_courses:method"
+                       class="context" value="Reject Course List" />
+
+            </td>
+          </tr>
+        </table>
+
       </form>
       <br />
-      <form action="add_course_result" method="post" class="group"
+        <form action="add_course_result" method="post" class="group"
             tal:condition="show_check_boxes">
         <table>
@@ -155 +186 @@
             <td tal:condition="request/error|nothing" 
                 tal:content="request/error" />
-          </tr>
-          <tr>
-            <th></th>
             <td colspan="2">
               <input type="submit" name="add"
                      class="context" value="Add Course" 
                      />
-            </td>
-          </tr>
+            </td>                
         </table>
-      </form>          
+        </form>         
     </metal:main>
   </metal:body>