Merge pull request #164 from oat-sa/backport/DPF-514/tt-update-event-…

…trigger-timing Backport/dpf 514/tt update event trigger timing
oat-sa · Sep 29, 2020 · 8a452d0 · 8a452d0
2 parents 0354ed6 + 6bb03d6
commit 8a452d0
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/actions/TestTaker.php b/actions/TestTaker.php
@@ -179,10 +179,6 @@ public function editSubject()
                 $data = ['hashForKey' => UserHashForEncryption::hash($plainPassword)];
             }
 
-            $this->getEventManager()->trigger(
-                new TestTakerUpdatedEvent($subjectUri,array_merge($values, $data))
-            );
-
             if ($addMode) {
                 // force default subject roles to be the Delivery Role:
                 $this->getClassService()->setTestTakerRole($subject);
@@ -195,6 +191,10 @@ public function editSubject()
                 GenerisRdf::PROPERTY_USER_DEFLG => $lang->getUri()
             ));
 
+            $this->getEventManager()->trigger(
+                new TestTakerUpdatedEvent($subjectUri, array_merge($values, $data))
+            );
+
             $message = __('Test taker saved');
 
             if ($addMode) {

diff --git a/manifest.php b/manifest.php
@@ -33,7 +33,7 @@
     'label' => 'Test-taker core extension',
 	'description' => 'TAO TestTaker extension',
     'license' => 'GPL-2.0',
-    'version' => '7.1.3',
+    'version' => '7.1.4',
 	'author' => 'Open Assessment Technologies, CRP Henri Tudor',
 	'requires' => array(
 	    'tao' => '>=36.1.0',

diff --git a/scripts/update/Updater.php b/scripts/update/Updater.php
@@ -98,6 +98,6 @@ public function update($initialVersion)
             $this->setVersion('3.11.0');
         }
 
-        $this->skip('3.11.0', '7.1.3');
+        $this->skip('3.11.0', '7.1.4');
     }
 }