Context / Context Activities

[drewbertola]

Hi. I'd sure love to see an example of context (w/ contextActivities) in the docs. I'm struggling with this - probably unnecessarily.

Specifically, I'm trying to transcribe xAPI v0.9 statements (from Articulate StoryLine) into v.1.0.0 so that they can be pushed onto a LearningLocker LRS. I have the 0.9 statement decomposed, and have everything working except the contextActivities piece.

My code looks like this:

if ( ! empty($data->context) )
{
  $context = new TinCan_Context(array(
    "registration" => $data->context->registration,
  ));

  if ( ! empty($data->context->contextActivities) )
  {
    $contextActivities = new TinCan_ContextActivities(array(
      "grouping" => array("id" => $data->context->contextActivities->grouping->id),
      "parent"   => array("id" => $data->context->contextActivities->parent->id)
    ));

    $context->setContextActivities($contextActivities);
  }
}

The notice I'm getting is this:
PHP Warning: array_push() expects parameter 1 to be array, null given in /var/www/lc/library/TinCan/ContextActivities.php on line 80

which happens in ContextActivities.php in the array_push line here:

private function _listSetter($prop, $value) {
    if (is_array($value)) {
        if ( ! is_array($prop) ) {
        if (isset($value['id'])) {
            array_push($this->$prop, new TinCan_Activity($value));
        }

So, can you provide an example. It's skipped in the docs but it should prove very helpful.

Thanks, Drew

[drewbertola]

BTW, I've forked this code to modify it so that it works with our project under ZF 1.x's autoloader. It's just some naming conventions of the classes.

[drewbertola]

Correct me if I'm wrong, but it seems like property declarations at the top of ContextActivity could be:

protected $category = array();
protected $parent   = array();
protected $grouping = array();
protected $other    = array();

This solved the issue (now array_push(null, $data) doesn't happen but array_push(array(), $data) happens).

[brianjmiller]

Sorry been swamped and out of town at the ADL event, I'll try to have a look at this over the next couple days.

[drewbertola]

No problem. Found a fix and added it to a comment at the end.

Drew

On Jun 24, 2014, at 1:07 AM, Brian J. Miller notifications@github.com wrote:

Sorry been swamped and out of town at the ADL event, I'll try to have a look at this over the next couple days.

—
Reply to this email directly or view it on GitHub.

[brianjmiller]

@drewbertola sorry for the delay on this, I wanted to make sure the length handling was already in there (which it was) and to get it minimally unit tested so needed time to give proper attention. See #12, I'll cut a release tag as soon as it is merged.

[brianjmiller]

This should be resolved, tagged as 0.11.3 and available via packagist. Not sure if there is a way to prevent the need for the fork for ZF's autoloader, I tried to follow the PSR-4 recs so not sure there is anything we can do. If there is way to make them compat let me know via new issue.