It makes no sense to access properties through methods, as we need to…

… work with the raw values.
reprovinci · Jun 26, 2013 · efc29b5 · efc29b5
1 parent 0f8bb7d
commit efc29b5
Showing 1 changed file with 4 additions and 15 deletions.
diff --git a/Subscribers/DoctrineEncryptSubscriber.php b/Subscribers/DoctrineEncryptSubscriber.php
@@ -145,21 +145,10 @@ private function processFields($entity, $isEncryptOperation = true)
             if ($this->annReader->getPropertyAnnotation($refProperty, self::ENCRYPTED_ANN_NAME)) {
                 $withAnnotation = true;
                 // we have annotation and if it decrypt operation, we must avoid duble decryption
-                $propName = $refProperty->getName();
-                if ($refProperty->isPublic()) {
-                    $entity->$propName = $this->encryptor->$encryptorMethod($refProperty->getValue());
-                } else {
-                    $methodName = self::capitalize($propName);
-                    if ($reflectionClass->hasMethod($getter = 'get' . $methodName) && $reflectionClass->hasMethod($setter = 'set' . $methodName)) {
-                        $currentPropValue = $this->encryptor->$encryptorMethod($entity->$getter());
-                        $entity->$setter($currentPropValue);
-                    } else {
-                        $refProperty->setAccessible(true);
-                        $value = $refProperty->getValue($entity);
-                        $value = $this->encryptor->$encryptorMethod($value);
-                        $refProperty->setValue($entity, $value);
-                    }
-                }
+                $refProperty->setAccessible(true);
+                $value = $refProperty->getValue($entity);
+                $value = $this->encryptor->$encryptorMethod($value);
+                $refProperty->setValue($entity, $value);
             }
         }