Fixes

Author: kvdnberg

lib/Gedmo/AbstractTrackingListener.php

@@ -106,8 +106,18 @@ public function onFlush(EventArgs $args)
                         $trackedFields = $options['trackedField'];
                     }
 
+//                    foreach ($trackedFields as $tracked) {
+//                        $trackedChild = null;
+//                        $parts = explode('.', $tracked);
+//                        if (isset($parts[1])) {
+//                            $tracked = $parts[0];
+//                            $trackedChild = $parts[1];
+//                        }
+
+
                     foreach ($trackedFields as $trackedField) {
                         $trackedChild = null;
+                        $tracked = null;
                         $parts = explode('.', $trackedField);
                         if (isset($parts[1])) {
                             $tracked = $parts[0];
@@ -116,7 +126,7 @@ public function onFlush(EventArgs $args)
 
                         if (!isset($tracked) || array_key_exists($trackedField, $changeSet)) {
                             $tracked = $trackedField;
-                            unset($trackedChild);
+                            $trackedChild = null;
                         }
 
                         if (isset($changeSet[$tracked])) {

tests/Gedmo/Timestampable/Fixture/Article.php

@@ -177,4 +177,9 @@ public function getContentChanged()
     {
         return $this->contentChanged;
     }
+
+    public function getAuthorChanged()
+    {
+        return $this->authorChanged;
+    }
 }

tests/Gedmo/Timestampable/Fixture/Author.php

@@ -11,12 +11,12 @@
 class Author
 {
     /**
-     * @ORM\Column(name="author_name", type="string", length=128)
+     * @ORM\Column(name="author_name", type="string", length=128, nullable=true)
      */
     private $name;
 
     /**
-     * @ORM\Column(name="author_email", type="string", length=50)
+     * @ORM\Column(name="author_email", type="string", length=50, nullable=true)
      */
     private $email;
 

tests/Gedmo/Timestampable/TimestampableTest.php

@@ -144,7 +144,7 @@ function shouldHandleStandardBehavior()
         $this->assertNotSame($su2 = $sport->getUpdated(), $su, "Date updated should change after update");
         $this->assertSame($sport->getPublished(), $sp, "Date published should remain the same after update");
         $this->assertNotSame($scc2 = $sport->getContentChanged(), $scc, "Content must have changed after update");
-        $this->assertSame($sport->authorChanged(), $sa, "Author should remain same after update");
+        $this->assertSame($sport->getAuthorChanged(), $sa, "Author should remain same after update");
 
         $author = $sport->getAuthor();
         $author->setName('Third author');