William Skates
William Skates
I'm having a similar problem the only differences being that my host is running osx 10.9.4 and my guest is running Centos 6.5 x86_64.
Maybe using [Generators](http://php.net/manual/en/language.generators.syntax.php)? ``` php $all = $statementsResponse->content->getStatements(); yield $all; while($more = $statementsResponse->content->getMore()) { $batch = $this->fetchStatements($more); foreach ($batch as $statement) { $all[] = $statement; } yield $all; } ?>...
@brianjmiller I'd avoid doing single statements as you would have to make a single request for each one. ``` php $statements = $lrs->findStatements(40); $allStatements = $lrs->findAllStatements(); ``` Is closer to...
@brianjmiller Yielding after each statement would probably be fine. Something like this maybe: ``` php public function fetchAllStatements($batchSize = 10) { $current = 0; $batch = []; while ($batch) {...
@brianjmiller The real benefit to taking this on is giving people a style model that they can follow. Even though the code is consistent we have to infer what the...
@brianjmiller There is no sweeping (i.e. wide range or amount) style change provided by PSR2 that would affect your downstream users. The only way that PSR2 would affect this is...
@brianjmiller I know. I didn't think that their would be anything that could cause significant issue. That was my intuition however so I went back and checked. Essentially what we...
Seems like it's failing due to an existing error. This should be fine to merge.
Yeah it is strange. Let me know how you get on. It may have just been that Scorm Cloud was having issues at the time. I'll reopen this to trigger...
Now that it's been a couple of days lets check it again.