SCORMCloud_PHPLibrary

SCORMCloud_PHPLibrary copied to clipboard

Hi, I've changed a PHP keyword to comply with [this standard](https://www.php-fig.org/psr/psr-2/#25-keywords-and-truefalsenull) in PSR2. It's admittedly a small fix but I hope it helps!

mullikine

In many methods, there is this code used to validate the request: ```php $xml = simplexml_load_string($response); write_log($xml->result); return ($xml->result == 'true'); ``` However, I found that the response is different...

blefort-uo

I'm unable to set the learner tags with the API. The API gives a OK response (``), but the tags are not set in the SCORM cloud environment. I'm using...

RobertBoes

Use of constants breaks compatibility with PHP 5.3

1

I can see specially on this commit: https://github.com/RusticiSoftware/SCORMCloud_PHPLibrary/commit/7d387002eb71c60156765da1bb195d6f72b3aec6 Several PHP CMSs and such still need to be run in PHP 5.3, and those constants are only present in PHP 5.4

gardon

Upload service returning empty string

2

I investigated a service call that was apparently successful but returned an empty location. Turns out the XML comes with CDATA so you need to add a parameter do simpleXML

gardon

The library currently imports using the now deprecated uploadFile, then importUploadCourse. Ideally, ImportCourse should be changed to use importCourseAsync and post the file directly to that. For reference implementation, checkout...

ghost