Merge pull request #16 from trms/#13-ModelRequest-can-take-grandchildren

#13 ModelRequest should accept grandchildren

Author: sethphillips

Tests/ModelRequestTest.php

@@ -3,6 +3,7 @@
 use TRMS\Carousel\Requests\ModelRequest;
 use TRMS\Carousel\Exceptions\CarouselRequestException;
 use TRMS\Carousel\Models\Bulletin;
+use TRMS\Carousel\Models\BulletinTag;
 
 
 class ModelRequestTest extends PHPUnit_Framework_TestCase
@@ -43,4 +44,10 @@ function test_when_passed_an_id_and_other_params_it_throws_an_exception()
     }
     $this->fail('the exception was not thrown');
   }
+
+  function test_it_can_handle_a_class_with_CarouselModel_as_a_grandparent()
+  {
+    $request = \Mockery::mock(ModelRequest::class,[BulletinTag::class])->makePartial();
+    $this->assertEquals('bulletintags',$request->url());
+  }
 }

src/Requests/ModelRequest.php

@@ -12,7 +12,7 @@ public function __construct(string $responseClassName, Array $params=[])
   {
     $responseClass = new $responseClassName();
 
-    if(get_parent_class($responseClass) !== CarouselModel::class){
+    if(get_parent_class($responseClass) !== CarouselModel::class && get_parent_class(get_parent_class($responseClass)) !== CarouselModel::class ){
       throw new CarouselRequestException('The classname passed in to the ModelRequest must be a child of CarouselModel');
     }