Merge pull request #5 from gdubost/cr/133910

[CR#133910] Ajout du trait setProperties

Tests/Model.php

@@ -6,6 +6,7 @@
 use ETNA\Doctrine\Extensions\CreatedAt;
 use ETNA\Doctrine\Extensions\UpdatedAt;
 use ETNA\Doctrine\Extensions\DoNotDelete;
+use ETNA\Doctrine\Extensions\SetProperties;
 
 /**
  * @Entity(repositoryClass="Tests\ModelRepository")
@@ -18,6 +19,24 @@ class Model implements \JsonSerializable
     use CreatedAt;
     use UpdatedAt;
     use DoNotDelete;
+    use SetProperties;
+
+    private $model_value;
+
+    public function __construct()
+    {
+        $this->model_value = '';
+    }
+
+    public function setModelValue($value)
+    {
+        $this->model_value = $value;
+    }
+
+    public function getModelValue()
+    {
+        return $this->model_value;
+    }
 
     public function populate($date)
     {

Tests/ModelTest.php

@@ -117,4 +117,11 @@ public function testSetDeletedAt()
         // On récupère un objet
         $this->assertEquals($this->datetime, $this->model->getDeletedAt());
     }
+
+    public function testSetProperties()
+    {
+        $this->assertEquals($this->model->getModelValue(), '');
+        $this->model->setProperties(["model_value" => "super value"]);
+        $this->assertEquals($this->model->getModelValue(), "super value");
+    }
 }

phpmd.xml

@@ -6,7 +6,11 @@
          xsi:noNamespaceSchemaLocation=" http://pmd.sf.net/ruleset_xml_schema.xsd">
     <description> phpmd ruleset for doctrine extensions</description>
     <rule ref="rulesets/codesize.xml" />
-    <rule ref="rulesets/controversial.xml" />
+
+    <rule ref="rulesets/controversial.xml/Superglobals" />
+    <rule ref="rulesets/controversial.xml/CamelCaseClassName" />
+    <rule ref="rulesets/controversial.xml/CamelCaseMethodName" />
+
     <rule ref="rulesets/design.xml" />
     <!-- <rule ref="rulesets/naming.xml" /> -->
     <rule ref="rulesets/naming.xml/ShortVariable">

src/SetProperties.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace ETNA\Doctrine\Extensions;
+
+/**
+ * Ce trait permet aux classes concernées de setter ses propriétés depuis un array associatif
+ */
+trait SetProperties
+{
+    /**
+     * Sette toute les valeurs d'un array associatif à $this.
+     *
+     * @param  array $request_data
+     *
+     * @return self
+     */
+    public function setProperties(array $request_data)
+    {
+        foreach ($request_data as $field_name => $field_value) {
+            $setter_name = 'set' . str_replace(' ', '', ucwords(str_replace('_', ' ', $field_name)));
+            if (method_exists($this, $setter_name)) {
+                $this->{$setter_name}($field_value);
+            }
+        }
+
+        return $this;
+    }
+}