LukeCarrier / moodle-componentmgr

etc/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
 
 // Error reporting
-ini_set('display_errors',  'on');
+ini_set('display_errors', 'on');
 ini_set('error_reporting', E_ALL);
 
 // Set the default timezone if not already set

lib/ComponentManager/HttpClient.php

Indentation +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
      * @return RequestInterface
      */
     public function createRequest($method, $uri, $headers=[], $body=null,
-                                  $protocolVersion='1.1') {
+                                    $protocolVersion='1.1') {
         return $this->messageFactory->createRequest(
                 $method, $uri, $headers, $body, $protocolVersion);
     }

Spacing +2 added lines, -2 removed lines

@@ -78,8 +78,8 @@
      *
      * @return RequestInterface
      */
-    public function createRequest($method, $uri, $headers=[], $body=null,
-                                  $protocolVersion='1.1') {
+    public function createRequest($method, $uri, $headers = [], $body = null,
+                                  $protocolVersion = '1.1') {
         return $this->messageFactory->createRequest(
                 $method, $uri, $headers, $body, $protocolVersion);
     }

test/unit/ComponentTest.php

Indentation +23 added lines, -23 removed lines

@@ -20,33 +20,33 @@
  * @coversDefaultClass \ComponentManager\Component
  */
 class ComponentTest extends TestCase {
-   public function testGetNameParts() {
-       $component = new Component('type_name', []);
-       $this->assertEquals(['type', 'name'], $component->getNameParts());
-       $this->assertEquals('type', $component->getPluginType());
-       $this->assertEquals('name', $component->getPluginName());
-   }
-
-   public function testGetVersion() {
-       $goodPackageRepository = $this->createMock(PackageRepository::class);
-       $goodPackageRepository->method('satisfiesVersion')
-           ->willReturn(true);
-       $badPackageRepository = $this->createMock(PackageRepository::class);
-       $badPackageRepository->method('satisfiesVersion')
-           ->willReturn(false);
-
-       $componentVersion = new ComponentVersion(
+    public function testGetNameParts() {
+        $component = new Component('type_name', []);
+        $this->assertEquals(['type', 'name'], $component->getNameParts());
+        $this->assertEquals('type', $component->getPluginType());
+        $this->assertEquals('name', $component->getPluginName());
+    }
+
+    public function testGetVersion() {
+        $goodPackageRepository = $this->createMock(PackageRepository::class);
+        $goodPackageRepository->method('satisfiesVersion')
+            ->willReturn(true);
+        $badPackageRepository = $this->createMock(PackageRepository::class);
+        $badPackageRepository->method('satisfiesVersion')
+            ->willReturn(false);
+
+        $componentVersion = new ComponentVersion(
             '2015021800', 'Genesis', ComponentVersion::MATURITY_STABLE, []);
 
-       $component = new Component(
+        $component = new Component(
             'type_name', [$componentVersion], $goodPackageRepository);
-       $this->assertEquals(
+        $this->assertEquals(
             $componentVersion, $component->getVersion('2015021800'));
 
-       $this->expectException(UnsatisfiedVersionException::class);
-       $this->expectExceptionCode(UnsatisfiedVersionException::CODE_UNKNOWN_VERSION);
-       $component = new Component(
+        $this->expectException(UnsatisfiedVersionException::class);
+        $this->expectExceptionCode(UnsatisfiedVersionException::CODE_UNKNOWN_VERSION);
+        $component = new Component(
             'type_name', [$componentVersion], $badPackageRepository);
-       $component->getVersion('2015021800');
-   }
+        $component->getVersion('2015021800');
+    }
 }

lib/ComponentManager/VersionControl/Git/GitVersionControl.php

Spacing +3 added lines, -3 removed lines

@@ -85,7 +85,7 @@ 
      *
      * @throws VersionControlException
      */
-    public function runCommand(Command $command, $exceptionCode=null) {
+    public function runCommand(Command $command, $exceptionCode = null) {
         $process = $this->createProcess($command->getCommandLine());
         $process->run();
 
@@ -159,7 +159,7 @@ 
      *
      * @codeCoverageIgnore
      */
-    public function fetch($remote=null) {
+    public function fetch($remote = null) {
         $command = new FetchCommand($remote);
         return $this->runCommand(
                 $command, VersionControlException::CODE_FETCH_FAILED);
@@ -176,7 +176,7 @@ 
      *
      * @codeCoverageIgnore
      */
-    public function fetchTags($remote=null) {
+    public function fetchTags($remote = null) {
         $command = new FetchCommand($remote);
         $command->setTags(true);
         return $this->runCommand(

lib/ComponentManager/VersionControl/Git/Command/FetchCommand.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      *
      * @param string|null $remote
      */
-    public function __construct($remote=null) {
+    public function __construct($remote = null) {
         $this->remote = $remote;
     }
 

lib/ComponentManager/PackageRepository/PackageRepository.php

Doc Comments +1 added lines

@@ -33,6 +33,7 @@
      * @param HttpClient $httpClient
      * @param Platform   $platform
      * @param stdClass   $options
+     * @return void
      */
     public function __construct(Filesystem $filesystem, HttpClient $httpClient,
                                 Platform $platform, stdClass $options);

Unused Use Statements +1 added lines, -1 removed lines

@@ -16,8 +16,8 @@
 use ComponentManager\Exception\InvalidProjectException;
 use ComponentManager\HttpClient;
 use ComponentManager\Platform\Platform;
-use stdClass;
 use Symfony\Component\Filesystem\Filesystem;
+use stdClass;
 
 /**
  * Package repository interface.

lib/ComponentManager/Platform/Platform.php

Doc Comments +1 added lines

@@ -24,6 +24,7 @@
      * Initialiser.
      *
      * @param Filesystem $filesystem
+     * @return void
      */
     public function __construct(Filesystem $filesystem);
 

lib/ComponentManager/ComponentSpecification.php

Spacing +2 added lines, -2 removed lines

@@ -66,8 +66,8 @@
      * @param string   $packageSource
      * @param stdClass $extra
      */
-    public function __construct($name, $version, $packageRepository=null,
-                                $packageSource=null, stdClass $extra=null) {
+    public function __construct($name, $version, $packageRepository = null,
+                                $packageSource = null, stdClass $extra = null) {
         $this->name    = $name;
         $this->version = $version;
 

lib/ComponentManager/Command/ProjectAwareCommand.php

Spacing +2 added lines, -2 removed lines

@@ -137,7 +137,7 @@ 
      *
      * @return Moodle
      */
-    protected function getMoodle($moodleDirectory=null) {
+    protected function getMoodle($moodleDirectory = null) {
         if ($this->moodle === null) {
             $moodleDirectory = ($moodleDirectory === null)
                     ? $this->platform->getWorkingDirectory() : $moodleDirectory;
@@ -156,7 +156,7 @@ 
      *
      * @return Project
      */
-    protected function getProject($projectFilename=null, $projectLockFilename=null) {
+    protected function getProject($projectFilename = null, $projectLockFilename = null) {
         $workingDirectory = $this->platform->getWorkingDirectory();
 
         if ($this->project === null) {