mpociot / captainhook

src/database/2015_10_29_000000_captain_hook_setup_table.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function up()
     {
-        Schema::create('webhooks', function (Blueprint $table) {
+        Schema::create('webhooks', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('tenant_id')->nullable();
             $table->string('url');

src/database/2015_10_29_000001_captain_hook_setup_logs_table.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function up()
     {
-        Schema::create('webhook_logs', function (Blueprint $table) {
+        Schema::create('webhook_logs', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('webhook_id')->unsigned()->nullable();
             $table->foreign('webhook_id')->references('id')->on('webhooks')->onDelete('set null');

src/Mpociot/CaptainHook/Jobs/TriggerWebhooksJob.php

Doc Comments +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
     /**
      * Create a new job instance.
      *
-     * @param array|\Illuminate\Support\Collection $webhooks
+     * @param \Illuminate\Support\Collection $webhooks
      * @param mixed $eventData
      */
     public function __construct($webhooks, $eventData)
@@ -48,7 +48,7 @@ 
     /**
      * Resolves a string or callable to a valid callable.
      * @param string|callable $transformer
-     * @param $defaultMethodName
+     * @param string $defaultMethodName
      * @return callable
      */
     private function resolveCallable($transformer, $defaultMethodName)

Spacing +9 added lines, -9 removed lines

@@ -54,7 +54,7 @@ 
     private function resolveCallable($transformer, $defaultMethodName)
     {
         if (is_string($transformer)) {
-            return function () use ($transformer, $defaultMethodName) {
+            return function() use ($transformer, $defaultMethodName) {
                 list($class, $method) = Str::parseCallback($transformer, $defaultMethodName);
 
                 return call_user_func_array([app($class), $method], func_get_args());
@@ -63,7 +63,7 @@ 
             return $transformer;
         }
 
-        return function () {};
+        return function() {};
     }
 
     /**
@@ -87,14 +87,14 @@ 
                     $webhook->logs()->orderBy('updated_at', 'desc')->first()->delete();
                 }
                 $log = new WebhookLog([
-                    'webhook_id' => $webhook[ 'id' ],
-                    'url' => $webhook[ 'url' ],
+                    'webhook_id' => $webhook['id'],
+                    'url' => $webhook['url'],
                 ]);
-                $middleware = Middleware::tap(function (RequestInterface $request, $options) use ($log) {
+                $middleware = Middleware::tap(function(RequestInterface $request, $options) use ($log) {
                     $log->payload_format = isset($request->getHeader('Content-Type')[0]) ? $request->getHeader('Content-Type')[0] : null;
                     $log->payload = $request->getBody()->getContents();
-                }, function ($request, $options, PromiseInterface $response) use ($log, $webhook, $responseCallback) {
-                    $response->then(function (ResponseInterface $response) use ($log, $webhook, $responseCallback) {
+                }, function($request, $options, PromiseInterface $response) use ($log, $webhook, $responseCallback) {
+                    $response->then(function(ResponseInterface $response) use ($log, $webhook, $responseCallback) {
                         $log->status = $response->getStatusCode();
                         $log->response = $response->getBody()->getContents();
                         $log->response_format = $log->payload_format = isset($response->getHeader('Content-Type')[0]) ? $response->getHeader('Content-Type')[0] : null;
@@ -110,14 +110,14 @@ 
                     });
                 });
 
-                $client->post($webhook[ 'url' ], [
+                $client->post($webhook['url'], [
                     'exceptions' => false,
                     'body' => $transformer($this->eventData, $webhook),
                     'verify' => false,
                     'handler' => $middleware($client->getConfig('handler')),
                 ]);
             } else {
-                $client->post($webhook[ 'url' ], [
+                $client->post($webhook['url'], [
                     'exceptions' => false,
                     'body' => $transformer($this->eventData, $webhook),
                     'verify' => false,

src/config/config.php

Spacing +3 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
     | The tenant_id field can be used for verification.
     |
     */
-    'filter' => function ($webhook) {
+    'filter' => function($webhook) {
         return true;
     },
 
@@ -48,7 +48,7 @@ 
     | You can also use the 'Foo\Class@transform' notation if you want.
     |
     */
-    'transformer' => function ($eventData, $webhook) {
+    'transformer' => function($eventData, $webhook) {
         return json_encode($eventData);
     },
 
@@ -65,7 +65,7 @@ 
     | You can also use the 'Foo\Class@handle' notation if you want.
     |
     */
-    'response_callback' => function ($webhook, $response) {
+    'response_callback' => function($webhook, $response) {
         // Handle custom response status codes, ...
     },
 

src/Mpociot/CaptainHook/Http/WebhookController.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      * Create a new webhook for the user.
      *
      * @param  CreateWebhookRequest  $request
-     * @return Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function store(CreateWebhookRequest $request)
     {

src/Mpociot/CaptainHook/Http/WebhookEventsController.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * Get all of the available webhook events.
      *
-     * @return Response
+     * @return \Illuminate\Support\Collection
      */
     public function all(Request $request)
     {

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      */
     public function all(Request $request)
     {
-        return collect(config('captain_hook.listeners', []))->transform(function ($key, $value) {
+        return collect(config('captain_hook.listeners', []))->transform(function($key, $value) {
             return [
                 'name' => $value,
                 'event' => $key

src/routes.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-Route::group(['middleware' => 'web'], function ($router) {
+Route::group(['middleware' => 'web'], function($router) {
     $router->get('/settings/api/webhooks', 'Mpociot\CaptainHook\Http\WebhookController@all');
     $router->post('/settings/api/webhook', 'Mpociot\CaptainHook\Http\WebhookController@store');
     $router->put('/settings/api/webhook/{webhook_id}', 'Mpociot\CaptainHook\Http\WebhookController@update');

src/Mpociot/CaptainHook/Webhook.php

Spacing +4 added lines, -4 removed lines

@@ -31,15 +31,15 @@ 
     {
         parent::boot();
 
-        static::created(function ($results) {
+        static::created(function($results) {
             Cache::forget(self::CACHE_KEY);
         });
 
-        static::updated(function ($results) {
+        static::updated(function($results) {
             Cache::forget(self::CACHE_KEY);
         });
 
-        static::deleted(function ($results) {
+        static::deleted(function($results) {
             Cache::forget(self::CACHE_KEY);
         });
     }
@@ -61,6 +61,6 @@ 
      */
     public function lastLog()
     {
-        return $this->hasOne(WebhookLog::class)->orderBy('created_at','DESC');
+        return $this->hasOne(WebhookLog::class)->orderBy('created_at', 'DESC');
     }
 }

src/Mpociot/CaptainHook/CaptainHookServiceProvider.php

Spacing +10 added lines, -10 removed lines

@@ -112,7 +112,7 @@ 
 
         $this->publishes([
             __DIR__.'/../../resources/assets/js/' => base_path('resources/assets/js/components/'),
-            __DIR__ . '/../../resources/views/' => base_path('resources/views/vendor/captainhook/settings/'),
+            __DIR__.'/../../resources/views/' => base_path('resources/views/vendor/captainhook/settings/'),
         ], 'spark-resources');
     }
 
@@ -122,7 +122,7 @@ 
     protected function registerEventListeners()
     {
         foreach ($this->listeners as $eventName) {
-            $this->app[ 'events' ]->listen($eventName, [$this, 'handleEvent']);
+            $this->app['events']->listen($eventName, [$this, 'handleEvent']);
         }
     }
 
@@ -142,7 +142,7 @@ 
     public function setWebhooks($webhooks)
     {
         $this->webhooks = $webhooks;
-        $this->getCache()->rememberForever(Webhook::CACHE_KEY, function () {
+        $this->getCache()->rememberForever(Webhook::CACHE_KEY, function() {
             return $this->webhooks;
         });
     }
@@ -152,8 +152,8 @@ 
      */
     public function getWebhooks()
     {
-        if (! $this->getCache()->has(Webhook::CACHE_KEY)) {
-            $this->getCache()->rememberForever(Webhook::CACHE_KEY, function () {
+        if (!$this->getCache()->has(Webhook::CACHE_KEY)) {
+            $this->getCache()->rememberForever(Webhook::CACHE_KEY, function() {
                 return Webhook::all();
             });
         }
@@ -204,7 +204,7 @@ 
         $webhooks = $this->getWebhooks()->where('event', $eventName);
         $webhooks = $webhooks->filter($this->config->get('captain_hook.filter', null));
 
-        if (! $webhooks->isEmpty()) {
+        if (!$webhooks->isEmpty()) {
             $this->dispatch(new TriggerWebhooksJob($webhooks, $eventData));
         }
     }
@@ -214,15 +214,15 @@ 
      */
     protected function registerCommands()
     {
-        $this->app[ 'hook.list' ] = $this->app->share(function () {
+        $this->app['hook.list'] = $this->app->share(function() {
             return new ListWebhooks();
         });
 
-        $this->app[ 'hook.add' ] = $this->app->share(function () {
+        $this->app['hook.add'] = $this->app->share(function() {
             return new AddWebhook();
         });
 
-        $this->app[ 'hook.delete' ] = $this->app->share(function () {
+        $this->app['hook.delete'] = $this->app->share(function() {
             return new DeleteWebhook();
         });
 
@@ -238,7 +238,7 @@ 
     protected function registerRoutes()
     {
         if (class_exists('Laravel\Spark\Providers\AppServiceProvider')) {
-            include __DIR__ . '/../../routes.php';
+            include __DIR__.'/../../routes.php';
         }
     }
 }