dynamic / silverstripe-manageable-dataobject

src/Form/ManageableDataObjectForm.php

Indentation +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
             $object = Injector::inst()->get($object);
         }
 
-		/** @var \SilverStripe\ORM\DataObject|\Dynamic\ManageableDataObject\Interfaces\ManageableDataObjectInterface $object */
+        /** @var \SilverStripe\ORM\DataObject|\Dynamic\ManageableDataObject\Interfaces\ManageableDataObjectInterface $object */
         $fields = $object->getFrontEndFields();
         $actions = $object->getFrontEndActions();
         if (!$actions->dataFieldByName('action_doSaveObject')) {

tests/Model/SampleManageableDataObject.php

Indentation +10 added lines, -10 removed lines

@@ -22,10 +22,10 @@ 
 class SampleManageableDataObject extends DataObject implements PermissionProvider, ManageableDataObjectInterface, TestOnly
 {
 
-	/**
-	 * @var string
-	 */
-	private static $table_name = 'SampleManageableDataObject';
+    /**
+     * @var string
+     */
+    private static $table_name = 'SampleManageableDataObject';
 
     /**
      * @var string
@@ -53,12 +53,12 @@ 
         ];
     }
 
-	/**
-	 * @param null $member
-	 * @param array $context
-	 *
-	 * @return bool|int
-	 */
+    /**
+     * @param null $member
+     * @param array $context
+     *
+     * @return bool|int
+     */
     public function canCreate($member = null, $context = array())
     {
         return Permission::check('MDO_Create', 'any', $member);

tests/Model/SampleManageableObjectPage.php

Indentation +4 added lines, -4 removed lines

@@ -10,8 +10,8 @@
  */
 class SampleManageableObjectPage extends \Page implements TestOnly
 {
-	/**
-	 * @var string
-	 */
-	private static $table_name = 'SampleManageableObjectPage';
+    /**
+     * @var string
+     */
+    private static $table_name = 'SampleManageableObjectPage';
 }

tests/Extensions/ManageableObjectDataExtensionTest.php

Indentation +20 added lines, -20 removed lines

@@ -23,20 +23,20 @@ 
     /**
      * @var array
      */
-	protected static $extra_dataobjects = [
+    protected static $extra_dataobjects = [
         SampleManageableDataObject::class,
         SampleManageableObjectPage::class,
     ];
 
-	/**
-	 * Ensure any current member is logged out
-	 */
-	public function logOut()
-	{
-		if ($member = Security::getCurrentUser()) {
-			Security::setCurrentUser(null);
-		}
-	}
+    /**
+     * Ensure any current member is logged out
+     */
+    public function logOut()
+    {
+        if ($member = Security::getCurrentUser()) {
+            Security::setCurrentUser(null);
+        }
+    }
 
     /**
      *
@@ -52,10 +52,10 @@ 
      */
     public function testGetAddLink()
     {
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-		/** @var SampleManageableDataObject $object */
-		$object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+        /** @var SampleManageableDataObject $object */
+        $object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
 
         $this->logOut();
         $this->assertFalse($object->getAddLink());
@@ -72,10 +72,10 @@ 
      */
     public function testGetEditLink()
     {
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-		/** @var SampleManageableDataObject $object */
-		$object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+        /** @var SampleManageableDataObject $object */
+        $object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
 
         $this->logOut();
         $this->assertFalse($object->getEditLink());
@@ -92,9 +92,9 @@ 
      */
     public function testGetDeleteLink()
     {
-    	/** @var SampleManageableObjectPage $page */
+        /** @var SampleManageableObjectPage $page */
         $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-		/** @var SampleManageableDataObject $object */
+        /** @var SampleManageableDataObject $object */
         $object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
 
         $this->logOut();

tests/Extensions/ManageableDataObjectExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
 		$this->assertEquals(200, $response->getStatusCode());
 		$this->logOut();
 
-		$response2 = $this->get('/SampleManageableObjectPageController/edit/' . 0);
+		$response2 = $this->get('/SampleManageableObjectPageController/edit/'.0);
 		$this->assertEquals(404, $response2->getStatusCode());
 	}
 

Indentation +135 added lines, -135 removed lines

@@ -16,139 +16,139 @@
 class ManageableDataObjectExtensionTest extends FunctionalTest
 {
 
-	/**
-	 * @var string
-	 */
-	protected static $fixture_file = '../fixtures.yml';
-
-	/**
-	 * @var array
-	 */
-	protected static $extra_dataobjects = [
-		SampleManageableDataObject::class,
-		SampleManageableObjectPage::class,
-	];
-
-	/**
-	 *
-	 */
-	public function setUp()
-	{
-		parent::setUp();
-		// Suppress themes
-		SSViewer::config()->update('theme_enabled', false);
-		$this->session()->set('readingMode', 'Stage.Stage');
-		$this->session()->set('unsecuredDraftSite', true);
-	}
-
-	/**
-	 * Ensure any current member is logged out
-	 */
-	public function logOut()
-	{
-		if ($member = Security::getCurrentUser()) {
-			Security::setCurrentUser(null);
-		}
-	}
-
-	/**
-	 *
-	 */
-	public function testAdd()
-	{
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-
-		$this->logInWithPermission('MDO_Create');
-		$response = $this->get($page->Link('add'));
-		$this->assertEquals(200, $response->getStatusCode());
-		$this->logOut();
-	}
-
-	/**
-	 *
-	 */
-	public function testEdit()
-	{
-		/** @var SampleManageableDataObject $object */
-		$object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-
-		$this->logInWithPermission('MDO_Edit');
-		$response = $this->get(Controller::join_links(
-			$page->Link(),
-			'edit',
-			$object->ID
-		));
-		$this->assertEquals(200, $response->getStatusCode());
-		$this->logOut();
-
-		$response2 = $this->get('/SampleManageableObjectPageController/edit/' . 0);
-		$this->assertEquals(404, $response2->getStatusCode());
-	}
-
-	/**
-	 *
-	 */
-	public function testDelete()
-	{
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-
-		$newObject = SampleManageableDataObject::create();
-		$newObject->Title = 'Foo';
-		$newObject->write();
-
-		$id = $newObject->ID;
-
-		$this->logInWithPermission('MDO_Delete');
-
-		$response404 = $this->get(Controller::join_links(
-			$page->Link(),
-			'delete',
-			0
-		));
-		$this->assertEquals(404, $response404->getStatusCode());
-
-		$success = $this->get(Controller::join_links(
-			$page->Link(),
-			'delete',
-			$id
-		));
-		$this->assertEquals(200, $success->getStatusCode());
-		$this->assertNull(SampleManageableDataObject::get()->byID($id));
-
-		$this->logOut();
-	}
-
-	/**
-	 *
-	 */
-	public function testDoSaveObject()
-	{
-		/** @var SampleManageableObjectPage $page */
-		$page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
-
-		$this->logInWithPermission('MDO_Create');
-
-		$controller = SampleManageableObjectPageController::create(
-			$this->objFromFixture(SampleManageableObjectPage::class, 'one')
-		);
-		$form = $controller->Form();
-
-		$response = $this->get($page->Link('add'));
-		$this->assertEquals(200, $response->getStatusCode(), 'Submission successful');
-
-		$data = ['Title' => 'Foobar'];
-		$responseSubmission = $this->submitForm(
-			'ManageableDataObjectForm_Form',
-			'action_doSaveObject',
-			$data
-		);
-
-		$record = SampleManageableDataObject::get()->filter($data)->first();
-		$this->assertInstanceOf(SampleManageableDataObject::class, $record);
-		$this->assertTrue($record->exists());
-	}
+    /**
+     * @var string
+     */
+    protected static $fixture_file = '../fixtures.yml';
+
+    /**
+     * @var array
+     */
+    protected static $extra_dataobjects = [
+        SampleManageableDataObject::class,
+        SampleManageableObjectPage::class,
+    ];
+
+    /**
+     *
+     */
+    public function setUp()
+    {
+        parent::setUp();
+        // Suppress themes
+        SSViewer::config()->update('theme_enabled', false);
+        $this->session()->set('readingMode', 'Stage.Stage');
+        $this->session()->set('unsecuredDraftSite', true);
+    }
+
+    /**
+     * Ensure any current member is logged out
+     */
+    public function logOut()
+    {
+        if ($member = Security::getCurrentUser()) {
+            Security::setCurrentUser(null);
+        }
+    }
+
+    /**
+     *
+     */
+    public function testAdd()
+    {
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+
+        $this->logInWithPermission('MDO_Create');
+        $response = $this->get($page->Link('add'));
+        $this->assertEquals(200, $response->getStatusCode());
+        $this->logOut();
+    }
+
+    /**
+     *
+     */
+    public function testEdit()
+    {
+        /** @var SampleManageableDataObject $object */
+        $object = $this->objFromFixture(SampleManageableDataObject::class, 'one');
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+
+        $this->logInWithPermission('MDO_Edit');
+        $response = $this->get(Controller::join_links(
+            $page->Link(),
+            'edit',
+            $object->ID
+        ));
+        $this->assertEquals(200, $response->getStatusCode());
+        $this->logOut();
+
+        $response2 = $this->get('/SampleManageableObjectPageController/edit/' . 0);
+        $this->assertEquals(404, $response2->getStatusCode());
+    }
+
+    /**
+     *
+     */
+    public function testDelete()
+    {
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+
+        $newObject = SampleManageableDataObject::create();
+        $newObject->Title = 'Foo';
+        $newObject->write();
+
+        $id = $newObject->ID;
+
+        $this->logInWithPermission('MDO_Delete');
+
+        $response404 = $this->get(Controller::join_links(
+            $page->Link(),
+            'delete',
+            0
+        ));
+        $this->assertEquals(404, $response404->getStatusCode());
+
+        $success = $this->get(Controller::join_links(
+            $page->Link(),
+            'delete',
+            $id
+        ));
+        $this->assertEquals(200, $success->getStatusCode());
+        $this->assertNull(SampleManageableDataObject::get()->byID($id));
+
+        $this->logOut();
+    }
+
+    /**
+     *
+     */
+    public function testDoSaveObject()
+    {
+        /** @var SampleManageableObjectPage $page */
+        $page = $this->objFromFixture(SampleManageableObjectPage::class, 'one');
+
+        $this->logInWithPermission('MDO_Create');
+
+        $controller = SampleManageableObjectPageController::create(
+            $this->objFromFixture(SampleManageableObjectPage::class, 'one')
+        );
+        $form = $controller->Form();
+
+        $response = $this->get($page->Link('add'));
+        $this->assertEquals(200, $response->getStatusCode(), 'Submission successful');
+
+        $data = ['Title' => 'Foobar'];
+        $responseSubmission = $this->submitForm(
+            'ManageableDataObjectForm_Form',
+            'action_doSaveObject',
+            $data
+        );
+
+        $record = SampleManageableDataObject::get()->filter($data)->first();
+        $this->assertInstanceOf(SampleManageableDataObject::class, $record);
+        $this->assertTrue($record->exists());
+    }
 }

src/Extensions/ManageableControllerExtension.php

Indentation +20 added lines, -20 removed lines

@@ -52,9 +52,9 @@ 
         }
 
         return Security::permissionFailure(
-        	$this->owner,
-			"You don't have permission to add records."
-		);
+            $this->owner,
+            "You don't have permission to add records."
+        );
     }
 
     /**
@@ -78,9 +78,9 @@ 
             }
 
             return Security::permissionFailure(
-            	$this->owner,
-				"You don't have permission to edit this record."
-			);
+                $this->owner,
+                "You don't have permission to edit this record."
+            );
         }
 
         return $this->owner->httpError(404);
@@ -105,9 +105,9 @@ 
             }
 
             return Security::permissionFailure(
-            	$this->owner,
-				"You don't have permission to delete this record."
-			);
+                $this->owner,
+                "You don't have permission to delete this record."
+            );
         }
 
         return $this->owner->httpError(404);
@@ -144,18 +144,18 @@ 
         return $form;
     }
 
-	/**
-	 * Save object
-	 *
-	 * @param $data
-	 * @param Form $form
-	 *
-	 * @return \SilverStripe\Control\HTTPResponse
-	 * @throws \SilverStripe\ORM\ValidationException
-	 */
+    /**
+     * Save object
+     *
+     * @param $data
+     * @param Form $form
+     *
+     * @return \SilverStripe\Control\HTTPResponse
+     * @throws \SilverStripe\ORM\ValidationException
+     */
     public function doSaveObject($data, Form $form)
     {
-		/** @var \SilverStripe\ORM\DataObject $model */
+        /** @var \SilverStripe\ORM\DataObject $model */
         $model = $this->owner->config()->get('managed_object');
 
         /** @var \SilverStripe\ORM\DataObject|\Dynamic\ViewableDataObject\Extensions\ViewableDataObject $object */
@@ -168,7 +168,7 @@ 
             $object = $model::create();
             if ($object->hasDatabaseField('URLSegment')) {
                 $object->URLSegment = Injector::inst()->create(SiteTree::class)
-					->generateURLSegment($data['Title']);
+                    ->generateURLSegment($data['Title']);
             }
             // write on create to relations are saved on final write (needs ID)
             $object->write();

Spacing +3 added lines, -3 removed lines

@@ -39,14 +39,14 @@ 
         if ($object->canCreate(Security::getCurrentUser())) {
 
             $form = $this->ManageableDataObjectForm();
-            if($object->config()->get('add_form_cancel_button')){
+            if ($object->config()->get('add_form_cancel_button')) {
                 $form->Actions()->push(new CancelFormAction($this->owner->Link(), 'Cancel'));
             }
 
             return $this->owner->customise([
                 'Title' => ($this->owner->config()->get('add_item_title'))
                     ? $this->owner->config()->get('add_item_title')
-                    : 'Add new ' . $object->singular_name(),
+                    : 'Add new '.$object->singular_name(),
                 'ManageableDataObjectForm' => $form,
             ]);
         }
@@ -71,7 +71,7 @@ 
                 $form = $this->ManageableDataObjectForm($item);
 
                 return $this->owner->customise([
-                    'Title' => 'Edit ' . $item->singular_name(),
+                    'Title' => 'Edit '.$item->singular_name(),
                     'ManageableDataObjectForm' => $form,
                     'Item' => $item,
                 ]);

tests/Model/SampleManageableObjectPageController.php

Indentation +32 added lines, -32 removed lines

@@ -16,36 +16,36 @@
 class SampleManageableObjectPageController extends \PageController implements TestOnly
 {
 
-	/**
-	 * @var array
-	 */
-	private static $allowed_actions = [
-		'Form',
-	];
-
-	/**
-	 * @var array
-	 */
-	private static $extensions = [
-		ManageableControllerExtension::class
-	];
-
-	/**
-	 * @var string
-	 */
-	private static $managed_object = SampleManageableDataObject::class;
-
-	/**
-	 *
-	 */
-	public function init()
-	{
-		parent::init();
-		Requirements::clear();
-	}
-
-	public function Form()
-	{
-		return ManageableDataObjectForm::create($this, 'Form', $this->config()->get('managed_object'));
-	}
+    /**
+     * @var array
+     */
+    private static $allowed_actions = [
+        'Form',
+    ];
+
+    /**
+     * @var array
+     */
+    private static $extensions = [
+        ManageableControllerExtension::class
+    ];
+
+    /**
+     * @var string
+     */
+    private static $managed_object = SampleManageableDataObject::class;
+
+    /**
+     *
+     */
+    public function init()
+    {
+        parent::init();
+        Requirements::clear();
+    }
+
+    public function Form()
+    {
+        return ManageableDataObjectForm::create($this, 'Form', $this->config()->get('managed_object'));
+    }
 }