interserver / myadmin-plugin-installer

src/Command/Command.php

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

@@ -9,9 +9,9 @@
 
 namespace MyAdmin\Plugins\Command;
 
+use Composer\Command\BaseCommand;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Composer\Command\BaseCommand;
 
 /**
  * Class Command

Indentation +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 			'User Creator',
 			'============',
 			''
-		                 ]);
+						 ]);
 		/*
 		$output->writeln('Username: '.$input->getArgument('username')); // retrieve the argument value using getArgument()
 		*/

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 	 * @param OutputInterface $output
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output) {
-		$output->writeln([ // outputs multiple lines to the console (adding "\n" at the end of each line)
+		$output->writeln([// outputs multiple lines to the console (adding "\n" at the end of each line)
 			'User Creator',
 			'============',
 			''

src/Command/CreateUser.php

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

@@ -9,12 +9,10 @@
 
 namespace MyAdmin\Plugins\Command;
 
+use Composer\Command\BaseCommand;
 use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Composer\Command\BaseCommand;
 
 /**
  * Class CreateUser

Indentation +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 			'User Creator',
 			'============',
 			''
-		                 ]);
+						 ]);
 		$output->writeln('Username: '.$input->getArgument('username')); // retrieve the argument value using getArgument()
 		$output->write('You are about to '); // outputs a message without adding a "\n" at the end of the line
 		$output->write('create a user.');

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 	 * @param OutputInterface $output
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output) {
-		$output->writeln([ // outputs multiple lines to the console (adding "\n" at the end of each line)
+		$output->writeln([// outputs multiple lines to the console (adding "\n" at the end of each line)
 			'User Creator',
 			'============',
 			''

src/Command/Parse.php

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

@@ -9,9 +9,9 @@
 
 namespace MyAdmin\Plugins\Command;
 
+use Composer\Command\BaseCommand;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Composer\Command\BaseCommand;
 
 /**
  * Code Parser Comand

Indentation +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 			'MyAdmin DocBlock Parser',
 			'=======================',
 			''
-		                 ]);
+						 ]);
 		$output->writeln('<info>foo</info>'); // green text
 		$output->writeln('<comment>foo</comment>'); // yellow text
 		$output->writeln('<question>foo</question>'); // black text on a cyan background

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 	 * @param OutputInterface $output
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output) {
-		$output->writeln([ // outputs multiple lines to the console (adding "\n" at the end of each line)
+		$output->writeln([// outputs multiple lines to the console (adding "\n" at the end of each line)
 			'MyAdmin DocBlock Parser',
 			'=======================',
 			''

src/Command/UpdatePlugins.php

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

@@ -9,9 +9,9 @@
 
 namespace MyAdmin\Plugins\Command;
 
+use Composer\Command\BaseCommand;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Composer\Command\BaseCommand;
 
 /**
  * Class UpdatePlugins

Indentation +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 			'User Creator',
 			'============',
 			''
-		                 ]);
+						 ]);
 		$output->write('You are about to '); // outputs a message without adding a "\n" at the end of the line
 		$output->write('create a user.');
 

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 	 * @param OutputInterface $output
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output) {
-		$output->writeln([ // outputs multiple lines to the console (adding "\n" at the end of each line)
+		$output->writeln([// outputs multiple lines to the console (adding "\n" at the end of each line)
 			'User Creator',
 			'============',
 			''

src/CommandProvider.php

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

@@ -11,8 +11,8 @@
 
 use Composer\Plugin\Capability\CommandProvider as CommandProviderCapability;
 use MyAdmin\Plugins\Command\Command;
-use MyAdmin\Plugins\Command\Parse;
 use MyAdmin\Plugins\Command\CreateUser;
+use MyAdmin\Plugins\Command\Parse;
 use MyAdmin\Plugins\Command\UpdatePlugins;
 
 /**

src/Installer.php

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

@@ -25,13 +25,13 @@
 namespace MyAdmin\Plugins;
 
 use Composer\Composer;
+use Composer\IO\IOInterface;
 use Composer\Installer\BinaryInstaller;
 use Composer\Installer\LibraryInstaller;
-use Composer\IO\IOInterface;
 use Composer\Package\PackageInterface;
+use Composer\Repository\InstalledRepositoryInterface;
 use Composer\Util\Filesystem;
 use Composer\Util\Silencer;
-use Composer\Repository\InstalledRepositoryInterface;
 
 /**
  * Class Installer

Spacing +3 added lines, -3 removed lines

@@ -143,13 +143,13 @@
 	public function getInstallPath(PackageInterface $package) {
 		if ($this->type == 'myadmin-template') {
 			$this->initializeTemplateDir();
-			$basePath = ($this->templateDir ? $this->templateDir.'/' : '') . $package->getPrettyName();
+			$basePath = ($this->templateDir ? $this->templateDir.'/' : '').$package->getPrettyName();
 		} else {
 			$this->initializeVendorDir();
-			$basePath = ($this->vendorDir ? $this->vendorDir.'/' : '') . $package->getPrettyName();
+			$basePath = ($this->vendorDir ? $this->vendorDir.'/' : '').$package->getPrettyName();
 		}
 		$targetDir = $package->getTargetDir();
-		return $basePath . ($targetDir ? '/'.$targetDir : '');
+		return $basePath.($targetDir ? '/'.$targetDir : '');
 	}
 
 	/**

src/Plugin.php

Indentation +2 added lines, -2 removed lines

@@ -99,8 +99,8 @@
 	}
 
 	/**
-	* @param PreFileDownloadEvent $event
-	*/
+	 * @param PreFileDownloadEvent $event
+	 */
 	public function onPreFileDownload(PreFileDownloadEvent $event) {
 		/*$protocol = parse_url($event->getProcessedUrl(), PHP_URL_SCHEME);
 		if ($protocol === 's3') {

Doc Comments +3 added lines

@@ -165,6 +165,9 @@
 		self::runCommand('chmod +a "$USER allow delete,write,append,file_inherit,directory_inherit" %path%', $path);
 	}
 
+	/**
+	 * @param string $command
+	 */
 	public static function runCommand($command, $path) {
 		return self::runProcess(str_replace(['%httpduser%', '%path%'], [self::getHttpdUser(), $path], $command));
 	}

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

@@ -17,9 +17,9 @@
 use Composer\Composer;
 use Composer\EventDispatcher\EventSubscriberInterface;
 use Composer\IO\IOInterface;
-use Composer\Plugin\PluginInterface;
 use Composer\Plugin\Capable;
 use Composer\Plugin\PluginEvents;
+use Composer\Plugin\PluginInterface;
 use Composer\Plugin\PreFileDownloadEvent;
 use Composer\Script\Event;
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
 
 	public static function SetfaclPermissionsSetter($path) {
 		if (!is_dir($path))
-			mkdir($path, 0777, true);
+			mkdir($path, 0777, TRUE);
 		if (!is_dir($path))
 			throw new \Exception('Path Not Found: '.$path);
 		self::runCommand('setfacl -m u:"%httpduser%":rwX -m u:$USER:rwX %path%', $path);
@@ -162,7 +162,7 @@ 
 
 	public static function ChmodPermissionsSetter($path) {
 		if (!is_dir($path))
-			mkdir($path, 0777, true);
+			mkdir($path, 0777, TRUE);
 		if (!is_dir($path))
 			throw new \Exception('Path Not Found: '.$path);
 		self::runCommand('chmod +a "%httpduser% allow delete,write,append,file_inherit,directory_inherit" %path%', $path);

src/modules.php

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -14,8 +14,8 @@ 
  * @param array|bool $settings
  * @return void
  */
-function register_module($module, $settings = false) {
-	if ($settings === false)
+function register_module($module, $settings = FALSE) {
+	if ($settings === FALSE)
 		$settings = [];
 	if (!isset($GLOBALS['modules']))
 		$GLOBALS['modules'] = [];
@@ -70,16 +70,16 @@ 
  * @param bool|string $setting optional parameter, false to return all settings, or a specific setting name to return that setting
  * @return array|false array of settings or false if no setting
  */
-function get_module_settings($module = 'default', $setting = false) {
+function get_module_settings($module = 'default', $setting = FALSE) {
 	if (!isset($GLOBALS['modules'][$module])) {
 		$keys = array_keys($GLOBALS['modules']);
 		$module = $keys[0];
 	}
-	if ($setting !== false) {
+	if ($setting !== FALSE) {
 		if (isset($GLOBALS['modules'][$module][$setting]))
 			return $GLOBALS['modules'][$module][$setting];
 		else
-			return false;
+			return FALSE;
 	} else
 		return $GLOBALS['modules'][$module];
 }

src/TemplateInstaller.php

Indentation +17 added lines, -17 removed lines

@@ -22,22 +22,22 @@
 	 * {@inheritDoc}
 	 * @throws \InvalidArgumentException
 	 */
-    public function getInstallPath(PackageInterface $package) {
-        $prefix = mb_substr($package->getPrettyName(), 0, 23);
-        if ('myadmin/template-' !== $prefix) {
-            throw new \InvalidArgumentException(
-                'Unable to install template, myadmin templates '
-                .'should always start their package name with '
-                .'"myadmin/template-"'
-            );
-        }
-        return 'data/templates/'.mb_substr($package->getPrettyName(), 23);
-    }
+	public function getInstallPath(PackageInterface $package) {
+		$prefix = mb_substr($package->getPrettyName(), 0, 23);
+		if ('myadmin/template-' !== $prefix) {
+			throw new \InvalidArgumentException(
+				'Unable to install template, myadmin templates '
+				.'should always start their package name with '
+				.'"myadmin/template-"'
+			);
+		}
+		return 'data/templates/'.mb_substr($package->getPrettyName(), 23);
+	}
 
-    /**
-     * {@inheritDoc}
-     */
-    public function supports($packageType) {
-        return 'myadmin-template' === $packageType;
-    }
+	/**
+	 * {@inheritDoc}
+	 */
+	public function supports($packageType) {
+		return 'myadmin-template' === $packageType;
+	}
 }