xpressengine / xpressengine

core/tests/Tag/TagHandlerTest.php

Doc Comments +4 added lines

@@ -191,6 +191,10 @@
         $this->assertInstanceOf('Xpressengine\Tag\Tag', current($tags));
     }
 
+    /**
+     * @param TagHandler $object
+     * @param string $methodName
+     */
     private function invokeMethod(&$object, $methodName, array $parameters = array())
     {
         $reflection = new \ReflectionClass(get_class($object));

Spacing +2 added lines, -2 removed lines

@@ -99,7 +99,7 @@ 
         $mockTag->shouldReceive('getKey')->andReturn(1);
 
         $mockConn->shouldReceive('table')->with('taggable')->andReturnSelf();
-        $mockConn->shouldReceive('insert')->once()->with(m::on(function ($args) {
+        $mockConn->shouldReceive('insert')->once()->with(m::on(function($args) {
             return $args['tagId'] === 1
                 && $args['taggableId'] === 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
                 && $args['position'] === 0;
@@ -124,7 +124,7 @@ 
         $mockTag->shouldReceive('getKey')->andReturn(1);
 
         $mockConn->shouldReceive('table')->with('taggable')->andReturnSelf();
-        $mockConn->shouldReceive('insert')->once()->with(m::on(function ($args) {
+        $mockConn->shouldReceive('insert')->once()->with(m::on(function($args) {
             return $args['tagId'] === 1
             && $args['taggableId'] === 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
             && $args['position'] === 0;

resources/FieldSkins/Category/DefaultSkin.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Xpressengine\DynamicField\AbstractSkin;
 use Xpressengine\Config\ConfigEntity;
 use View;
-use XeCategory;
 
 class DefaultSkin extends AbstractSkin
 {

Spacing +5 added lines, -5 removed lines

@@ -32,7 +32,7 @@ 
      */
     public function settings(ConfigEntity $config = null, $view = 'dynamicField/category/default/createSkin')
     {
-        return View::make($view, ['config' => $config,])->render();
+        return View::make($view, ['config' => $config, ])->render();
     }
 
     /**
@@ -65,8 +65,8 @@ 
 
         $itemId = '';
         $item = '';
-        if (isset($args[$config->get('id') . 'ItemId'])) {
-            $itemId = $args[$config->get('id') . 'ItemId'];
+        if (isset($args[$config->get('id').'ItemId'])) {
+            $itemId = $args[$config->get('id').'ItemId'];
             $item = CategoryItem::find($itemId);
         }
 
@@ -87,8 +87,8 @@ 
     {
         $config = $this->config;
         $item = '';
-        if (isset($args[$config->get('id') . 'ItemId'])) {
-            $itemId = $args[$config->get('id') . 'ItemId'];
+        if (isset($args[$config->get('id').'ItemId'])) {
+            $itemId = $args[$config->get('id').'ItemId'];
             $item = CategoryItem::find($itemId);
         }
 

core/tests/Temporary/TemporaryHandlerTest.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
         $auth->shouldReceive('guest')->andReturn(false);
         $auth->shouldReceive('user')->andReturn($mockUser);
 
-        $repo->shouldReceive('insert')->once()->with(m::on(function ($entity) {
+        $repo->shouldReceive('insert')->once()->with(m::on(function($entity) {
             return (
                 $entity->userId == 'userId'
                 && $entity->key == 'someKey'

core/tests/Category/CategoryHandlerTest.php

Spacing +3 added lines, -3 removed lines

@@ -193,7 +193,7 @@ 
         $instance = new CategoryHandler();
 
         $collection = m::mock('Illuminate\Database\Eloquent\Collection');
-        $collection->shouldReceive('filter')->once()->with(m::on(function () { return true; }))->andReturnSelf();
+        $collection->shouldReceive('filter')->once()->with(m::on(function() { return true; }))->andReturnSelf();
 
         $mockParent = m::mock('Xpressengine\Category\Models\CategoryItem');
         $mockParent->shouldReceive('getChildren')->andReturn($collection);
@@ -215,7 +215,7 @@ 
         $mockItem->shouldReceive('setAttribute')->once()->with('ordering', 1)->andReturnNull();
         $mockItem->shouldReceive('save')->once()->andReturnNull();
 
-        $collection->shouldReceive('each')->once()->with(m::on(function ($closure) use ($mockItem, $mockSibling) {
+        $collection->shouldReceive('each')->once()->with(m::on(function($closure) use ($mockItem, $mockSibling) {
             $arr = [$mockSibling, $mockItem];
 
             foreach ($arr as $idx => $model) {
@@ -225,7 +225,7 @@ 
             return true;
         }))->andReturnSelf();
 
-        $invoke = function (&$object, $methodName, array $parameters = array()) {
+        $invoke = function(&$object, $methodName, array $parameters = array()) {
             $reflection = new \ReflectionClass(get_class($object));
             $method = $reflection->getMethod($methodName);
             $method->setAccessible(true);

core/tests/Document/DocumentHandlerTest.php

Indentation +4 added lines, -4 removed lines

@@ -25,10 +25,10 @@
     protected $request;
 
         /**
-     * tear down
-     *
-     * @return void
-     */
+         * tear down
+         *
+         * @return void
+         */
     public function tearDown()
     {
         m::close();

Doc Comments +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
 
 
     /**
-     * @return m\MockInterface|\Xpressengine\Document\Models\Document
+     * @return m\MockInterface
      */
     private function getDocModel()
     {
@@ -63,7 +63,7 @@ 
     }
 
     /**
-     * @return m\MockInterface|\Xpressengine\Document\Models\Revision
+     * @return m\MockInterface
      */
     private function getRevisionModel()
     {
@@ -75,7 +75,7 @@ 
     /**
      * get User instance
      *
-     * @return m\MockInterface|\Xpressengine\User\UserInterface
+     * @return m\MockInterface
      */
     private function getUser()
     {
@@ -89,7 +89,7 @@ 
     /**
      * get config entity
      *
-     * @return m\MockInterface|\Xpressengine\Config\ConfigEntity
+     * @return m\MockInterface
      */
     private function getConfigEntity()
     {
@@ -99,7 +99,7 @@ 
     /**
      * @param array $items items
      *
-     * @return m\MockInterface|\Illuminate\Pagination\LengthAwarePaginator
+     * @return m\MockInterface
      */
     private function getPaginator(array $items)
     {

core/src/Xpressengine/User/Models/Guest.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
      */
     public static function setDefaultProfileImage($img)
     {
-        self::$profileImage= $img;
+        self::$profileImage = $img;
     }
 
     /**

core/src/Xpressengine/User/EmailBroker.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
         $this->mailer->send(
             $this->view,
             compact('mail'),
-            function ($m) use ($mail, $callback) {
+            function($m) use ($mail, $callback) {
                 $m->to($mail->getAddress());
 
                 if (!is_null($callback)) {

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Closure;
 use Illuminate\Contracts\Mail\Mailer;
 use Xpressengine\User\Exceptions\InvalidConfirmationCodeException;
-use Xpressengine\User\Exceptions\PendingEmailNotExistsException;
 
 /**
  * 이 클래스는 Xpressengine에서 이메일 인증 처리를 수행하는 클래스이다.

app/Providers/MenuServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      */
     public function register()
     {
-        $this->app->singleton([MenuHandler::class => 'xe.menu'], function ($app) {
+        $this->app->singleton([MenuHandler::class => 'xe.menu'], function($app) {
             return new MenuHandler(
                 $app['xe.keygen'],
                 $app['xe.config'],

app/Providers/DatabaseServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
     {
         if ($this->app['config']->get('app.debug') === true) {
 
-            \DB::listen(function ($query, $bindings, $time) {
+            \DB::listen(function($query, $bindings, $time) {
                 $logFile = storage_path('logs/query.log');
                 $monoLog = new Logger('log');
                 $monoLog->pushHandler(new StreamHandler($logFile, Logger::INFO));
@@ -65,11 +65,11 @@ 
     {
         DynamicModel::clearBootedModels();
 
-        $this->app->singleton('xe.db.proxy', function ($app) {
+        $this->app->singleton('xe.db.proxy', function($app) {
             return ProxyManager::instance($app['xe.register']);
         });
 
-        $this->app->singleton('xe.db', function ($app) {
+        $this->app->singleton('xe.db', function($app) {
 
             $coupler = DatabaseCoupler::instance(
                 $app['db'],