Boolean check added along with exception thrown if variable not a boolean

2 files changed, 19 insertions, 1 deletions

diff --git a/library/SSRS/Report.php b/library/SSRS/Report.php
index c12a325..9b3f890 100755
--- a/library/SSRS/Report.php
+++ b/library/SSRS/Report.php
@@ -189,7 +189,10 @@ class SSRS_Report {
      * @param boolean $recursive
      * @return SSRS_Object_CatalogItems
      */
-    public function listChildren($itemPath, $recursive = false) {
+    public function listChildren($itemPath, $recursive=false) {
+        if (!is_bool($recursive)) {
+            throw new SSRS_Report_Exception('Recursive Must Be a Boolean');
+        }
         $params = array(
             'ItemPath' => $itemPath,
             'Recursive' => $recursive
diff --git a/tests/library/SSRS/ReportTest.php b/tests/library/SSRS/ReportTest.php
index 68a41de..a0e1245 100755
--- a/tests/library/SSRS/ReportTest.php
+++ b/tests/library/SSRS/ReportTest.php
@@ -83,6 +83,21 @@ class SSRS_ReportTest extends PHPUnit_Framework_TestCase {
         $this->assertEquals($expected, $result);
     }
 
+    public function testLoadChildrenCheckRecursiveParameterIsSetAndIsBoolean() {
+        $soapMock = $this->getMockFromWsdl(dirname(__FILE__) . '/ReportTest/ReportService2010.wsdl', 'SoapClientMockChildren2');
+
+        $recursiveParam = true;
+
+        $soapMock->expects($this->any())
+                ->method('ListChildren')
+                ->with($this->equalTo(array('ItemPath' => '/Reports', 'Recursive' => true)));
+
+        $ssrs = new SSRS_Report('http://test/ReportServer');
+        $ssrs->setSoapService($soapMock);
+
+        $result = $ssrs->listChildren('/Reports', $recursiveParam);
+    }
+
     public function testLoadItemDefinitionsReturnsXMLStringWithInStdClass() {
         $soapMock = $this->getMockFromWsdl(dirname(__FILE__) . '/ReportTest/ReportService2010.wsdl', 'SoapClientMockDefinitions');