silverstripe / silverstripe-documentconverter

tests/SettingsFieldTest.php

Indentation +49 added lines, -49 removed lines

@@ -14,61 +14,61 @@
 
 class SettingsFieldTest extends SapphireTest
 {
-    /**
-     * @expectedException InvalidArgumentException
-     */
-    public function testConstructorThrowsExceptionWhenGivenString()
-    {
-        new SettingsField('exception time!');
-    }
+	/**
+	 * @expectedException InvalidArgumentException
+	 */
+	public function testConstructorThrowsExceptionWhenGivenString()
+	{
+		new SettingsField('exception time!');
+	}
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
-    public function testConstructorThrowsExceptionWhenGivenChildren()
-    {
-        new SettingsField(['i', 'don\'t', 'like', 'kids']);
-    }
+	/**
+	 * @expectedException InvalidArgumentException
+	 */
+	public function testConstructorThrowsExceptionWhenGivenChildren()
+	{
+		new SettingsField(['i', 'don\'t', 'like', 'kids']);
+	}
 
-    public function testFieldAddsJavascriptRequirements()
-    {
-        // Start with a clean slate (no global state interference)
-        Requirements::backend()->clear();
+	public function testFieldAddsJavascriptRequirements()
+	{
+		// Start with a clean slate (no global state interference)
+		Requirements::backend()->clear();
 
-        new SettingsField();
-        $javascript = Requirements::backend()->getJavascript();
-        $this->assertNotEmpty($javascript);
-    }
+		new SettingsField();
+		$javascript = Requirements::backend()->getJavascript();
+		$this->assertNotEmpty($javascript);
+	}
 
-    public function testFieldListGeneration()
-    {
-        $importField = new SettingsField();
+	public function testFieldListGeneration()
+	{
+		$importField = new SettingsField();
 
-        $fields = $importField->getChildren();
-        $this->assertInstanceOf(FieldList::class, $fields);
+		$fields = $importField->getChildren();
+		$this->assertInstanceOf(FieldList::class, $fields);
 
-        // We don't need to check that all of the fields are there, but just check a couple
-        $this->assertInstanceOf(HeaderField::class, $fields->fieldByName('FileWarningHeader'));
-        $innerField = $fields->fieldByName('ImportedFromFile');
-        $this->assertInstanceOf(ImportField::class, $innerField);
+		// We don't need to check that all of the fields are there, but just check a couple
+		$this->assertInstanceOf(HeaderField::class, $fields->fieldByName('FileWarningHeader'));
+		$innerField = $fields->fieldByName('ImportedFromFile');
+		$this->assertInstanceOf(ImportField::class, $innerField);
 
-        // Check the getter works
-        $this->assertSame($innerField, $importField->getInnerField());
+		// Check the getter works
+		$this->assertSame($innerField, $importField->getInnerField());
 
-        // Check the fields have been given has the change tracker disabled
-        $settingsFields = [
-            'SplitHeader' => DropdownField::class,
-            'KeepSource' => CheckboxField::class,
-            'ChosenFolderID' => TreeDropdownField::class,
-            'IncludeTOC' => CheckboxField::class,
-            'PublishPages' => CheckboxField::class
-        ];
-        foreach ($settingsFields as $fieldName => $className) {
-            $field = $fields->fieldByName(
-                'DocumentConversionSettings-' . $fieldName
-            );
-            $this->assertInstanceOf($className, $field);
-            $this->assertContains('no-change-track', $field->extraClass());
-        }
-    }
+		// Check the fields have been given has the change tracker disabled
+		$settingsFields = [
+			'SplitHeader' => DropdownField::class,
+			'KeepSource' => CheckboxField::class,
+			'ChosenFolderID' => TreeDropdownField::class,
+			'IncludeTOC' => CheckboxField::class,
+			'PublishPages' => CheckboxField::class
+		];
+		foreach ($settingsFields as $fieldName => $className) {
+			$field = $fields->fieldByName(
+				'DocumentConversionSettings-' . $fieldName
+			);
+			$this->assertInstanceOf($className, $field);
+			$this->assertContains('no-change-track', $field->extraClass());
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         ];
         foreach ($settingsFields as $fieldName => $className) {
             $field = $fields->fieldByName(
-                'DocumentConversionSettings-' . $fieldName
+                'DocumentConversionSettings-'.$fieldName
             );
             $this->assertInstanceOf($className, $field);
             $this->assertContains('no-change-track', $field->extraClass());

tests/Stubs/MockDocvertService.php

Indentation +4 added lines, -4 removed lines

@@ -7,8 +7,8 @@
 
 class MockDocvertService extends ServiceConnector implements TestOnly
 {
-    public function import()
-    {
-        return '<h1>Fake document</h1><p>For testing purposes.</p>';
-    }
+	public function import()
+	{
+		return '<h1>Fake document</h1><p>For testing purposes.</p>';
+	}
 }

src/PageExtension.php

Indentation +12 added lines, -12 removed lines

@@ -9,13 +9,13 @@ 
 class PageExtension extends DataExtension
 {
 
-    private static $has_one = [
-        'ImportedFromFile' => File::class
-    ];
+	private static $has_one = [
+		'ImportedFromFile' => File::class
+	];
 
-    public function updateCMSFields(FieldList $fields)
-    {
-        /*
+	public function updateCMSFields(FieldList $fields)
+	{
+		/*
         // Currently the ToggleCompositeField plays badly with TreeDropdownField formatting.
         // Could be switched back in the future, if this is fixed.
         $fields->addFieldToTab('Root.Main', 
@@ -24,10 +24,10 @@ 
             ])->setHeadingLevel(4)
         );
          */
-        $fields->findOrMakeTab(
-            'Root.Import',
-            _t(__CLASS__ . '.ImportTab', 'Import')
-        );
-        $fields->addFieldToTab('Root.Import', SettingsField::create());
-    }
+		$fields->findOrMakeTab(
+			'Root.Import',
+			_t(__CLASS__ . '.ImportTab', 'Import')
+		);
+		$fields->addFieldToTab('Root.Import', SettingsField::create());
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
          */
         $fields->findOrMakeTab(
             'Root.Import',
-            _t(__CLASS__ . '.ImportTab', 'Import')
+            _t(__CLASS__.'.ImportTab', 'Import')
         );
         $fields->addFieldToTab('Root.Import', SettingsField::create());
     }