From 303b4e0ee2c1de564d0024c815d2e0d72b4a5e97 Mon Sep 17 00:00:00 2001
From: Niels Nijens <nijens.niels@gmail.com>
Date: Fri, 3 Feb 2017 15:40:04 +0100
Subject: [PATCH 1/2] Add tests for UploadTrait::setFileUpload

---
 Tests/Model/UploadTraitTest.php | 64 +++++++++++++++++++++++++++++++++
 Tests/UploadEntityMock.php      | 18 ++++++++++
 Tests/UploadEntityProxyMock.php | 23 ++++++++++++
 3 files changed, 105 insertions(+)
 create mode 100644 Tests/Model/UploadTraitTest.php
 create mode 100644 Tests/UploadEntityMock.php
 create mode 100644 Tests/UploadEntityProxyMock.php

diff --git a/Tests/Model/UploadTraitTest.php b/Tests/Model/UploadTraitTest.php
new file mode 100644
index 0000000..4b4067c
--- /dev/null
+++ b/Tests/Model/UploadTraitTest.php
@@ -0,0 +1,64 @@
+<?php
+
+namespace FlexModel\FlexModelBundle\Tests\Model;
+
+use FlexModel\FlexModelBundle\Tests\UploadEntityMock;
+use FlexModel\FlexModelBundle\Tests\UploadEntityProxyMock;
+use PHPUnit_Framework_TestCase;
+use Symfony\Component\HttpFoundation\File\UploadedFile;
+
+/**
+ * UploadTraitTest.
+ *
+ * @author Niels Nijens <niels@connectholland.nl>
+ */
+class UploadTraitTest extends PHPUnit_Framework_TestCase
+{
+    /**
+     * Tests if the UploadEntityMock::setImageUpload (alias of UploadTrait::setFileUpload)
+     * sets the expected file uploads property.
+     */
+    public function testSetFileUpload()
+    {
+        $uploadedFileMock = $this->getMockBuilder(UploadedFile::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $entityMock = new UploadEntityMock();
+        $entityMock->setImageUpload($uploadedFileMock);
+
+        $this->assertAttributeSame(
+            array(
+                'image' => $uploadedFileMock,
+            ),
+            'fileUploads',
+            $entityMock
+        );
+    }
+
+    /**
+     * Tests if the UploadEntityProxyMock::setImageUpload (alias of UploadTrait::setFileUpload)
+     * sets the expected file uploads property.
+     *
+     * This tests the scenario of a Doctrine entity being a parent class of
+     * a proxy class with all the method overloaded as this changes the
+     * PHP stack to determine the caller method.
+     */
+    public function testSetFileUploadFromProxy()
+    {
+        $uploadedFileMock = $this->getMockBuilder(UploadedFile::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $uploadEntityProxyMock = new UploadEntityProxyMock();
+        $uploadEntityProxyMock->setImageUpload($uploadedFileMock);
+
+        $this->assertAttributeSame(
+            array(
+                'image' => $uploadedFileMock,
+            ),
+            'fileUploads',
+            $uploadEntityProxyMock
+        );
+    }
+}
diff --git a/Tests/UploadEntityMock.php b/Tests/UploadEntityMock.php
new file mode 100644
index 0000000..425f15d
--- /dev/null
+++ b/Tests/UploadEntityMock.php
@@ -0,0 +1,18 @@
+<?php
+
+namespace FlexModel\FlexModelBundle\Tests;
+
+use FlexModel\FlexModelBundle\Model\UploadTrait;
+
+/**
+ * Mock class for testing the UploadTrait.
+ *
+ * @author Niels Nijens <niels@connectholland.nl>
+ */
+class UploadEntityMock
+{
+    use UploadTrait {
+        getFileUpload as getImageUpload;
+        setFileUpload as setImageUpload;
+    }
+}
diff --git a/Tests/UploadEntityProxyMock.php b/Tests/UploadEntityProxyMock.php
new file mode 100644
index 0000000..bef5dbe
--- /dev/null
+++ b/Tests/UploadEntityProxyMock.php
@@ -0,0 +1,23 @@
+<?php
+
+namespace FlexModel\FlexModelBundle\Tests;
+
+use Symfony\Component\HttpFoundation\File\UploadedFile;
+
+/**
+ * Mock class mimicking the behavior of a Doctrine proxy class.
+ *
+ * @author Niels Nijens <niels@connectholland.nl>
+ */
+class UploadEntityProxyMock extends UploadEntityMock
+{
+    /**
+     * Overloaded trait method alias.
+     *
+     * @param UploadedFile $file
+     */
+    public function setImageUpload(UploadedFile $file = null)
+    {
+        parent::setImageUpload($file);
+    }
+}

From b791192e3acade68f3060e22e5b39ec205bc8006 Mon Sep 17 00:00:00 2001
From: Niels Nijens <nijens.niels@gmail.com>
Date: Fri, 3 Feb 2017 15:41:09 +0100
Subject: [PATCH 2/2] Fix UploadTrait::getFileUploadPropertyName for Doctrine
 proxy entity

---
 Model/UploadTrait.php | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/Model/UploadTrait.php b/Model/UploadTrait.php
index afd7422..45dc383 100644
--- a/Model/UploadTrait.php
+++ b/Model/UploadTrait.php
@@ -51,7 +51,7 @@ public function getFileUploads()
      */
     public function setFileUpload(UploadedFile $file = null)
     {
-        $propertyName = $this->getFileUploadPropertyName();
+        $propertyName = $this->getFileUploadPropertyName(__FUNCTION__);
 
         unset($this->fileUploads[$propertyName]);
         if ($file instanceof UploadedFile) {
@@ -74,8 +74,14 @@ public function setFileUploadPath($directory)
      *
      * @return string
      */
-    private function getFileUploadPropertyName()
+    private function getFileUploadPropertyName($realCallerMethod)
     {
-        return lcfirst(substr(debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS)[1]['function'], 3, -6));
+        $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 3);
+        $callerMethodName = $backtrace[1]['function'];
+        if ($callerMethodName === $realCallerMethod) {
+            $callerMethodName = $backtrace[2]['function'];
+        }
+
+        return lcfirst(substr($callerMethodName, 3, -6));
     }
 }