projectmentor / laravel-quota

database/migrations/2017_05_01_015818_create_quotalog_table.php

Spacing +2 added lines, -2 removed lines

@@ -30,10 +30,10 @@
      */
     public function up()
     {
-        Schema::create('quotalog', function (Blueprint $table) {
+        Schema::create('quotalog', function(Blueprint $table) {
             $table->increments('id');
             $table->date('date');
-            $table->string('connection');   //@See config/quota.php
+            $table->string('connection'); //@See config/quota.php
             $table->integer('hits')->default(0);
             $table->integer('misses')->default(0);
 

config/quota.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 
         'bandwidth' => [
             'limit' => env('QUOTA_BANDWIDTH_LIMIT', 60),
-            'period' => env('QUOTA_BANDWIDTH_PERIOD','second'),
+            'period' => env('QUOTA_BANDWIDTH_PERIOD', 'second'),
             'driver' => 'quota.storage.file',
             'path' => env('QUOTA_BANDWIDTH_PATH', '/tmp/bandwidth.quota'),
             'capacity' => env('QUOTA_BANDWIDTH_CAPACITY', 60),
@@ -44,7 +44,7 @@ 
 
         'daily' => [
             'limit' => env('QUOTA_DAILY_LIMIT', 2500),
-            'period' => env('QUOTA_DAILY_PERIOD','day'),
+            'period' => env('QUOTA_DAILY_PERIOD', 'day'),
             'log_table' => env('QUOTA_DAILY_LOG_TABLE', 'quotalog'),
             'timezone' => env('QUOTA_DAILY_TIMEZONE', 'America/New_York'),
         ],

src/Contracts/Quota.php

Doc Comments +1 added lines

@@ -18,6 +18,7 @@
 {
     /**
      * Enforce a quota.
+     * @return boolean
      */
     public function enforce();
 }

src/Quota.php

Indentation +6 added lines, -6 removed lines

@@ -99,12 +99,12 @@
         return $result;
     }
 
-   /**
-    * Public interface.
-    *
-    * NOTE: this class and method are really abstract,
-    * but we leave it instantiable for easier testing.
-    */
+    /**
+     * Public interface.
+     *
+     * NOTE: this class and method are really abstract,
+     * but we leave it instantiable for easier testing.
+     */
     public function enforce()
     {
         //Override this in descendants.

Spacing +7 added lines, -7 removed lines

@@ -60,11 +60,11 @@ 
     public function __construct($connection)
     {
         $this->connection = $connection;
-        $this->index = 'quota.connections.' . $connection;
+        $this->index = 'quota.connections.'.$connection;
 
-        $this->limit = config($this->index . '.limit');
+        $this->limit = config($this->index.'.limit');
 
-        $period = config($this->index . '.period');
+        $period = config($this->index.'.period');
         $this->setPeriod($period);
     }
 
@@ -194,14 +194,14 @@ 
      */
     public function setPeriod($period)
     {
-        if (! $this->validatePeriod($period)) {
+        if (!$this->validatePeriod($period)) {
             $expected = $this->validPeriods();
 
             throw new \InvalidArgumentException(
                 __CLASS__.'::'.__FUNCTION__.
-                ' Invalid period: ' . $period .
-                ' in connection configuration: ' . $this->connection .
-                ' expected one of: ' . PHP_EOL . print_r($expected, 1)
+                ' Invalid period: '.$period.
+                ' in connection configuration: '.$this->connection.
+                ' expected one of: '.PHP_EOL.print_r($expected, 1)
             );
         }
 

src/BandwidthQuota.php

Spacing +8 added lines, -8 removed lines

@@ -87,14 +87,14 @@ 
     {
         parent::__construct($connection);
 
-        $this->driver = config($this->index . '.driver');
+        $this->driver = config($this->index.'.driver');
 
         //TODO: REFACTOR for multiple storage types.
         if ($this->driver != 'quota.storage.file') {
-            throw new \Exception('Driver: ' . $this->driver . ' not supported.');
+            throw new \Exception('Driver: '.$this->driver.' not supported.');
         }
 
-        $this->path = config($this->index . '.path');
+        $this->path = config($this->index.'.path');
         $this->storage = app($this->driver, ['path' =>  $this->path]);
         //END REFACTOR
 
@@ -105,7 +105,7 @@ 
         );
 
         //Resolve bucket via IoC container.
-        $capacity = config($this->index . '.capacity');
+        $capacity = config($this->index.'.capacity');
         $this->bucket = app('quota.bucket', [
             'capacity' => $capacity,
             'rate' => $this->rate,
@@ -116,7 +116,7 @@ 
         $this->bucket->bootstrap($capacity);
 
         //Optionally enclose bucket within a blocking consumer.
-        if (config($this->index . '.block') == true) {
+        if (config($this->index.'.block') == true) {
             $this->blocker = app('quota.blocker', ['bucket' => $this->bucket]);
         }
     }
@@ -177,12 +177,12 @@ 
     public function consume($tokens = 1)
     {
         if (isset($this->bucket)) {
-            if (! isset($this->blocker)) {
+            if (!isset($this->blocker)) {
                 $result = $this->bucket->consume($tokens, $seconds);
                 if ($result === false) {
                     throw new \ErrorException(
-                        __CLASS__ . '::' . __FUNCTION__ .
-                        ' Overquota. Exceeded bandwidth. Wait ' . $seconds . ' seconds.'
+                        __CLASS__.'::'.__FUNCTION__.
+                        ' Overquota. Exceeded bandwidth. Wait '.$seconds.' seconds.'
                     );
                 }
             } else {

src/QuotaServiceProvider.php

Indentation +17 added lines, -17 removed lines

@@ -87,15 +87,15 @@ 
         }
     }
 
-     /**
-     * Register a database migration path.
-     *
-     * NOTE: This function is already part of 5.4
-     * it is backported here for 5.2
-     *
-     * @param  array|string  $paths
-     * @return void
-     */
+        /**
+         * Register a database migration path.
+         *
+         * NOTE: This function is already part of 5.4
+         * it is backported here for 5.2
+         *
+         * @param  array|string  $paths
+         * @return void
+         */
     //protected function loadMigrationsFrom($paths)
     //{
     //    dump(__CLASS__.'::'.__FUNCTION__);
@@ -146,20 +146,20 @@ 
     public function bindInterfaces()
     {
         $this->app->when(RateFactory::class)
-          ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
-          ->give(RateData::class);
+            ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
+            ->give(RateData::class);
 
         $this->app->when(FileStorageFactory::class)
-          ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
-          ->give(FileStorageData::class);
+            ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
+            ->give(FileStorageData::class);
 
         $this->app->when(TokenBucketFactory::class)
-          ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
-          ->give(TokenBucketData::class);
+            ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
+            ->give(TokenBucketData::class);
 
         $this->app->when(BlockingConsumerFactory::class)
-          ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
-          ->give(BlockingConsumerData::class);
+            ->needs('Projectmentor\Quota\Contracts\PayloadInterface')
+            ->give(BlockingConsumerData::class);
     }
 
     /**

Spacing +8 added lines, -8 removed lines

@@ -168,19 +168,19 @@ 
      */
     protected function registerFactories()
     {
-        $this->app->singleton('quota.factory.rate', function ($app) {
+        $this->app->singleton('quota.factory.rate', function($app) {
             return new RateFactory;
         });
 
-        $this->app->singleton('quota.factory.storage.file', function ($app) {
+        $this->app->singleton('quota.factory.storage.file', function($app) {
             return new FileStorageFactory;
         });
 
-        $this->app->singleton('quota.factory.tokenbucket', function ($app) {
+        $this->app->singleton('quota.factory.tokenbucket', function($app) {
             return new TokenBucketFactory;
         });
 
-        $this->app->singleton('quota.factory.blockingconsumer', function ($app) {
+        $this->app->singleton('quota.factory.blockingconsumer', function($app) {
             return new BlockingConsumerFactory;
         });
 
@@ -208,19 +208,19 @@ 
      */
     protected function registerClasses()
     {
-        $this->app->bind('quota.storage.file', function ($app, $params) {
+        $this->app->bind('quota.storage.file', function($app, $params) {
             return new FileStorage($params['path']);
         });
 
-        $this->app->bind('quota.rate', function ($app, $params) {
+        $this->app->bind('quota.rate', function($app, $params) {
             return new Rate($params['limit'], $params['period']);
         });
 
-        $this->app->bind('quota.bucket', function ($app, $params) {
+        $this->app->bind('quota.bucket', function($app, $params) {
             return new TokenBucket($params['capacity'], $params['rate'], $params['storage']);
         });
 
-        $this->app->bind('quota.blocker', function ($app, $params) {
+        $this->app->bind('quota.blocker', function($app, $params) {
             return new BlockingConsumer($params['bucket']);
         });
 

src/Console/Commands/ResetQuotaLog.php

Indentation +17 added lines, -17 removed lines

@@ -80,23 +80,23 @@
                 $changes = DB::select('SELECT changes()');
                 if (! $this->sqliteUpdateDidChange($changes)) {
                     $sql = 'INSERT INTO quotalog' .
-                      ' (' .
-                      ' `date`' .
-                      ', `connection`' .
-                      ', `hits`' .
-                      ', `misses`' .
-                      ', `created_at`' .
-                      ', `updated_at`' .
-                      ')' .
-                      ' VALUES' .
-                      ' (' .
-                      ' :date' .
-                      ', :connection' .
-                      ', :hits' .
-                      ', :misses' .
-                      ', :created_at' .
-                      ', :updated_at' .
-                      ')';
+                        ' (' .
+                        ' `date`' .
+                        ', `connection`' .
+                        ', `hits`' .
+                        ', `misses`' .
+                        ', `created_at`' .
+                        ', `updated_at`' .
+                        ')' .
+                        ' VALUES' .
+                        ' (' .
+                        ' :date' .
+                        ', :connection' .
+                        ', :hits' .
+                        ', :misses' .
+                        ', :created_at' .
+                        ', :updated_at' .
+                        ')';
 
                     $this->doUpsertStatement($sql);
                 }

Spacing +34 added lines, -34 removed lines

@@ -50,13 +50,13 @@ 
     {
         switch (DB::connection()->getPDO()->getAttribute(\PDO::ATTR_DRIVER_NAME)) {
             case 'mysql':
-                $sql = 'INSERT INTO quotalog' .
-                ' (date, connection, hits, misses, created_at, updated_at)' .
-                ' VALUES' .
-                ' ( :date, :connection, :hits, :misses, :created_at, :updated_at)' .
-                ' ON DUPLICATE KEY UPDATE' .
-                ' hits = VALUES(hits),' .
-                ' misses = VALUES(misses),' .
+                $sql = 'INSERT INTO quotalog'.
+                ' (date, connection, hits, misses, created_at, updated_at)'.
+                ' VALUES'.
+                ' ( :date, :connection, :hits, :misses, :created_at, :updated_at)'.
+                ' ON DUPLICATE KEY UPDATE'.
+                ' hits = VALUES(hits),'.
+                ' misses = VALUES(misses),'.
                 ' updated_at = VALUES(updated_at)';
 
                 $this->doUpsertStatement($sql);
@@ -66,36 +66,36 @@ 
                 $date = $this->argument('date');
                 $connection = $this->argument('connection');
 
-                $sql = 'UPDATE quotalog' .
-                ' SET `date` = :date' .
-                ', `connection` = :connection' .
-                ', `hits` = :hits' .
-                ', `misses` = :misses' .
-                ', `updated_at` = :updated_at' .
-                ' WHERE `date` = ' . '\''. $date . '\'' .
-                ' AND `connection` = ' . '\'' . $connection . '\'';
+                $sql = 'UPDATE quotalog'.
+                ' SET `date` = :date'.
+                ', `connection` = :connection'.
+                ', `hits` = :hits'.
+                ', `misses` = :misses'.
+                ', `updated_at` = :updated_at'.
+                ' WHERE `date` = '.'\''.$date.'\''.
+                ' AND `connection` = '.'\''.$connection.'\'';
 
                 $this->doUpdateStatement($sql);
 
                 $changes = DB::select('SELECT changes()');
-                if (! $this->sqliteUpdateDidChange($changes)) {
-                    $sql = 'INSERT INTO quotalog' .
-                      ' (' .
-                      ' `date`' .
-                      ', `connection`' .
-                      ', `hits`' .
-                      ', `misses`' .
-                      ', `created_at`' .
-                      ', `updated_at`' .
-                      ')' .
-                      ' VALUES' .
-                      ' (' .
-                      ' :date' .
-                      ', :connection' .
-                      ', :hits' .
-                      ', :misses' .
-                      ', :created_at' .
-                      ', :updated_at' .
+                if (!$this->sqliteUpdateDidChange($changes)) {
+                    $sql = 'INSERT INTO quotalog'.
+                      ' ('.
+                      ' `date`'.
+                      ', `connection`'.
+                      ', `hits`'.
+                      ', `misses`'.
+                      ', `created_at`'.
+                      ', `updated_at`'.
+                      ')'.
+                      ' VALUES'.
+                      ' ('.
+                      ' :date'.
+                      ', :connection'.
+                      ', :hits'.
+                      ', :misses'.
+                      ', :created_at'.
+                      ', :updated_at'.
                       ')';
 
                     $this->doUpsertStatement($sql);
@@ -105,7 +105,7 @@ 
             default:
                 throw new \ErrorException(
                     __CLASS__.'::'.__FUNCTION__.
-                    ' Driver: ' . $driver . ' not supported.'
+                    ' Driver: '.$driver.' not supported.'
                 );
         }
     }

src/Helpers/MigrateTrait.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
             //run specific files
 
             foreach ($files as $file) {
-                $file = $this->migrations_path . "/" . $file;
+                $file = $this->migrations_path."/".$file;
 
                 $fileSystem->requireOnce($file);
                 $migrationClass = $classFinder->findClass($file);