Merge pull request #101 from reedmclean/ActivityAndVerbIDs

Allow for a verb/activity or its ID when querying for statements

Author: reedmclean

src/RemoteLRS.php

@@ -524,7 +524,12 @@ private function _queryStatementsRequestParams($query) {
             ) as $k
         ) {
             if (isset($query[$k])) {
-                $result[$k] = $query[$k]->getId();
+                if (is_string($query[$k])) {
+                    $result[$k] = $query[$k];
+                }
+                else {
+                    $result[$k] = $query[$k]->getId();
+                }
             }
         }
         foreach (

tests/RemoteLRSTest.php

@@ -329,6 +329,24 @@ public function testQueryStatements() {
         $this->assertInstanceOf('TinCan\StatementsResult', $response->content);
     }
 
+    public function testQueryStatementsWithActivityId() {
+        $lrs = new RemoteLRS(self::$endpoint, self::$version, self::$username, self::$password);
+        $response = $lrs->queryStatements(['activity' => COMMON_ACTIVITY_ID]);
+
+        $this->assertInstanceOf('TinCan\LRSResponse', $response);
+        $this->assertTrue($response->success, 'success');
+        $this->assertInstanceOf('TinCan\StatementsResult', $response->content);
+    }
+
+    public function testQueryStatementsWithVerbId() {
+        $lrs = new RemoteLRS(self::$endpoint, self::$version, self::$username, self::$password);
+        $response = $lrs->queryStatements(['verb' => COMMON_VERB_ID]);
+
+        $this->assertInstanceOf('TinCan\LRSResponse', $response);
+        $this->assertTrue($response->success, 'success');
+        $this->assertInstanceOf('TinCan\StatementsResult', $response->content);
+    }
+
     public function testQueryStatementsWithAttachments() {
         $lrs = new RemoteLRS(self::$endpoint, self::$version, self::$username, self::$password);
         $response = $lrs->queryStatements(['limit' => 4, 'attachments' => true]);