AsgardCms / Block

Providers/RouteServiceProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     protected $namespace = 'Modules\Block\Http\Controllers';
 
     /**
-     * @return string
+     * @return boolean
      */
     protected function getFrontendRoute()
     {
@@ -27,7 +27,7 @@ 
     }
 
     /**
-     * @return string
+     * @return boolean
      */
     protected function getApiRoute()
     {

Database/Migrations/2015_04_01_180809_create_blocks_tables.php

Spacing +3 added lines, -3 removed lines

@@ -11,14 +11,14 @@ 
      */
     public function up()
     {
-        Schema::create('block__blocks', function (Blueprint $table) {
+        Schema::create('block__blocks', function(Blueprint $table) {
             $table->engine = 'InnoDB';
             $table->increments('id');
             $table->string('name');
             $table->timestamps();
         });
 
-        Schema::create('block__blocks_translations', function (Blueprint $table) {
+        Schema::create('block__blocks_translations', function(Blueprint $table) {
             $table->engine = 'InnoDB';
             $table->increments('id');
             $table->tinyInteger('online')->nullable();
@@ -26,7 +26,7 @@ 
 
             $table->integer('block_id')->unsigned();
             $table->string('locale')->index();
-            $table->unique(['block_id', 'locale']);
+            $table->unique([ 'block_id', 'locale' ]);
             $table->foreign('block_id')->references('id')->on('block__blocks')->onDelete('cascade');
         });
     }

Entities/BlockTranslation.php

Spacing +1 added lines, -1 removed lines

@@ -5,6 +5,6 @@
 class BlockTranslation extends Model
 {
     public $timestamps = false;
-    protected $fillable = ['online', 'body'];
+    protected $fillable = [ 'online', 'body' ];
     protected $table = 'block__blocks_translations';
 }

Http/Requests/UpdateBlockRequest.php

Spacing +1 added lines, -1 removed lines

@@ -32,6 +32,6 @@
 
     public function messages()
     {
-        return [];
+        return [ ];
     }
 }

Repositories/Cache/CacheBlockDecorator.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
         return $this->cache
             ->tags($this->entityName, 'global')
             ->remember("{$this->locale}.{$this->entityName}.allOnlineInLang", $this->cacheTime,
-                function () use ($lang) {
+                function() use ($lang) {
                     return $this->repository->allOnlineInLang($lang);
                 }
             );
@@ -38,7 +38,7 @@ 
         return $this->cache
             ->tags($this->entityName, 'global')
             ->remember("{$this->locale}.{$this->entityName}.get", $this->cacheTime,
-                function () use ($name) {
+                function() use ($name) {
                     return $this->repository->get($name);
                 }
             );

Repositories/Eloquent/EloquentBlockRepository.php

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
      */
     public function allOnlineInLang($lang)
     {
-        return $this->model->whereHas('translations', function (Builder $q) use ($lang) {
+        return $this->model->whereHas('translations', function(Builder $q) use ($lang) {
             $q->where('locale', "$lang");
             $q->where('online', true);
         })->with('translations')->orderBy('created_at', 'DESC')->get();
@@ -56,7 +56,7 @@ 
     {
         $lang = $this->getCurrentLocale();
 
-        $block = $this->model->whereHas('translations', function (Builder $q) use ($lang) {
+        $block = $this->model->whereHas('translations', function(Builder $q) use ($lang) {
             $q->where('locale', "$lang");
             $q->where('online', true);
         })->with('translations')->whereName($name)->first();
@@ -79,7 +79,7 @@ 
      */
     private function sluggify(array &$data)
     {
-        $data['name'] = $this->makeNameUnique($data['name']);
+        $data[ 'name' ] = $this->makeNameUnique($data[ 'name' ]);
     }
 
     /**
@@ -95,7 +95,7 @@ 
 
         if (
             $this->isEmptyArray($list) ||
-            ! $this->isSlugInList($slug, $list) ||
+            !$this->isSlugInList($slug, $list) ||
             ($this->isForModel($list) && $this->slugIsInList($slug, $list))
         ) {
             return $slug;
@@ -104,7 +104,7 @@ 
         // map our list to keep only the increments
         $len = strlen($slug . '-');
 
-        array_walk($list, function (&$value, $key) use ($len) {
+        array_walk($list, function(&$value, $key) use ($len) {
             $value = intval(substr($value, $len));
         });
 
@@ -164,7 +164,7 @@ 
      */
     private function slugIsInList($slug, array $list)
     {
-        return $list[$this->model->getKey()] === $slug;
+        return $list[ $this->model->getKey() ] === $slug;
     }
 
     /**

Doc Comments +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
 {
     /**
      * @param mixed $data
-     * @return mixed
+     * @return \Illuminate\Database\Eloquent\Model
      */
     public function create($data)
     {
@@ -176,7 +176,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return integer
      */
     private function findHighestIncrement($list)
     {

Sidebar/SidebarExtender.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
      */
     public function extendWith(Menu $menu)
     {
-        $menu->group(trans('core::sidebar.content'), function (Group $group) {
-            $group->item(trans('block::blocks.title.blocks'), function (Item $item) {
+        $menu->group(trans('core::sidebar.content'), function(Group $group) {
+            $group->item(trans('block::blocks.title.blocks'), function(Item $item) {
                 $item->authorize(
                     $this->auth->hasAccess('block.blocks.index')
                 );

Tests/Integration/BaseBlockTest.php

Spacing +4 added lines, -4 removed lines

@@ -45,14 +45,14 @@
 
     protected function getEnvironmentSetUp($app)
     {
-        $app['path.base'] = __DIR__ . '/..';
-        $app['config']->set('database.default', 'sqlite');
-        $app['config']->set('database.connections.sqlite', array(
+        $app[ 'path.base' ] = __DIR__ . '/..';
+        $app[ 'config' ]->set('database.default', 'sqlite');
+        $app[ 'config' ]->set('database.connections.sqlite', array(
             'driver' => 'sqlite',
             'database' => ':memory:',
             'prefix' => '',
         ));
-        $app['config']->set('translatable.locales', ['en', 'fr']);
+        $app[ 'config' ]->set('translatable.locales', [ 'en', 'fr' ]);
     }
 
     private function resetDatabase()

Tests/Integration/BlockRepositoryTest.php

Spacing +15 added lines, -15 removed lines

@@ -9,7 +9,7 @@ 
     /** @test */
     public function it_creates_blocks()
     {
-        $block = $this->block->create(['name' => 'testBlock', 'en' => ['body' => 'lorem en'], 'fr' => ['body' => 'lorem fr']]);
+        $block = $this->block->create([ 'name' => 'testBlock', 'en' => [ 'body' => 'lorem en' ], 'fr' => [ 'body' => 'lorem fr' ] ]);
         $blocks = $this->block->all();
 
         $this->assertCount(1, $blocks);
@@ -38,7 +38,7 @@ 
     /** @test */
     public function it_gets_block_by_name()
     {
-        $this->block->create(['name' => 'testblock', 'en' => ['body' => 'lorem en', 'online' => true], 'fr' => ['body' => 'lorem fr', 'online' => true]]);
+        $this->block->create([ 'name' => 'testblock', 'en' => [ 'body' => 'lorem en', 'online' => true ], 'fr' => [ 'body' => 'lorem fr', 'online' => true ] ]);
         $this->createRandomBlock(true, true);
         $this->createRandomBlock(true, true);
         $this->createRandomBlock(true, true);
@@ -51,7 +51,7 @@ 
     /** @test */
     public function it_gets_block_by_name_if_online()
     {
-        $this->block->create(['name' => 'testBlock', 'en' => ['body' => 'lorem en', 'online' => true], 'fr' => ['body' => 'lorem fr', 'online' => false]]);
+        $this->block->create([ 'name' => 'testBlock', 'en' => [ 'body' => 'lorem en', 'online' => true ], 'fr' => [ 'body' => 'lorem fr', 'online' => false ] ]);
         $this->createRandomBlock(true, true);
         $this->createRandomBlock(true, true);
         $this->createRandomBlock(true, true);
@@ -68,7 +68,7 @@ 
     /** @test */
     public function it_gets_block_by_facade()
     {
-        $this->block->create(['name' => 'testblock', 'en' => ['body' => 'lorem en', 'online' => true], 'fr' => ['body' => 'lorem fr', 'online' => false]]);
+        $this->block->create([ 'name' => 'testblock', 'en' => [ 'body' => 'lorem en', 'online' => true ], 'fr' => [ 'body' => 'lorem fr', 'online' => false ] ]);
         $this->createRandomBlock(true, true);
         $this->createRandomBlock(true, true);
 
@@ -80,15 +80,15 @@ 
     /** @test */
     public function it_slugifies_the_name_property()
     {
-        $block = $this->block->create(['name' => 'test block', 'en' => ['body' => 'lorem en', 'online' => true], 'fr' => ['body' => 'lorem fr', 'online' => false]]);
+        $block = $this->block->create([ 'name' => 'test block', 'en' => [ 'body' => 'lorem en', 'online' => true ], 'fr' => [ 'body' => 'lorem fr', 'online' => false ] ]);
 
         $this->assertEquals('test-block', $block->name);
     }
 
     public function it_makes_name_unique()
     {
-        $this->block->create(['name' => 'test block']);
-        $block = $this->block->create(['name' => 'test block']);
+        $this->block->create([ 'name' => 'test block' ]);
+        $block = $this->block->create([ 'name' => 'test block' ]);
 
         $this->assertEquals('test-block-1', $block->name);
     }
@@ -96,10 +96,10 @@ 
     /** @test */
     public function it_increments_name_if_not_unique()
     {
-        $this->block->create(['name' => 'test block']);
-        $this->block->create(['name' => 'test block']);
-        $block1 = $this->block->create(['name' => 'test block']);
-        $block2 = $this->block->create(['name' => 'test block']);
+        $this->block->create([ 'name' => 'test block' ]);
+        $this->block->create([ 'name' => 'test block' ]);
+        $block1 = $this->block->create([ 'name' => 'test block' ]);
+        $block2 = $this->block->create([ 'name' => 'test block' ]);
 
         $this->assertEquals('test-block-2', $block1->name);
         $this->assertEquals('test-block-3', $block2->name);
@@ -108,8 +108,8 @@ 
     /** @test */
     public function it_updates_block_without_name_change()
     {
-        $block = $this->block->create(['name' => 'test block']);
-        $this->block->update($block, ['name' => 'test-block']);
+        $block = $this->block->create([ 'name' => 'test block' ]);
+        $this->block->update($block, [ 'name' => 'test-block' ]);
 
         $this->assertEquals($block->name, 'test-block');
     }
@@ -117,8 +117,8 @@ 
     /** @test */
     public function it_updates_block_with_name_change()
     {
-        $block = $this->block->create(['name' => 'test block']);
-        $this->block->update($block, ['name' => 'my awesome block']);
+        $block = $this->block->create([ 'name' => 'test block' ]);
+        $this->block->update($block, [ 'name' => 'my awesome block' ]);
 
         $this->assertEquals($block->name, 'my-awesome-block');
     }