Changeset 12620 for main/waeup.kofa/trunk/src/waeup/kofa/university

Timestamp:

16 Feb 2015, 11:27:24 (10 years ago)

Author:

Henrik Bettermann

Message:

Improve department and certificate movers. Add utility view for moving certificates.

Location:

main/waeup.kofa/trunk/src/waeup/kofa/university

Files:

• certificate.py (modified) (1 diff)
• department.py (modified) (1 diff)
• tests/test_move.py (modified) (8 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/university/certificate.py

@@ -120 +120 @@
         return
 
-    def moveCertificate(self, fac, dep):
+    def moveCertificate(self, facname, depname, newcode):
         self.moved = True
         cert = self
-        del self.__parent__[cert.code]
-        grok.getSite()['faculties'][fac][dep].certificates[cert.code] = cert
-        self.__parent__._p_changed = True
+        oldcode = cert.code
+        cert.code = newcode
+        del self.__parent__[oldcode]
+        newdep = grok.getSite()['faculties'][facname][depname]
+        newdep.certificates[newcode] = cert
+        #self.__parent__._p_changed = True
         cat = getUtility(ICatalog, name='students_catalog')
-        results = cat.searchResults(certcode=(cert.code, cert.code))
+        results = cat.searchResults(certcode=(oldcode, oldcode))
         for student in results:
             notify(grok.ObjectModifiedEvent(student))
             student.__parent__.logger.info(
                 '%s - Certificate moved' % student.__name__)
-
         return
 

main/waeup.kofa/trunk/src/waeup/kofa/university/department.py

@@ -102 +102 @@
         """
         self.moved = True
-        fac = grok.getSite()['faculties'][facname]
+        newfac = grok.getSite()['faculties'][facname]
         oldcode = self.code
-        fac[depname] = self
-        del self.__parent__[oldcode]
-        fac[depname].code = depname
+        oldfac = self.__parent__
+        newfac[depname] = self
+        del oldfac[oldcode]
+        newfac[depname].code = depname
         #self.__parent__._p_changed = True
         cat = getUtility(ICatalog, name='students_catalog')

main/waeup.kofa/trunk/src/waeup/kofa/university/tests/test_move.py

@@ -22 +22 @@
 from zope.catalog.interfaces import ICatalog
 from zope.component import queryUtility
+from waeup.kofa.university.faculty import Faculty
 from waeup.kofa.university.department import Department
 from waeup.kofa.students.tests.test_browser import StudentsFullSetup
@@ -28 +29 @@
 
     def test_move_department(self):
+        self.app['faculties']['fac2'] = Faculty(code=u'fac2')
         self.assertEqual(
             self.app['faculties']['fac1']['dep1'].certificates['CERT1'],
@@ -36 +38 @@
         # We move the depart using the UtilityView
         self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
-        self.browser.open('http://localhost/app/faculties/fac1/dep1/move_department?fac=fac1&dep=newdep')
+        self.browser.open('http://localhost/app/faculties/fac1/dep1/move_department?fac=fac2&dep=newdep')
 
         # New department is created and old department is removed
         self.assertEqual(
-            [key for key in self.app['faculties']['fac1']], ['newdep'])
+            [key for key in self.app['faculties']['fac2']], ['newdep'])
 
         # All subobjects have been moved too
         self.assertEqual(
-            self.app['faculties']['fac1']['newdep'].courses['COURSE1'],
+            self.app['faculties']['fac2']['newdep'].courses['COURSE1'],
             self.course)
         self.assertEqual(
-            self.app['faculties']['fac1']['newdep'].certificates['CERT1'],
+            self.app['faculties']['fac2']['newdep'].certificates['CERT1'],
             self.certificate)
         self.assertEqual(
-            [key for key in self.app['faculties']['fac1'][
+            [key for key in self.app['faculties']['fac2'][
                 'newdep'].certificates['CERT1'].keys()],
             [u'COURSE1_100'])
@@ -67 +69 @@
         assert len(results) == 1
         assert results[0] is self.course
+
         # We can still find the certificatecourse in the catalog
         cat = queryUtility(ICatalog, name='certcourses_catalog')
@@ -72 +75 @@
         results = [x for x in results] 
         assert len(results) == 1
-        assert results[0] is self.app['faculties']['fac1'][
+        assert results[0] is self.app['faculties']['fac2'][
             'newdep'].certificates['CERT1']['COURSE1_100']
 
@@ -91 +94 @@
             self.app['datacenter'].storage, 'logs', 'main.log')
         logcontent = open(logfile).read()
-        self.assertTrue('INFO - zope.mgr - Department dep1 moved to fac1/newdep'
+        self.assertTrue('INFO - zope.mgr - Department dep1 moved to fac2/newdep'
             in logcontent)
 
 
     def test_move_certificate(self):
+        self.app['faculties']['fac2'] = Faculty(code=u'fac2')
+        self.app['faculties']['fac2']['dep2'] = Department(code=u'dep2')
         self.assertEqual(
             self.app['faculties']['fac1']['dep1'].certificates['CERT1'],
             self.certificate)
-        self.app['faculties']['fac1']['dep2'] = Department(code=u'dep2')
-        self.certificate.moveCertificate('fac1', 'dep2')
+
+        #self.certificate.moveCertificate('fac2', 'dep2', 'NEWCODE')
+
+        # We move the depart using the UtilityView
+        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+        self.browser.open('http://localhost/app/faculties/fac1/dep1/certificates/CERT1/move_certificate?fac=fac2&dep=dep2&cert=NEWCODE')
+
         self.assertEqual(
-            self.app['faculties']['fac1']['dep2'].certificates['CERT1'],
+            self.app['faculties']['fac2']['dep2'].certificates['NEWCODE'],
             self.certificate)
         self.assertEqual(
-            [key for key in self.app['faculties']['fac1'][
-                'dep2'].certificates['CERT1'].keys()],
-            [u'COURSE1_100'])
+            [key for key in self.app['faculties']['fac2'][
+                'dep2'].certificates['NEWCODE'].keys()], [u'COURSE1_100'])
         self.assertEqual(
             self.student['studycourse'].certificate,
             self.certificate)
 
-        # We can still find the certificate in the catalog
+        # We can find the moved certificate in the catalog
         cat = queryUtility(ICatalog, name='certificates_catalog')
-        results = cat.searchResults(code=('CERT1','CERT1'))
+        results = cat.searchResults(code=('NEWCODE','NEWCODE'))
         results = [x for x in results] # Turn results generator into list
         assert len(results) == 1
@@ -124 +133 @@
         results = [x for x in results] # Turn results generator into list
         assert len(results) == 1
-        assert results[0] is self.app['faculties']['fac1'][
-                'dep2'].certificates['CERT1']['COURSE1_100']
+        assert results[0] is self.app['faculties']['fac2'][
+                'dep2'].certificates['NEWCODE']['COURSE1_100']
 
         # We can find the student studying in the new department
@@ -133 +142 @@
         assert len(results) == 1
         assert results[0] is self.app['students'][self.student_id]
+
+        # We can find the student studying the new course ...
+        cat = queryUtility(ICatalog, name='students_catalog')
+        results = cat.searchResults(certcode=('NEWCODE','NEWCODE'))
+        results = [x for x in results] # Turn results generator into list
+        assert len(results) == 1
+        assert results[0] is self.app['students'][self.student_id]
+        # ... but not the old course
+        results = cat.searchResults(certcode=('CERT1','CERT1'))
+        assert len(results) == 0
+
         logfile = os.path.join(
             self.app['datacenter'].storage, 'logs', 'students.log')
         logcontent = open(logfile).read()
-        self.assertTrue('INFO - system - K1000000 - Certificate moved'
+        self.assertTrue('INFO - zope.mgr - K1000000 - Certificate moved'
             in logcontent)
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'main.log')
+        logcontent = open(logfile).read()
+        self.assertTrue('INFO - zope.mgr - Certificate CERT1 moved to fac2/dep2/NEWCODE'
+            in logcontent)