ntd / silverstripe-autotoc

src/Autotoc.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
      * (e.g. because of malformed content) no further attempts will be
      * made.
      *
-     * @param DataObject $owner
+     * @param \SilverStripe\ORM\DataObject $owner
      * @return Tocifier|false|null
      */
     private static function getTocifier($owner)

Spacing +6 added lines, -6 removed lines

@@ -89,7 +89,7 @@ 
             Hacks::addCallbackMethodToInstance(
                 $owner,
                 'getContent',
-                function () use ($owner) {
+                function() use ($owner) {
                     return $owner->getContentField();
                 }
             );
@@ -122,7 +122,7 @@ 
                 $callback = Config::inst()->get(self::class, 'augment_callback');
             }
             $tocifier->setAugmentCallback(explode('::', $callback));
-            if (! $tocifier->process()) {
+            if (!$tocifier->process()) {
                 $tocifier = false;
             }
             self::$tocifiers[$owner] = $tocifier;
@@ -150,7 +150,7 @@ 
     public function getAutotoc()
     {
         $tocifier = self::getTocifier($this->owner);
-        if (! $tocifier) {
+        if (!$tocifier) {
             return null;
         }
 
@@ -171,15 +171,15 @@ 
     public function getOriginalContentField()
     {
         $model = $this->owner->getCustomisedObj();
-        if (! $model) {
+        if (!$model) {
             $model = $this->owner->data();
         }
-        if (! $model) {
+        if (!$model) {
             return null;
         }
 
         $field = $this->contentField();
-        if (! $model->hasField($field)) {
+        if (!$model->hasField($field)) {
             return null;
         }
 

src/Hacks.php

Doc Comments +4 added lines

@@ -26,6 +26,10 @@
         ];
     }
 
+    /**
+     * @param string $method
+     * @param \Closure $callback
+     */
     public static function addCallbackMethodToInstance($instance, $method, $callback)
     {
         // hasMethod() trigger the population of $extra_methods

tests/AutotocTest.php

Spacing +13 added lines, -13 removed lines

@@ -28,8 +28,8 @@ 
     private function populatedTestObject()
     {
         $obj          = new TestObject;
-        $obj->Content = file_get_contents(__DIR__ . '/test1');
-        $obj->Test2   = file_get_contents(__DIR__ . '/test2');
+        $obj->Content = file_get_contents(__DIR__.'/test1');
+        $obj->Test2   = file_get_contents(__DIR__.'/test2');
         return $obj;
     }
 
@@ -66,8 +66,8 @@ 
         $toc = $obj->getAutotoc();
         $this->assertNull($toc);
 
-        $obj->Content = file_get_contents(__DIR__ . '/test1');
-        $obj->Test2   = file_get_contents(__DIR__ . '/test2');
+        $obj->Content = file_get_contents(__DIR__.'/test1');
+        $obj->Test2   = file_get_contents(__DIR__.'/test2');
 
         // Old TOC should still be cached
         $toc = $obj->getAutotoc();
@@ -78,9 +78,9 @@ 
         $toc = $obj->getAutotoc();
         $this->assertTrue($toc instanceof ArrayData);
         $this->assertEquals(5, $toc->Children->count());
-        $this->assertStringEqualsFile(__DIR__ . '/test1', $obj->OriginalContentField);
-        $this->assertStringEqualsFile(__DIR__ . '/html2', $obj->ContentField);
-        $this->assertStringEqualsFile(__DIR__ . '/html2', $obj->Content);
+        $this->assertStringEqualsFile(__DIR__.'/test1', $obj->OriginalContentField);
+        $this->assertStringEqualsFile(__DIR__.'/html2', $obj->ContentField);
+        $this->assertStringEqualsFile(__DIR__.'/html2', $obj->Content);
 
         // Change the content field
         $obj->config()->update('content_field', 'Test2');
@@ -88,15 +88,15 @@ 
 
         $toc = $obj->getAutotoc();
         $this->assertNull($toc);
-        $this->assertStringEqualsFile(__DIR__ . '/test2', $obj->OriginalContentField);
-        $this->assertStringEqualsFile(__DIR__ . '/test2', $obj->ContentField);
+        $this->assertStringEqualsFile(__DIR__.'/test2', $obj->OriginalContentField);
+        $this->assertStringEqualsFile(__DIR__.'/test2', $obj->ContentField);
     }
 
     public function testAugmentCallback()
     {
         $obj = new TestObject;
-        $obj->Content = file_get_contents(__DIR__ . '/test1');
-        $obj->Test2   = file_get_contents(__DIR__ . '/test2');
+        $obj->Content = file_get_contents(__DIR__.'/test1');
+        $obj->Test2   = file_get_contents(__DIR__.'/test2');
 
         // Change the augmenter at class level
         Config::inst()->update(
@@ -108,7 +108,7 @@ 
 
         $toc = $obj->getAutotoc();
         $this->assertEquals(5, $toc->Children->count());
-        $this->assertStringEqualsFile(__DIR__ . '/html1', $obj->Content);
+        $this->assertStringEqualsFile(__DIR__.'/html1', $obj->Content);
 
         // Change the augmenter at install level: should have higher
         // precedence
@@ -120,6 +120,6 @@ 
 
         $toc = $obj->getAutotoc();
         $this->assertEquals(5, $toc->Children->count());
-        $this->assertStringEqualsFile(__DIR__ . '/html2', $obj->Content);
+        $this->assertStringEqualsFile(__DIR__.'/html2', $obj->Content);
     }
 }

tests/TocifierTest.php

Spacing +8 added lines, -8 removed lines

@@ -27,27 +27,27 @@ 
 
     public function testPrependAnchor()
     {
-        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
+        $tocifier = new Tocifier(file_get_contents(__DIR__.'/test1'));
         $this->assertEquals('', $tocifier->getHtml());
 
         $tocifier->setAugmentCallback(array('\eNTiDi\Autotoc\Tocifier', 'prependAnchor'));
         $this->assertTrue($tocifier->process());
-        $this->assertStringEqualsFile(__DIR__ . '/html1', $tocifier->getHtml());
+        $this->assertStringEqualsFile(__DIR__.'/html1', $tocifier->getHtml());
     }
 
     public function testSetId()
     {
-        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
+        $tocifier = new Tocifier(file_get_contents(__DIR__.'/test1'));
         $this->assertEquals('', $tocifier->getHtml());
 
         // The default augmenting method should already be setId
         $this->assertTrue($tocifier->process());
-        $this->assertStringEqualsFile(__DIR__ . '/html2', $tocifier->getHtml());
+        $this->assertStringEqualsFile(__DIR__.'/html2', $tocifier->getHtml());
     }
 
     public function testTOC()
     {
-        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test1'));
+        $tocifier = new Tocifier(file_get_contents(__DIR__.'/test1'));
         $this->assertEquals(array(), $tocifier->getTOC());
         $this->assertTrue($tocifier->process());
         $this->assertNotNull($tocifier->getTOC());
@@ -55,17 +55,17 @@ 
         ob_start();
         $tocifier->dumpTOC();
         $returned = ob_get_clean();
-        $this->assertStringEqualsFile(__DIR__ . '/toc1', $returned);
+        $this->assertStringEqualsFile(__DIR__.'/toc1', $returned);
     }
 
     public function testDataHideFromTOC()
     {
-        $tocifier = new Tocifier(file_get_contents(__DIR__ . '/test2'));
+        $tocifier = new Tocifier(file_get_contents(__DIR__.'/test2'));
         $this->assertEquals('', $tocifier->getHtml());
         $this->assertTrue($tocifier->process());
 
         // Check the augmented HTML is equal to the original one
-        $this->assertStringEqualsFile(__DIR__ . '/test2', $tocifier->getHtml());
+        $this->assertStringEqualsFile(__DIR__.'/test2', $tocifier->getHtml());
 
         ob_start();
         $tocifier->dumpTOC();