enigmatix / yii2-confirmation

ConfirmationBehavior.php

Spacing +29 added lines, -29 removed lines

@@ -90,7 +90,7 @@ 
      * @param $event Event;
      */
 
-    public function beforeSave($event){
+    public function beforeSave($event) {
         $this->protectAttributes();
     }
 
@@ -109,7 +109,7 @@ 
             if ($this->skipOnNull && $value === null || $attribute == $this->timestampAttribute)
                 continue;
 
-            if(!$this->isAuthorised($user, $attribute, $value)){
+            if (!$this->isAuthorised($user, $attribute, $value)) {
                 $this->createConfirmationRequest();
                 $this->resetAttribute($attribute);
             }
@@ -124,17 +124,17 @@ 
      *
      * @return bool
      */
-    protected function isAuthorised($user, $attribute, $value){
+    protected function isAuthorised($user, $attribute, $value) {
 
         //Check for pre-defined administration roles
         if ($this->userIsAuthorised($user))
             return true;
 
         //Check for valid release token , eg that the token exists and is for the same record as this
-        if($this->releaseToken != null){
-            $confirmation =  ConfirmationRequest::findOne(['release_token' => $this->releaseToken]);
+        if ($this->releaseToken != null) {
+            $confirmation = ConfirmationRequest::findOne(['release_token' => $this->releaseToken]);
 
-            if($confirmation == null)
+            if ($confirmation == null)
                 return false;
 
             $model = $confirmation->constructObject();
@@ -145,7 +145,7 @@ 
 
         //Check to see if any protected attributes have been altered
         foreach ($this->protectedAttributes as $attribute)
-            if($this->hasChanged($attribute))
+            if ($this->hasChanged($attribute))
                 return false;
 
         return true;
@@ -157,9 +157,9 @@ 
      *
      * @return bool
      */
-    protected function userIsAuthorised($user){
+    protected function userIsAuthorised($user) {
         foreach ($this->allow as $role)
-            if($user->can($role))
+            if ($user->can($role))
                 return true;
 
         return false;
@@ -168,7 +168,7 @@ 
     /**
      * Business logic handling the creation of the Confirmation Request, and sending the second factor message.
      */
-    protected function createConfirmationRequest(){
+    protected function createConfirmationRequest() {
 
         $model         = $this->owner;
         $changedValues = $this->getChangedValues();
@@ -193,7 +193,7 @@ 
      *
      * @return bool
      */
-    protected function hasChanged($attribute){
+    protected function hasChanged($attribute) {
         return $this->owner->oldAttributes[$attribute] != $this->owner->{$attribute};
     }
 
@@ -201,11 +201,11 @@ 
      * Fetches all values which have changed, expect for the timestamp attribute.
      * @return array
      */
-    public function getChangedValues(){
+    public function getChangedValues() {
         $changedAttributes = [];
 
         foreach ($this->owner->attributes() as $attribute)
-            if($this->hasChanged($attribute))
+            if ($this->hasChanged($attribute))
                 $changedAttributes[$attribute] = $this->owner->$attribute;
 
         unset($changedAttributes[$this->timestampAttribute]);
@@ -217,7 +217,7 @@ 
      * Sets an attribute back to it's original value when it was fetched.
      * @param string $attribute
      */
-    protected function resetAttribute($attribute){
+    protected function resetAttribute($attribute) {
         $this->owner->$attribute = $this->owner->oldAttributes[$attribute];
     }
 
@@ -225,7 +225,7 @@ 
      * Adds a flash message to the interface stating the change has been held over pending confirmation.
      * @param $model
      */
-    public function createFeedbackMessage($model){
+    public function createFeedbackMessage($model) {
         $this->displayMessage($model);
     }
 
@@ -233,7 +233,7 @@ 
      * Business logic around displaying an appropriate feedback message to the user regbarding the change.
      * @param $model
      */
-    protected function displayMessage($model){
+    protected function displayMessage($model) {
         Yii::$app->session->setFlash('warning', 'Your update is pending confirmation.  Please check your email for a confirmation link.');
     }
 
@@ -241,8 +241,8 @@ 
      * Business logic around transmitting the second factor message.
      * @param ConfirmationRequest $model
      */
-    public function sendSecondFactorMessage($model){
-        switch ($this->secondFactor){
+    public function sendSecondFactorMessage($model) {
+        switch ($this->secondFactor) {
             case 'email':
                 Yii::$app->mailer
                     ->compose($this->confirmationViewPath, ['model' => $model])
@@ -265,29 +265,29 @@ 
      * @return string
      * @throws InvalidCallException
      */
-    protected function getEmail($model){
+    protected function getEmail($model) {
 
         $values = unserialize($model->values);
-        $email  = ArrayHelper::getValue($values,'email');
+        $email  = ArrayHelper::getValue($values, 'email');
         $object = $model->constructObject();
 
-        if($email == null){
-            $email = ArrayHelper::getValue($values,'email_address');
+        if ($email == null) {
+            $email = ArrayHelper::getValue($values, 'email_address');
         }
 
-        if($email == null){
-            $email = ArrayHelper::getValue($object,'email');
+        if ($email == null) {
+            $email = ArrayHelper::getValue($object, 'email');
         }
 
-        if($email == null){
-            $email = ArrayHelper::getValue($object,'email_address');
+        if ($email == null) {
+            $email = ArrayHelper::getValue($object, 'email_address');
         }
 
-        if($email == null){
-            $email = ArrayHelper::getValue($object,$this->createdByAttribute.'.email');
+        if ($email == null) {
+            $email = ArrayHelper::getValue($object, $this->createdByAttribute . '.email');
         }
 
-        if($email == null){
+        if ($email == null) {
             throw new InvalidCallException('Unable to locate email address via record, changed values, or user account');
         }
 

Braces +18 added lines, -12 removed lines

@@ -106,8 +106,9 @@ 
 
         foreach ($changedValues as $attribute => $value) {
 
-            if ($this->skipOnNull && $value === null || $attribute == $this->timestampAttribute)
-                continue;
+            if ($this->skipOnNull && $value === null || $attribute == $this->timestampAttribute) {
+                            continue;
+            }
 
             if(!$this->isAuthorised($user, $attribute, $value)){
                 $this->createConfirmationRequest();
@@ -127,15 +128,17 @@ 
     protected function isAuthorised($user, $attribute, $value){
 
         //Check for pre-defined administration roles
-        if ($this->userIsAuthorised($user))
-            return true;
+        if ($this->userIsAuthorised($user)) {
+                    return true;
+        }
 
         //Check for valid release token , eg that the token exists and is for the same record as this
         if($this->releaseToken != null){
             $confirmation =  ConfirmationRequest::findOne(['release_token' => $this->releaseToken]);
 
-            if($confirmation == null)
-                return false;
+            if($confirmation == null) {
+                            return false;
+            }
 
             $model = $confirmation->constructObject();
 
@@ -144,9 +147,10 @@ 
         }
 
         //Check to see if any protected attributes have been altered
-        foreach ($this->protectedAttributes as $attribute)
-            if($this->hasChanged($attribute))
+        foreach ($this->protectedAttributes as $attribute) {
+                    if($this->hasChanged($attribute))
                 return false;
+        }
 
         return true;
     }
@@ -158,9 +162,10 @@ 
      * @return bool
      */
     protected function userIsAuthorised($user){
-        foreach ($this->allow as $role)
-            if($user->can($role))
+        foreach ($this->allow as $role) {
+                    if($user->can($role))
                 return true;
+        }
 
         return false;
     }
@@ -204,9 +209,10 @@ 
     public function getChangedValues(){
         $changedAttributes = [];
 
-        foreach ($this->owner->attributes() as $attribute)
-            if($this->hasChanged($attribute))
+        foreach ($this->owner->attributes() as $attribute) {
+                    if($this->hasChanged($attribute))
                 $changedAttributes[$attribute] = $this->owner->$attribute;
+        }
 
         unset($changedAttributes[$this->timestampAttribute]);
 

ConfirmationRequest.php

Spacing +8 added lines, -8 removed lines

@@ -51,7 +51,7 @@ 
             [['object_id'], 'integer'],
             [['object', 'values'], 'string'],
             [['model', 'release_token'], 'string', 'max' => 255],
-            [['release_token'], 'default',  'value' => function ($model, $attribute){ return $this->generateReleaseToken();}],
+            [['release_token'], 'default', 'value' => function($model, $attribute) { return $this->generateReleaseToken(); }],
             [['created_by'], 'exist', 'skipOnError' => true, 'targetClass' => $this->getUserClassName(), 'targetAttribute' => ['created_by' => 'id']],
             [['updated_by'], 'exist', 'skipOnError' => true, 'targetClass' => $this->getUserClassName(), 'targetAttribute' => ['updated_by' => 'id']],
         ];
@@ -97,8 +97,8 @@ 
             ]);
     }
 
-    public function getViewLink(){
-        return Url::to(['@web/confirmation-requests', 'release_token' => $this->release_token],true);
+    public function getViewLink() {
+        return Url::to(['@web/confirmation-requests', 'release_token' => $this->release_token], true);
     }
 
     /**
@@ -129,20 +129,20 @@ 
     /**
      * @return string ActiveRecord user class, as per application implementation
      */
-    protected function getUserClassName(){
+    protected function getUserClassName() {
         return Yii::$app->user->identityClass;
     }
 
-    public function release(){
+    public function release() {
 
         $model          = $this->constructObject();
         $changedValues  = $model->getChangedValues();
         $current        = clone $model;
         $current->refresh();
 
-        foreach ($changedValues as $field => $value){
+        foreach ($changedValues as $field => $value) {
             $oldValue = $model->oldAttributes[$field];
-            if($current->$field !== $oldValue && $current->$field !== $value){
+            if ($current->$field !== $oldValue && $current->$field !== $value) {
                 throw new ErrorException(
                     sprintf('Unable to release change, protected field %s has been updated since this request.'
                     . ' Expected to find %s or %s, found %s', $field, $value, $oldValue, $current->$field));
@@ -159,7 +159,7 @@ 
     /**
      * @return \enigmatix\core\Model
      */
-    public function constructObject(){
+    public function constructObject() {
         return unserialize($this->object);
     }