Merge branch 'feature/laravel-7' of github.com:dingo/api into feat/laravel-7-support

Author: Harlan Wilton

src/Exception/Handler.php

@@ -5,17 +5,20 @@
 use Dingo\Api\Http\Request;
 use Exception;
 use Illuminate\Database\Eloquent\ModelNotFoundException;
-use ReflectionFunction;
 use Illuminate\Support\Str;
 use Illuminate\Http\Response;
 use Dingo\Api\Contract\Debug\ExceptionHandler;
 use Dingo\Api\Contract\Debug\MessageBagErrors;
+use Exception;
+use Illuminate\Contracts\Debug\ExceptionHandler as IlluminateExceptionHandler;
+use Illuminate\Http\Response;
+use Illuminate\Support\Str;
 use Illuminate\Validation\ValidationException;
+use ReflectionFunction;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\HttpFoundation\Response as BaseResponse;
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
-use Illuminate\Contracts\Debug\ExceptionHandler as IlluminateExceptionHandler;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Throwable;
 
 class Handler implements ExceptionHandler, IlluminateExceptionHandler
@@ -161,11 +164,11 @@ public function handle($exception)
                     $response = new Response($response, $this->getExceptionStatusCode($exception));
                 }
 
-                return $response;
+                return $response->withException($exception);
             }
         }
 
-        return $this->genericResponse($exception);
+        return $this->genericResponse($exception)->withException($exception);
     }
 
     /**

src/Http/Response/Factory.php

@@ -123,7 +123,12 @@ public function collection(Collection $collection, $transformer, $parameters = [
      */
     public function item($item, $transformer, $parameters = [], Closure $after = null)
     {
-        $class = get_class($item);
+        // Check for $item being null
+        if (! is_null($item)) {
+            $class = get_class($item);
+        } else {
+            $class = \StdClass::class;
+        }
 
         if ($parameters instanceof \Closure) {
             $after = $parameters;

src/Routing/Route.php

@@ -511,8 +511,8 @@ public function methods()
      */
     public function httpOnly()
     {
-        return in_array('https', $this->action, true)
-            || (array_key_exists('https', $this->action) && $this->action['https']);
+        return in_array('http', $this->action, true)
+            || (array_key_exists('http', $this->action) && $this->action['http']);
     }
 
     /**
@@ -532,7 +532,8 @@ public function httpsOnly()
      */
     public function secure()
     {
-        return in_array('https', $this->action, true);
+        return in_array('https', $this->action, true)
+            || (array_key_exists('https', $this->action) && $this->action['https']);
     }
 
     /**

tests/Exception/HandlerTest.php

@@ -2,15 +2,15 @@
 
 namespace Dingo\Api\Tests\Exception;
 
-use Mockery as m;
-use RuntimeException;
-use Illuminate\Http\Response;
-use PHPUnit\Framework\TestCase;
 use Dingo\Api\Exception\Handler;
+use Dingo\Api\Exception\ResourceException;
+use Dingo\Api\Http\Request as ApiRequest;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\RedirectResponse;
-use Dingo\Api\Http\Request as ApiRequest;
-use Dingo\Api\Exception\ResourceException;
+use Illuminate\Http\Response;
+use Mockery as m;
+use PHPUnit\Framework\TestCase;
+use RuntimeException;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 
 class HandlerTest extends TestCase
@@ -55,6 +55,7 @@ public function testExceptionHandlerHandlesException()
 
         $this->assertSame('foo', $response->getContent());
         $this->assertSame(404, $response->getStatusCode());
+        $this->assertSame($exception, $response->exception);
     }
 
     public function testExceptionHandlerHandlesExceptionAndCreatesNewResponse()
@@ -142,6 +143,7 @@ public function testRegularExceptionsAreHandledByGenericHandler()
 
         $this->assertSame('{"message":"Uh oh","status_code":500}', $response->getContent());
         $this->assertSame(500, $response->getStatusCode());
+        $this->assertSame($exception, $response->exception);
     }
 
     public function testResourceExceptionErrorsAreIncludedInResponse()