xpressengine / xpressengine

app/Providers/PresenterServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
     {
         if (app()->runningInConsole() === false) {
             $this->app->booted(
-                function () {
+                function() {
 
                     $frontendHandler = app('xe.frontend');
 
@@ -76,7 +76,7 @@ 
      */
     private function registerPresenter()
     {
-        $this->app->singleton('xe.presenter', function ($app) {
+        $this->app->singleton('xe.presenter', function($app) {
 
             $proxyClass = $app['xe.interception']->proxy(Presenter::class, 'Presenter');
 
@@ -100,13 +100,13 @@ 
 
             $presenter->register(
                 $proxyHtmlRenderer::format(),
-                function (Presenter $presenter) use ($proxyHtmlRenderer) {
+                function(Presenter $presenter) use ($proxyHtmlRenderer) {
                     return new $proxyHtmlRenderer($presenter, app('xe.seo'), app('xe.widget.parser'));
                 }
             );
             $presenter->register(
                 $proxyJsonRenderer::format(),
-                function (Presenter $presenter) use ($proxyJsonRenderer) {
+                function(Presenter $presenter) use ($proxyJsonRenderer) {
                     return new $proxyJsonRenderer($presenter);
                 }
             );
@@ -122,7 +122,7 @@ 
      */
     private function registerFrontend()
     {
-        $this->app->singleton('xe.frontend', function ($app) {
+        $this->app->singleton('xe.frontend', function($app) {
             $tags = [
                 'title' => \Xpressengine\Presenter\Html\Tags\Title::class,
                 'meta' => \Xpressengine\Presenter\Html\Tags\Meta::class,

config/mail.php

Indentation +18 added lines, -18 removed lines

@@ -2,7 +2,7 @@ 
 
 return [
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| Mail Driver
 	|--------------------------------------------------------------------------
@@ -15,9 +15,9 @@ 
 	|
 	*/
 
-	'driver' => env('MAIL_DRIVER', 'mail'),
+    'driver' => env('MAIL_DRIVER', 'mail'),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| SMTP Host Address
 	|--------------------------------------------------------------------------
@@ -28,9 +28,9 @@ 
 	|
 	*/
 
-	'host' => env('MAIL_HOST', 'smtp.mailgun.org'),
+    'host' => env('MAIL_HOST', 'smtp.mailgun.org'),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| SMTP Host Port
 	|--------------------------------------------------------------------------
@@ -41,9 +41,9 @@ 
 	|
 	*/
 
-	'port' => env('MAIL_PORT', 587),
+    'port' => env('MAIL_PORT', 587),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| Global "From" Address
 	|--------------------------------------------------------------------------
@@ -54,12 +54,12 @@ 
 	|
 	*/
 
-	'from' => [
+    'from' => [
         'address' => env('MAIL_FROM_ADDRESS', null),
         'name' => env('MAIL_FROM_NAME', null)
     ],
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| E-Mail Encryption Protocol
 	|--------------------------------------------------------------------------
@@ -70,9 +70,9 @@ 
 	|
 	*/
 
-	'encryption' => env('MAIL_ENCRYPTION', 'tls'),
+    'encryption' => env('MAIL_ENCRYPTION', 'tls'),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| SMTP Server Username
 	|--------------------------------------------------------------------------
@@ -83,9 +83,9 @@ 
 	|
 	*/
 
-	'username' => env('MAIL_USERNAME', null),
+    'username' => env('MAIL_USERNAME', null),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| SMTP Server Password
 	|--------------------------------------------------------------------------
@@ -96,9 +96,9 @@ 
 	|
 	*/
 
-	'password' => env('MAIL_PASSWORD', null),
+    'password' => env('MAIL_PASSWORD', null),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| Sendmail System Path
 	|--------------------------------------------------------------------------
@@ -109,9 +109,9 @@ 
 	|
 	*/
 
-	'sendmail' => env('MAIL_SENDMAIL', '/usr/sbin/sendmail -bs'),
+    'sendmail' => env('MAIL_SENDMAIL', '/usr/sbin/sendmail -bs'),
 
-	/*
+    /*
 	|--------------------------------------------------------------------------
 	| Mail "Pretend"
 	|--------------------------------------------------------------------------
@@ -122,6 +122,6 @@ 
 	|
 	*/
 
-	'pretend' => env('MAIL_PRETEND', false)
+    'pretend' => env('MAIL_PRETEND', false)
 
 ];

app/Providers/DocumentServiceProvider.php

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function boot()
     {
-        Document::creating(function (Document $model) {
+        Document::creating(function(Document $model) {
             $model->setReply();
         });
     }
@@ -55,17 +55,17 @@ 
         // set reply code length config to Document model
         Document::setReplyCharLen($app['config']['xe.documentReplyCodeLen']);
 
-        $app->singleton('xe.document.config', function ($app) {
+        $app->singleton('xe.document.config', function($app) {
             return new ConfigHandler($app['xe.config']);
         });
-        $app->singleton('xe.document.instance', function ($app) {
+        $app->singleton('xe.document.instance', function($app) {
             $instanceManagerClass = $app['xe.interception']->proxy(InstanceManager::class, 'DocumentInstanceManager');
             return new $instanceManagerClass(
                 $app['xe.db']->connection('document'),
                 $app['xe.document.config']
             );
         });
-        $app->singleton('xe.document', function ($app) {
+        $app->singleton('xe.document', function($app) {
             $documentHandlerClass = $app['xe.interception']->proxy(DocumentHandler::class, 'Document');
             $document = new $documentHandlerClass(
                 $app['xe.db']->connection('document'),

app/Http/Controllers/DynamicFieldController.php

Unused Use Statements -6 removed lines

@@ -3,16 +3,10 @@
 
 use App;
 use XePresenter;
-use Validator;
-use Hash;
-use Auth;
 use Xpressengine\Config\ConfigEntity;
 use Xpressengine\Http\Request;
-use Document;
 use App\Sections\DynamicFieldSection;
 use XeDB;
-use XeLang;
-use XeDynamicField;
 use Xpressengine\Translation\Translator;
 
 class DynamicFieldController extends Controller

Spacing +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
         $fieldSkin = $registerHandler->getSkin($dynamicField, $request->get('skinId'));
 
         return XePresenter::makeApi([
-            'configure' => $fieldType->getSettingsView($config) . $fieldSkin->settings($config),
+            'configure' => $fieldType->getSettingsView($config).$fieldSkin->settings($config),
         ]);
     }
 

core/src/Xpressengine/Routing/Repositories/DatabaseRouteRepository.php

Doc Comments +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
     /**
      * Returns all route items
      *
-     * @return InstanceRoute[]
+     * @return \Illuminate\Database\Eloquent\Collection
      */
     public function all()
     {
@@ -112,7 +112,7 @@ 
      * Retrieve routes by site key
      *
      * @param string $siteKey site key
-     * @return InstanceRoute[]
+     * @return \Illuminate\Database\Eloquent\Collection
      */
     public function fetchBySiteKey($siteKey)
     {
@@ -125,7 +125,7 @@ 
      * Retrieve routes by module name
      *
      * @param string $module module name
-     * @return InstanceRoute[]
+     * @return \Illuminate\Database\Eloquent\Collection
      */
     public function fetchByModule($module)
     {

app/Console/Commands/PluginInstallCommand.php

Doc Comments +5 added lines, -1 removed lines

@@ -68,7 +68,7 @@ 
     /**
      * runComposer
      *
-     * @param $path
+     * @param string $path
      * @param $command
      *
      * @return int
@@ -82,6 +82,10 @@ 
         $output = $this->output;
 
         return $process->run(
+
+            /**
+             * @param string $type
+             */
             function ($type, $line) use ($output) {
                 $output->write($line);
             }

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
 
         $this->info(PHP_EOL."Running 'composer $command'.".PHP_EOL);
 
-        if($this->runComposer(base_path(), $command) !== 0) {
+        if ($this->runComposer(base_path(), $command) !== 0) {
             throw new \Exception('Plugin Installation was faild.');
         }
 
@@ -59,7 +59,7 @@ 
 
         $activate = $this->option('activate');
 
-        if($activate) {
+        if ($activate) {
             $this->activatePlugin($name);
             $this->info("Plugin '$name' is activated.".PHP_EOL);
         }
@@ -82,7 +82,7 @@ 
         $output = $this->output;
 
         return $process->run(
-            function ($type, $line) use ($output) {
+            function($type, $line) use ($output) {
                 $output->write($line);
             }
         );

app/Console/Kernel.php

Indentation +54 added lines, -54 removed lines

@@ -8,50 +8,50 @@ 
 class Kernel extends ConsoleKernel
 {
 
-	/**
-	 * The bootstrap classes for the application.
-	 *
-	 * @var array
-	 */
-	protected $bootstrappers = [
-			'Illuminate\Foundation\Bootstrap\DetectEnvironment',
-			'App\Bootstrappers\LoadConfiguration',
-			'Illuminate\Foundation\Bootstrap\ConfigureLogging',
-			'Illuminate\Foundation\Bootstrap\HandleExceptions',
-			'Illuminate\Foundation\Bootstrap\RegisterFacades',
-			'Illuminate\Foundation\Bootstrap\SetRequestForConsole',
-			'Illuminate\Foundation\Bootstrap\RegisterProviders',
-			'Illuminate\Foundation\Bootstrap\BootProviders',
-	];
-
-	/**
-	 * The Artisan commands provided by your application.
-	 *
-	 * @var array
-	 */
-	protected $commands = [
-		\App\Console\Commands\Inspire::class,
+    /**
+     * The bootstrap classes for the application.
+     *
+     * @var array
+     */
+    protected $bootstrappers = [
+            'Illuminate\Foundation\Bootstrap\DetectEnvironment',
+            'App\Bootstrappers\LoadConfiguration',
+            'Illuminate\Foundation\Bootstrap\ConfigureLogging',
+            'Illuminate\Foundation\Bootstrap\HandleExceptions',
+            'Illuminate\Foundation\Bootstrap\RegisterFacades',
+            'Illuminate\Foundation\Bootstrap\SetRequestForConsole',
+            'Illuminate\Foundation\Bootstrap\RegisterProviders',
+            'Illuminate\Foundation\Bootstrap\BootProviders',
+    ];
+
+    /**
+     * The Artisan commands provided by your application.
+     *
+     * @var array
+     */
+    protected $commands = [
+        \App\Console\Commands\Inspire::class,
         \App\Console\Commands\XeInstall::class,
-		\App\Console\Commands\Trash::class,
-		\App\Console\Commands\Schema::class,
-		\App\Console\Commands\PutTranslation::class,
-		\App\Console\Commands\XeCacheClear::class,
-		\App\Console\Commands\StorageOptimize::class,
-		\App\Console\Commands\Site::class,
-		\App\Console\Commands\PluginMakeCommand::class,
-		\App\Console\Commands\PluginInstallCommand::class,
-	];
-
-	/**
-	 * Define the application's command schedule.
-	 *
-	 * @param  \Illuminate\Console\Scheduling\Schedule  $schedule
-	 * @return void
-	 */
-	protected function schedule(Schedule $schedule)
-	{
-		$schedule->command('inspire')->hourly();
-	}
+        \App\Console\Commands\Trash::class,
+        \App\Console\Commands\Schema::class,
+        \App\Console\Commands\PutTranslation::class,
+        \App\Console\Commands\XeCacheClear::class,
+        \App\Console\Commands\StorageOptimize::class,
+        \App\Console\Commands\Site::class,
+        \App\Console\Commands\PluginMakeCommand::class,
+        \App\Console\Commands\PluginInstallCommand::class,
+    ];
+
+    /**
+     * Define the application's command schedule.
+     *
+     * @param  \Illuminate\Console\Scheduling\Schedule  $schedule
+     * @return void
+     */
+    protected function schedule(Schedule $schedule)
+    {
+        $schedule->command('inspire')->hourly();
+    }
 
     /**
      * Bootstrap the application for artisan commands.
@@ -83,18 +83,18 @@ 
      * @return void
      */
     protected function resetForFramework()
-	{
-		$this->resetProviders();
-		$this->setCommandBeforeInstall();
-	}
+    {
+        $this->resetProviders();
+        $this->setCommandBeforeInstall();
+    }
 
     /**
      * Define for providers of framework.
      *
      * @return void
      */
-	protected function resetProviders()
-	{
+    protected function resetProviders()
+    {
         $this->app['events']->listen('bootstrapped: App\Bootstrappers\LoadConfiguration', function ($app) {
             $config = $app['config'];
 
@@ -105,20 +105,20 @@ 
 
             $config->set('app.providers', $providers);
         });
-	}
+    }
 
     /**
      * Define commands for previously installation
      *
      * @return void
      */
-	protected function setCommandBeforeInstall()
-	{
-		$this->commands = array_intersect($this->commands, [
+    protected function setCommandBeforeInstall()
+    {
+        $this->commands = array_intersect($this->commands, [
             \App\Console\Commands\Inspire::class,
             \App\Console\Commands\XeInstall::class
         ]);
-	}
+    }
 
     /**
      * Define commands for after installation

Spacing +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
 
     protected function isInstalled()
     {
-        return file_exists($this->app->storagePath() . '/app/installed');
+        return file_exists($this->app->storagePath().'/app/installed');
     }
 
     /**
@@ -95,11 +95,11 @@ 
      */
 	protected function resetProviders()
 	{
-        $this->app['events']->listen('bootstrapped: App\Bootstrappers\LoadConfiguration', function ($app) {
+        $this->app['events']->listen('bootstrapped: App\Bootstrappers\LoadConfiguration', function($app) {
             $config = $app['config'];
 
             $providers = $config['app.providers'];
-            $providers = array_filter($providers, function ($p) {
+            $providers = array_filter($providers, function($p) {
                 return substr($p, 0, strlen('Illuminate')) == 'Illuminate';
             });
 

core/src/Xpressengine/Database/Eloquent/DynamicModel.php

Doc Comments +2 added lines, -2 removed lines

@@ -121,7 +121,7 @@ 
      * fill
      *
      * @param array $attributes attributes
-     * @return void
+     * @return DynamicModel
      */
     public function fill(array $attributes)
     {
@@ -247,7 +247,7 @@ 
      * Xpressengine\Database\Eloquent\Builder 울 사용하도록 변경
      *
      * @param  DynamicQuery $query dynamic query builder
-     * @return \Xpressengine\Database\Eloquent\Builder|static
+     * @return Builder
      */
     public function newEloquentBuilder($query)
     {

core/tests/User/UserAccountRepositoryTest.php

Doc Comments -1 removed lines

@@ -102,7 +102,6 @@
     /**
      * getRepository
      *
-     * @param null $model
      *
      * @return Mockery\MockInterface
      */