mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-17 21:49:15 +01:00
Code Issues Projects Releases Wiki Activity

MDL-77254 h5p: Add support to restored states

Browse Source 
The restored states should be treated differently, to avoid resetting
them by default the first time users access them.
In that case, the first time users access to content with restored
xAPI states, they should be displayed (instead of resetting them).
This commit is contained in:
Sara Arjona 2023-02-21 16:46:05 +01:00
parent fba0658777
commit c8628576e3
3 changed files with 28 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
h5p/classes/framework.php
View File

@ -960,7 +960,9 @@ class framework implements H5PFrameworkInterface {
// Reset user data.
try {
$xapihandler = handler::create($file->get_component());
$xapihandler->reset_states($file->get_contextid());
// Reset only entries with 'state' as stateid (the ones restored shouldn't be restored, because the H5P
// content hasn't been created yet).
$xapihandler->reset_states($file->get_contextid(), null, 'state');
} catch (xapi_exception $exception) {
// This component doesn't support xAPI State, so no content needs to be reset.
return;

24
h5p/classes/player.php
View File

@ -277,6 +277,30 @@ class player {
);
try {
$state = $xapihandler->load_state($state);
if (!$state) {
// Check if the state has been restored from a backup for the current user.
$state = new state(
item_agent::create_from_user($USER),
$xapiobject,
'restored',
null,
null
);
$state = $xapihandler->load_state($state);
if ($state && !is_null($state->get_state_data())) {
// A restored state has been found. It will be replaced with one with the proper stateid and statedata.
$xapihandler->delete_state($state);
$state = new state(
item_agent::create_from_user($USER),
$xapiobject,
'state',
$state->jsonSerialize(),
null
);
$xapihandler->save_state($state);
}
}
if (!$state) {
return $emptystatedata;
}

2
lib/xapi/classes/local/state.php
View File

@ -72,7 +72,7 @@ class state implements JsonSerializable {
/**
* Return the data to serialize in case JSON state when needed.
*
* @return stdClass The state data structure
* @return stdClass The state data structure. If statedata is null, this method will return an empty class.
*/
public function jsonSerialize(): stdClass {
if ($this->statedata) {