AsgardCms / Notification

Providers/RouteServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
      */
     protected function getBackendRoute()
     {
-        return __DIR__ . '/../Http/backendRoutes.php';
+        return __DIR__.'/../Http/backendRoutes.php';
     }
 
     /**
@@ -31,6 +31,6 @@ 
      */
     protected function getApiRoute()
     {
-        return __DIR__ . '/../Http/apiRoutes.php';
+        return __DIR__.'/../Http/apiRoutes.php';
     }
 }

Database/Migrations/2015_11_17_154243_create_notifications_table.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
      */
     public function up()
     {
-        Schema::create('notification__notifications', function (Blueprint $table) {
+        Schema::create('notification__notifications', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('user_id')->unsigned();
             $table->string('type')->nullable();

Events/BroadcastNotification.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function broadcastOn()
     {
-        return ['asgardcms.notifications.' . $this->notification->user_id];
+        return [ 'asgardcms.notifications.'.$this->notification->user_id ];
     }
 }

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
     /**
      * Get the channels the event should broadcast on.
-     * @return array
+     * @return string[]
      */
     public function broadcastOn()
     {

Http/backendRoutes.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use Illuminate\Routing\Router;
 
 /** @var Router $router */
-$router->group(['prefix' => '/notification'], function (Router $router) {
+$router->group([ 'prefix' => '/notification' ], function(Router $router) {
     $router->get('notifications', [
         'as' => 'admin.notification.notification.index',
         'uses' => 'NotificationsController@index',

Http/apiRoutes.php

Spacing +1 added lines, -1 removed lines

@@ -3,4 +3,4 @@
 use Illuminate\Routing\Router;
 
 /** @var Router $router */
-$router->post('notification/mark-read', ['as' => 'api.notification.read', 'uses' => 'NotificationsController@markAsRead']);
+$router->post('notification/mark-read', [ 'as' => 'api.notification.read', 'uses' => 'NotificationsController@markAsRead' ]);

Providers/NotificationServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -35,7 +35,7 @@ 
         $this->registerBindings();
         $this->registerViewComposers();
 
-        $this->app['events']->listen(
+        $this->app[ 'events' ]->listen(
             BuildingSidebar::class,
             $this->getSidebarClassForModule('blog', RegisterNotificationSidebar::class)
         );
@@ -45,7 +45,7 @@ 
     {
         $this->publishConfig('notification', 'config');
         $this->publishConfig('notification', 'permissions');
-        $this->loadMigrationsFrom(__DIR__ . '/../Database/Migrations');
+        $this->loadMigrationsFrom(__DIR__.'/../Database/Migrations');
     }
 
     /**
@@ -62,10 +62,10 @@ 
     {
         $this->app->bind(
             NotificationRepository::class,
-            function () {
+            function() {
                 $repository = new EloquentNotificationRepository(new Notification());
 
-                if (! config('app.cache')) {
+                if (!config('app.cache')) {
                     return $repository;
                 }
 
@@ -73,8 +73,8 @@ 
             }
         );
 
-        $this->app->bind(\Modules\Notification\Services\Notification::class, function ($app) {
-            return new AsgardNotification($app[NotificationRepository::class], $app[Authentication::class]);
+        $this->app->bind(\Modules\Notification\Services\Notification::class, function($app) {
+            return new AsgardNotification($app[ NotificationRepository::class ], $app[ Authentication::class ]);
         });
     }
 

Tests/BaseTestCase.php

Spacing +5 added lines, -5 removed lines

@@ -40,15 +40,15 @@
 
     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('modules.paths.modules', __DIR__ . '/../Modules');
+        $app[ 'config' ]->set('translatable.locales', [ 'en', 'fr' ]);
+        $app[ 'config' ]->set('modules.paths.modules', __DIR__.'/../Modules');
     }
 
     private function resetDatabase()

Tests/EloquentNotificationRepositoryTest.php

Spacing +18 added lines, -18 removed lines

@@ -30,11 +30,11 @@ 
         ]);
 
         $this->assertCount(1, $this->notification->all());
-        $this->assertEquals('1' , $notification->user_id);
-        $this->assertEquals('fa fa-link' , $notification->icon_class);
-        $this->assertEquals('http://localhost/users' , $notification->link);
-        $this->assertEquals('My notification' , $notification->title);
-        $this->assertEquals('Is awesome!' , $notification->message);
+        $this->assertEquals('1', $notification->user_id);
+        $this->assertEquals('fa fa-link', $notification->icon_class);
+        $this->assertEquals('http://localhost/users', $notification->link);
+        $this->assertEquals('My notification', $notification->title);
+        $this->assertEquals('Is awesome!', $notification->message);
     }
 
     /** @test */
@@ -51,8 +51,8 @@ 
         $this->createNotification();
         $this->createNotification();
         $this->createNotification();
-        $this->createNotification(['user_id' => 2]);
-        $this->createNotification(['user_id' => 2]);
+        $this->createNotification([ 'user_id' => 2 ]);
+        $this->createNotification([ 'user_id' => 2 ]);
 
         $this->assertCount(10, $this->notification->latestForUser(1));
         $this->assertCount(2, $this->notification->latestForUser(2));
@@ -72,8 +72,8 @@ 
         $this->createNotification();
         $this->createNotification();
         $this->createNotification();
-        $this->createNotification(['user_id' => 2]);
-        $this->createNotification(['user_id' => 2]);
+        $this->createNotification([ 'user_id' => 2 ]);
+        $this->createNotification([ 'user_id' => 2 ]);
 
         $this->assertCount(11, $this->notification->allForUser(1));
         $this->assertCount(2, $this->notification->allForUser(2));
@@ -95,8 +95,8 @@ 
     public function it_can_get_all_unread_notifications_for_user()
     {
         $this->createNotification();
-        $this->createNotification(['is_read' => true]);
-        $this->createNotification(['is_read' => true]);
+        $this->createNotification([ 'is_read' => true ]);
+        $this->createNotification([ 'is_read' => true ]);
         $this->createNotification();
 
         $this->assertCount(2, $this->notification->allUnreadForUser(1));
@@ -106,8 +106,8 @@ 
     public function it_can_get_all_read_notifications_for_user()
     {
         $this->createNotification();
-        $this->createNotification(['is_read' => true]);
-        $this->createNotification(['is_read' => true]);
+        $this->createNotification([ 'is_read' => true ]);
+        $this->createNotification([ 'is_read' => true ]);
         $this->createNotification();
         $this->createNotification();
 
@@ -120,8 +120,8 @@ 
         $this->createNotification();
         $this->createNotification();
         $this->createNotification();
-        $this->createNotification(['user_id' => 2]);
-        $this->createNotification(['user_id' => 2]);
+        $this->createNotification([ 'user_id' => 2 ]);
+        $this->createNotification([ 'user_id' => 2 ]);
 
         $this->assertCount(3, $this->notification->allForUser(1));
         $this->assertCount(2, $this->notification->allForUser(2));
@@ -138,8 +138,8 @@ 
         $this->createNotification();
         $this->createNotification();
         $this->createNotification();
-        $this->createNotification(['user_id' => 2]);
-        $this->createNotification(['user_id' => 2]);
+        $this->createNotification([ 'user_id' => 2 ]);
+        $this->createNotification([ 'user_id' => 2 ]);
 
         $this->assertCount(0, $this->notification->allReadForUser(1));
 
@@ -147,7 +147,7 @@ 
         $this->assertCount(3, $this->notification->allReadForUser(1));
     }
 
-    private function createNotification(array $properties = []) : Notification
+    private function createNotification(array $properties = [ ]) : Notification
     {
         $data = [
             'user_id' => 1,

Repositories/Cache/CacheNotificationDecorator.php

Spacing +8 added lines, -8 removed lines

@@ -21,11 +21,11 @@ 
     public function latestForUser($userId)
     {
         return $this->cache
-            ->tags([$this->entityName, 'global'])
+            ->tags([ $this->entityName, 'global' ])
             ->remember(
                 "{$this->locale}.{$this->entityName}.latestForUser.{$userId}",
                 $this->cacheTime,
-                function () use ($userId) {
+                function() use ($userId) {
                     return $this->repository->latestForUser($userId);
                 }
             );
@@ -51,11 +51,11 @@ 
     public function allForUser($userId)
     {
         return $this->cache
-            ->tags([$this->entityName, 'global'])
+            ->tags([ $this->entityName, 'global' ])
             ->remember(
                 "{$this->locale}.{$this->entityName}.allForUser.{$userId}",
                 $this->cacheTime,
-                function () use ($userId) {
+                function() use ($userId) {
                     return $this->repository->allForUser($userId);
                 }
             );
@@ -69,11 +69,11 @@ 
     public function allReadForUser($userId)
     {
         return $this->cache
-            ->tags([$this->entityName, 'global'])
+            ->tags([ $this->entityName, 'global' ])
             ->remember(
                 "{$this->locale}.{$this->entityName}.allReadForUser.{$userId}",
                 $this->cacheTime,
-                function () use ($userId) {
+                function() use ($userId) {
                     return $this->repository->allReadForUser($userId);
                 }
             );
@@ -87,11 +87,11 @@ 
     public function allUnreadForUser($userId)
     {
         return $this->cache
-            ->tags([$this->entityName, 'global'])
+            ->tags([ $this->entityName, 'global' ])
             ->remember(
                 "{$this->locale}.{$this->entityName}.allUnreadForUser.{$userId}",
                 $this->cacheTime,
-                function () use ($userId) {
+                function() use ($userId) {
                     return $this->repository->allUnreadForUser($userId);
                 }
             );