Fix mysql errors + minor format code

main/admin/settings.php

@@ -114,9 +114,9 @@ function get_settings($category = null)
 
                 // One more validation if is changeable.
                 if ($row['access_url_changeable'] == 1) {
-                    $settings_by_access_list[$row['variable']] [$row['subkey']] [$row['category']] = $row;
+                    $settings_by_access_list[$row['variable']][$row['subkey']][$row['category']] = $row;
                 } else {
-                    $settings_by_access_list[$row['variable']] [$row['subkey']] [$row['category']] = array();
+                    $settings_by_access_list[$row['variable']][$row['subkey']][$row['category']] = array();
                 }
             }
         }

main/newscorm/learnpath.class.php

@@ -216,9 +216,15 @@ class learnpath
                 error_log('New LP - learnpath::__construct() ' . __LINE__ . ' - NOT Found previous view', 0);
             }
             $this->attempt = 1;
-            $sql = "INSERT INTO $lp_table (c_id, lp_id, user_id, view_count, session_id)
-                    VALUES ($course_id, $lp_id, $user_id, 1, $session_id)";
-            Database::query($sql);
+            $params = [
+                'c_id' => $course_id,
+                'lp_id' => $lp_id,
+                'user_id' => $user_id,
+                'view_count' => 1,
+                'session_id' => $session_id,
+                'last_item' => 0
+            ];
+            Database::insert($lp_table, $params);
             $this->lp_view_id = Database::insert_id();
 
             if ($this->debug > 2) {
@@ -821,9 +827,22 @@ class learnpath
                     'js_lib' => '',
                     'session_id' => $session_id,
                     'created_on' => api_get_utc_datetime(),
+                    'modified_on'  => api_get_utc_datetime(),
                     'publicated_on' => $publicated_on,
                     'expired_on' => $expired_on,
-                    'category_id' => $categoryId
+                    'category_id' => $categoryId,
+                    'force_commit' => 0,
+                    'content_license' => '',
+                    'debug' => 0,
+                    'theme' => '',
+                    'preview_image' => '',
+                    'author' => '',
+                    'prerequisite' => 0,
+                    'hide_toc_frame' => 0,
+                    'seriousgame_mode' => 0,
+                    'autolaunch' => 0,
+                    'max_attempts' => 0,
+                    'subscribe_users' => 0
                 ];
 
                 $id = Database::insert($tbl_lp, $params);

main/webservices/registration.soap.php

@@ -1298,8 +1298,8 @@ function WSCreateUserPasswordCrypted($params)
         UrlManager::add_user_to_url($return, $url_id);
         if ($debug) error_log("Adding user_id = $return to URL id $url_id ");
 
-        // Save new fieldlabel into user_field table.
-        $field_id = UserManager::create_extra_field(
+        // Create extra field for the original_user_id_name
+        UserManager::create_extra_field(
             $original_user_id_name,
             1,
             $original_user_id_name,
@@ -1312,12 +1312,13 @@ function WSCreateUserPasswordCrypted($params)
             $original_user_id_value
         );
 
+        // Create extra fields
         if (is_array($extra_list) && count($extra_list) > 0) {
             foreach ($extra_list as $extra) {
                 $extra_field_name   = $extra['field_name'];
                 $extra_field_value  = $extra['field_value'];
                 // save new fieldlabel into user_field table
-                $field_id = UserManager::create_extra_field(
+                UserManager::create_extra_field(
                     $extra_field_name,
                     1,
                     $extra_field_name,