Emit postSetEnabled. Closes #23970

owncloud · Jan 20, 2017 · d4f79f5 · d4f79f5
1 parent 1adc983
commit d4f79f5
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/lib/private/User/User.php b/lib/private/User/User.php
@@ -337,6 +337,9 @@ public function setEnabled($enabled) {
 		$this->enabled = $enabled;
 		$enabled = ($enabled) ? 'true' : 'false';
 		$this->config->setUserValue($this->uid, 'core', 'enabled', $enabled);
+		if ($this->emitter) {
+			$this->emitter->emit('\OC\User', 'postSetEnabled', [$this, $enabled]);
+		}
 	}
 
 	/**

diff --git a/tests/lib/User/UserTest.php b/tests/lib/User/UserTest.php
@@ -502,6 +502,35 @@ public function testDeleteHooks() {
 		$this->assertEquals(2, $hooksCalled);
 	}
 
+	public function testSetEnabledHook(){
+		$hooksCalled = 0;
+		$test = $this;
+
+		/**
+		 * @var Backend | \PHPUnit_Framework_MockObject_MockObject $backend
+		 */
+		$backend = $this->createMock(Dummy::class);
+
+		/**
+		 * @param User $user
+		 * @param bool $enabled
+		 */
+		$hook = function ($user, $enabled) use ($test, &$hooksCalled) {
+			$hooksCalled++;
+			$expectedState = ($user->isEnabled()) ? 'true' : 'false';
+			$test->assertEquals($expectedState, $enabled);
+			$test->assertEquals('foo', $user->getUID());
+		};
+
+		$emitter = new PublicEmitter();
+		$emitter->listen('\OC\User', 'postSetEnabled', $hook);
+
+		$user = new User('foo', $backend, $emitter);
+		$user->setEnabled(true);
+		$user->setEnabled(false);
+		$this->assertEquals(2, $hooksCalled);
+	}
+
 	public function testGetCloudId() {
 		/**
 		 * @var Backend | \PHPUnit_Framework_MockObject_MockObject $backend