Fix style errors in download controller

Author: Jamie Snape

modules/demo/constant/module.php

@@ -27,10 +27,10 @@
 define('MIDAS_DEMO_USER_PASSWORD', 'user');
 
 define('MIDAS_DEMO_DYNAMIC_HELP', '
-  <b>To authenticate:</b><br /><br />
-  Demo Administrator<br />
-  - Login: '.MIDAS_DEMO_ADMIN_EMAIL.'<br />
-  - Password: '.MIDAS_DEMO_ADMIN_PASSWORD.'<br /><br />
-  Demo User<br />
-  - Login: '.MIDAS_DEMO_USER_EMAIL.'<br />
-  - Password: '.MIDAS_DEMO_USER_PASSWORD);
+    <b>To authenticate:</b><br /><br />
+    Demo Administrator<br />
+    - Login: '.MIDAS_DEMO_ADMIN_EMAIL.'<br />
+    - Password: '.MIDAS_DEMO_ADMIN_PASSWORD.'<br /><br />
+    Demo User<br />
+    - Login: '.MIDAS_DEMO_USER_EMAIL.'<br />
+    - Password: '.MIDAS_DEMO_USER_PASSWORD);

modules/keyfiles/tests/controllers/DownloadControllerTest.php

@@ -21,45 +21,45 @@
 /** Test keyfiles download controller. */
 class Keyfiles_DownloadControllerTest extends ControllerTestCase
 {
-  /** Set up tests. */
-  public function setUp()
-  {
-      $this->setupDatabase(array('default')); //core dataset
-    $this->enabledModules = array('keyfiles');
-      $this->_models = array('Bitstream', 'Item', 'User');
+    /** Set up tests. */
+    public function setUp()
+    {
+        $this->setupDatabase(array('default')); //core dataset
+        $this->enabledModules = array('keyfiles');
+        $this->_models = array('Bitstream', 'Item', 'User');
 
-      parent::setUp();
-  }
+        parent::setUp();
+    }
 
-  /** Test downloading of a single bitstream key file. */
-  public function testDownloadBitstreamKeyfile()
-  {
-      $usersFile = $this->loadData('User', 'default');
-      $bitstreamsFile = $this->loadData('Bitstream', 'default');
-      $userDao = $this->User->load($usersFile[2]->getKey());
-      $bitstreamDao = $this->Bitstream->load($bitstreamsFile[0]->getKey());
+    /** Test downloading of a single bitstream key file. */
+    public function testDownloadBitstreamKeyfile()
+    {
+        $usersFile = $this->loadData('User', 'default');
+        $bitstreamsFile = $this->loadData('Bitstream', 'default');
+        $userDao = $this->User->load($usersFile[2]->getKey());
+        $bitstreamDao = $this->Bitstream->load($bitstreamsFile[0]->getKey());
 
-      $url = '/keyfiles/download/bitstream?bitstreamId='.$bitstreamDao->getKey();
+        $url = '/keyfiles/download/bitstream?bitstreamId='.$bitstreamDao->getKey();
 
-      // Should throw an exception for no bitstream parameter
-      $this->dispatchUrl('/keyfiles/download/bitstream', null, true);
+        // Should throw an exception for no bitstream parameter
+        $this->dispatchUrl('/keyfiles/download/bitstream', null, true);
 
-      // Make sure we get the checksum as the response
-      $this->resetAll();
-      $this->dispatchUrl($url, $userDao);
-      $this->assertEquals($bitstreamDao->getChecksum(), $this->getBody());
-  }
+        // Make sure we get the checksum as the response
+        $this->resetAll();
+        $this->dispatchUrl($url, $userDao);
+        $this->assertEquals($bitstreamDao->getChecksum(), $this->getBody());
+    }
 
-  /** Test downloading of a recursive zip of keyfiles. */
-  public function testDownloadZip()
-  {
-      // Should throw an exception for no bitstream parameter
-      $this->dispatchUrl('/keyfiles/download/batch', null, true);
+    /** Test downloading of a recursive zip of keyfiles. */
+    public function testDownloadZip()
+    {
+        // Should throw an exception for no bitstream parameter
+        $this->dispatchUrl('/keyfiles/download/batch', null, true);
 
-      // Get some coverage on the batch controller
-      $this->resetAll();
-      $this->dispatchUrl('/keyfiles/download/batch?items=1-2-3-&folders=1000', null);
-      $this->assertController('download');
-      $this->assertAction('batch');
-  }
+        // Get some coverage on the batch controller
+        $this->resetAll();
+        $this->dispatchUrl('/keyfiles/download/batch?items=1-2-3-&folders=1000', null);
+        $this->assertController('download');
+        $this->assertAction('batch');
+    }
 }