Skip to content

Commit 410362b

Browse files
committed
Avoid warning in lp_ajax_save_item when object is not serialized
1 parent 5e37cd6 commit 410362b

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

main/newscorm/lp_ajax_save_item.php

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -62,16 +62,20 @@ function save_item($lp_id, $user_id, $view_id, $item_id, $score = -1, $max = -1,
6262
$lpobject = new learnpathItem($lp_id, $user_id, $courseId);
6363
}
6464
if (isset($lpobject)) {
65-
$oLP = unserialize($lpobject);
66-
if ($debug) error_log("lpobject was set");
67-
if (!is_object($oLP)) {
68-
unset($oLP);
69-
$code = api_get_course_id();
70-
$mylp = new learnpath($code, $lp_id, $user_id);
71-
if ($debug) error_log("Creating learnpath");
65+
if (is_object($lpobject)) {
66+
$mylp = $lpobject;
7267
} else {
73-
$mylp = $oLP;
74-
if ($debug) error_log("Loading learnpath from unserialize");
68+
$oLP = unserialize($lpobject);
69+
if ($debug) error_log("lpobject was set");
70+
if (!is_object($oLP)) {
71+
unset($oLP);
72+
$code = api_get_course_id();
73+
$mylp = new learnpath($code, $lp_id, $user_id);
74+
if ($debug) error_log("Creating learnpath");
75+
} else {
76+
$mylp = $oLP;
77+
if ($debug) error_log("Loading learnpath from unserialize");
78+
}
7579
}
7680
} else {
7781
if ($debug) {

0 commit comments

Comments
 (0)