Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

methods for Person class #30

Open
garemoko opened this issue Apr 29, 2015 · 4 comments
Open

methods for Person class #30

garemoko opened this issue Apr 29, 2015 · 4 comments

Comments

@garemoko
Copy link
Contributor

I'm working on a Person class right now. I won't include the following methods, but they would be nice to include later:

  • Take an array of agents and return a person
  • Take a person and return an array of agents
@GrantBailey
Copy link

I'm working on a Person class right now.

@garemoko Watching with interest but a bit confused: by Person do you mean an Agent of a specific type (i.e. a persona rather than a system - paragraph 2.4.2.1 of spec)?
If so, I am unclear how the Agent could be recognisable as a Person since none of the IFIs appear to be exclusive to personas (e.g. an mbox could be 'info@somecompany.com').

@brianjmiller
Copy link
Member

FWIW Person has mostly been implemented. See https://github.com/RusticiSoftware/TinCanPHP/blob/master/src/Person.php which landed in #41 available since 0.40.0.

@brianjmiller
Copy link
Member

@GrantBailey by Person he is referring to https://github.com/adlnet/xAPI-Spec/blob/master/xAPI-Communication.md#24-agents-resource which is its own data type, no idea why this wasn't included in the "Part Two" document, probably because it isn't directly related to Statements.

@GrantBailey
Copy link

@brianjmiller Many thanks, much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants