Merge pull request #28 from j0k3r/3/fix-error-over-capacity

Fix error when "Imgur is temporarily over capacity"

Author: j0k3r

.php_cs

@@ -1,14 +1,45 @@
 <?php
 
-$finder = PhpCsFixer\Finder::create()
-    ->in(__DIR__)
-    ->exclude(array('vendor'));
-
 return PhpCsFixer\Config::create()
-    ->setUsingCache(true)
+    ->setRiskyAllowed(true)
     ->setRules([
-        '@PSR2' => true,
-        'array_syntax' => ['syntax' => 'short'],
+        '@Symfony' => true,
+        '@Symfony:risky' => true,
+        'array_syntax' => [
+            'syntax' => 'short'
+        ],
+        'combine_consecutive_unsets' => true,
+        'heredoc_to_nowdoc' => true,
+        'no_extra_consecutive_blank_lines' => [
+        'break',
+            'continue',
+            'extra',
+            'return',
+            'throw',
+            'use',
+            'parenthesis_brace_block',
+            'square_brace_block',
+            'curly_brace_block'
+        ],
+        'no_unreachable_default_argument_value' => true,
+        'no_useless_else' => true,
+        'no_useless_return' => true,
+        'ordered_class_elements' => true,
+        'ordered_imports' => true,
+        'php_unit_strict' => true,
         'phpdoc_order' => true,
+        // 'psr4' => true,
+        'strict_comparison' => true,
+        'strict_param' => true,
+        'concat_space' => [
+            'spacing' => 'one'
+        ],
     ])
-    ->setFinder($finder);
+    ->setFinder(
+        PhpCsFixer\Finder::create()
+            ->exclude([
+                'vendor',
+            ])
+            ->in(__DIR__)
+    )
+;

.travis.yml

@@ -1,22 +1,20 @@
 language: php
 
 php:
-    - 5.5
     - 5.6
     - 7.0
     - 7.1
     - 7.2
+    - 7.3
     - nightly
-    - hhvm
 
 # run build against nightly but allow them to fail
 matrix:
     fast_finish: true
     include:
-        - php: 7.0
+        - php: 7.2
           env: CS_FIXER=run
     allow_failures:
-        - php: hhvm
         - php: nightly
 
 # faster builds on new travis setup not using sudo
@@ -32,10 +30,10 @@ install:
     - composer self-update
 
 before_script:
-    - composer install --prefer-dist --no-interaction
+    - composer install --prefer-dist --no-interaction -o
 
 script:
-    - phpunit --coverage-clover=coverage.clover
+    - ./vendor/bin/simple-phpunit -v --coverage-clover=coverage.clover
     - if [ "$CS_FIXER" = "run" ]; then php vendor/bin/php-cs-fixer fix --verbose --dry-run ; fi;
 
 after_script:

README.md

@@ -16,7 +16,7 @@ Uses [Imgur API v3](https://api.imgur.com/).
 * Branch [2.x](https://github.com/j0k3r/php-imgur-api-client/tree/2.x) use Guzzle 5
 * Branch [3.x](https://github.com/j0k3r/php-imgur-api-client/tree/3.x) use Guzzle 6
 
-All actives branches required PHP >= 5.5
+All actives branches required PHP >= 5.6
 
 ## Composer
 
@@ -54,7 +54,7 @@ if (isset($_SESSION['token'])) {
     $client->setAccessToken($_SESSION['token']);
 
     if ($client->checkAccessTokenExpired()) {
-          $client->refreshToken();
+        $client->refreshToken();
     }
 } elseif (isset($_GET['code'])) {
     $client->requestAccessToken($_GET['code']);

composer.json

@@ -34,13 +34,13 @@
         }
     },
     "require": {
-        "php": ">=5.5.0",
+        "php": ">=5.6.0",
         "ext-curl": "*",
         "guzzlehttp/guzzle": "^6.0"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^2",
-        "phpunit/phpunit": "^4.8|^6.5|^7.0"
+        "friendsofphp/php-cs-fixer": "^2.0",
+        "symfony/phpunit-bridge": "^4.0"
     },
     "extra": {
         "branch-alias": {

lib/Imgur/Api/AbstractApi.php

@@ -98,20 +98,6 @@ public function delete($url, $parameters = [])
         return $httpClient->parseResponse($response);
     }
 
-    /**
-     * Global method to validate an argument.
-     *
-     * @param string $type           The required parameter (used for the error message)
-     * @param string $input          Input value
-     * @param array  $possibleValues Possible values for this argument
-     */
-    private function validateArgument($type, $input, $possibleValues)
-    {
-        if (!in_array($input, $possibleValues, true)) {
-            throw new InvalidArgumentException($type . ' parameter "' . $input . '" is wrong. Possible values are: ' . implode(', ', $possibleValues));
-        }
-    }
-
     /**
      * Validate "sort" parameter and throw an exception if it's a bad value.
      *
@@ -144,4 +130,18 @@ protected function validateVoteArgument($vote, $possibleValues)
     {
         $this->validateArgument('Vote', $vote, $possibleValues);
     }
+
+    /**
+     * Global method to validate an argument.
+     *
+     * @param string $type           The required parameter (used for the error message)
+     * @param string $input          Input value
+     * @param array  $possibleValues Possible values for this argument
+     */
+    private function validateArgument($type, $input, $possibleValues)
+    {
+        if (!\in_array($input, $possibleValues, true)) {
+            throw new InvalidArgumentException($type . ' parameter "' . $input . '" is wrong. Possible values are: ' . implode(', ', $possibleValues));
+        }
+    }
 }