Fixes validation for groups.

Author: ryasmi

src/Group.php

@@ -13,9 +13,9 @@ protected function identifierError($used_identifiers) {
   }
 
   protected function validateIdentifiers($used_identifiers) {
-    $members = $this->getProp('members') ?: null;
-    $members = $members !== null ? $members->getValue() : null;
+    $members = $this->getPropValue('member') ?: null;
     $validateIdentifiers = parent::validateIdentifiers($used_identifiers);
-    return $validateIdentifiers && $members === null;
+    $validateMembers = $members === null || count($members) === 0;
+    return $validateIdentifiers && $validateMembers;
   }
 }

src/Members.php

@@ -1,5 +1,5 @@
 <?php namespace Locker\XApi;
 
 class Members extends Collection {
-  protected $member_type = 'Locker\XApi\Actor';
+  protected $member_type = 'Locker\XApi\Agent';
 }

tests/assets/test.json

@@ -1,42 +1,20 @@
 {
-  "id": "710cc790-9d8d-11e4-bd06-0800200c9a66",
-  "actor": {
-    "objectType": "Agent",
-    "mbox": "mailto:[email protected]",
-    "name": "quanvm"
-  },
-  "verb": {
-    "id": "http://adlnet.gov/expapi/verbs/experienced",
-    "display": {"en-us": "experienced"}
-  },
-  "context": {
-    "contextActivities": {
-      "parent": {
-        "id": "http://tincanapi.com/GolfExample_TCAPI",
-        "objectType": "Activity"
-      },
-      "grouping": [{
-        "id": "http://tincanapi.com/GolfExample_TCAPI",
-        "objectType": "Activity"
-      }]
+    "actor": {
+        "objectType": "Group",
+        "account": {
+            "homePage": "http://myaccount.bmj.com",
+            "name": "hrl"
+        },
+        "member": [
+        ]
+    },
+    "verb": {
+        "id": "http:\/\/adlnet.gov\/expapi\/verbs\/completed",
+        "display": {
+            "en-GB": "completed"
+        }
+    },
+    "object": {
+        "id": "http:\/\/localhost\/moodle\/mod\/quiz\/attempt.php?attempt=48"
     }
-  },
-  "object": {
-    "id": "http://tincanapi.com/GolfExample_TCAPI/Playing/Scoring.html",
-    "objectType": "Activity",
-    "definition": {
-      "name": {
-        "en-US": "Scoring"
-      },
-      "description": {
-        "en-US": "An overview of how to score a round of golf."
-      },
-      "type": "http:///activitystrea.ms/schema/1.0/badge"
-    }
-  },
-  "authority": {
-    "name": "",
-    "mbox": "mailto:[email protected]",
-    "objectType": "Agent"
-  }
 }