chamilo
/
chamilo-lms
mirror of https://github.com/chamilo/chamilo-lms/tree/1.11.x
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Minor - Fix Codacy issues #2471

Browse Source
pull/2473/head
Angel Fernando Quiroz Campos 8 years ago
parent 3f4800295b
commit 7dea652201
1 changed files with 55 additions and 51 deletions
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Show Stats Download Patch File Download Diff File
  1. 106
      main/inc/lib/webservices/Rest.php

106
main/inc/lib/webservices/Rest.php
Unescape View File

@ -1031,32 +1031,36 @@ class Rest extends WebService
}
/**
* @param array $course_param
* @param array $courseParam
*
* @return array
*/
public function addCourse($course_param)
public function addCourse(array $courseParam)
{
$table_course = Database::get_main_table(TABLE_MAIN_COURSE);
$extra_list = [];
$tableCourse = Database::get_main_table(TABLE_MAIN_COURSE);
$extraList = [];
$results = [];
$title = isset($course_param['title']) ? $course_param['title'] : '';
$category_code = isset($course_param['category_code']) ? $course_param['category_code'] : '';
$wanted_code = isset($course_param['wanted_code']) ? intval($course_param['wanted_code']) : 0;
$tutor_name = isset($course_param['tutor_name']) ? $course_param['tutor_name'] : '';
$course_language = isset($course_param['language']) ? $course_param['language'] : null;
$originalCourseIdName = isset($course_param['original_course_id_name']) ? $course_param['original_course_id_name'] : null;
$originalCourseIdValue = isset($course_param['original_course_id_value']) ? $course_param['original_course_id_value'] : null;
$diskQuota = isset($course_param['disk_quota']) ? $course_param['disk_quota'] : '100';
$visibility = isset($course_param['visibility']) ? (int) $course_param['visibility'] : null;
if (isset($course_param['visibility'])) {
if ($course_param['visibility'] &&
$course_param['visibility'] >= 0 &&
$course_param['visibility'] <= 3
$title = isset($courseParam['title']) ? $courseParam['title'] : '';
$categoryCode = isset($courseParam['category_code']) ? $courseParam['category_code'] : '';
$wantedCode = isset($courseParam['wanted_code']) ? intval($courseParam['wanted_code']) : 0;
$tutorName = isset($courseParam['tutor_name']) ? $courseParam['tutor_name'] : '';
$courseLanguage = isset($courseParam['language']) ? $courseParam['language'] : null;
$originalCourseIdName = isset($courseParam['original_course_id_name'])
? $courseParam['original_course_id_name']
: null;
$originalCourseIdValue = isset($courseParam['original_course_id_value'])
? $courseParam['original_course_id_value']
: null;
$diskQuota = isset($courseParam['disk_quota']) ? $courseParam['disk_quota'] : '100';
$visibility = isset($courseParam['visibility']) ? (int)$courseParam['visibility'] : null;
if (isset($courseParam['visibility'])) {
if ($courseParam['visibility'] &&
$courseParam['visibility'] >= 0 &&
$courseParam['visibility'] <= 3
) {
$visibility = (int) $course_param['visibility'];
$visibility = (int)$courseParam['visibility'];
}
}
@ -1068,26 +1072,26 @@ class Rest extends WebService
if (!empty($courseInfo)) {
if ($courseInfo['visibility'] != 0) {
$sql = "UPDATE $table_course SET
course_language = '".Database::escape_string($course_language)."',
$sql = "UPDATE $tableCourse SET
course_language = '".Database::escape_string($courseLanguage)."',
title = '".Database::escape_string($title)."',
category_code = '".Database::escape_string($category_code)."',
tutor_name = '".Database::escape_string($tutor_name)."',
visual_code = '".Database::escape_string($wanted_code)."'";
category_code = '".Database::escape_string($categoryCode)."',
tutor_name = '".Database::escape_string($tutorName)."',
visual_code = '".Database::escape_string($wantedCode)."'";
if ($visibility !== null) {
$sql .= ", visibility = $visibility ";
}
$sql .= " WHERE id = ".$courseInfo['real_id'];
Database::query($sql);
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'];
if (is_array($extraList) && count($extraList) > 0) {
foreach ($extraList as $extra) {
$extraFieldName = $extra['field_name'];
$extraFieldValue = $extra['field_value'];
// Save the external system's id into course_field_value table.
CourseManager::update_course_extra_field_value(
$courseInfo['code'],
$extra_field_name,
$extra_field_value
$extraFieldName,
$extraFieldValue
);
}
}
@ -1097,21 +1101,21 @@ class Rest extends WebService
$params = [];
$params['title'] = $title;
$params['wanted_code'] = $wanted_code;
$params['category_code'] = $category_code;
$params['course_category'] = $category_code;
$params['tutor_name'] = $tutor_name;
$params['course_language'] = $course_language;
$params['wanted_code'] = $wantedCode;
$params['category_code'] = $categoryCode;
$params['course_category'] = $categoryCode;
$params['tutor_name'] = $tutorName;
$params['course_language'] = $courseLanguage;
$params['user_id'] = $this->user->getId();
$params['visibility'] = $visibility;
$params['disk_quota'] = $diskQuota;
$params['subscribe'] = empty($course_param['subscribe']) ? 0 : 1;
$params['unsubscribe'] = empty($course_param['unsubscribe']) ? 0 : 1;
$params['subscribe'] = empty($courseParam['subscribe']) ? 0 : 1;
$params['unsubscribe'] = empty($courseParam['unsubscribe']) ? 0 : 1;
$course_info = CourseManager::create_course($params, $params['user_id']);
$courseInfo = CourseManager::create_course($params, $params['user_id']);
if (!empty($course_info)) {
$course_code = $course_info['code'];
if (!empty($courseInfo)) {
$courseCode = $courseInfo['code'];
// Save new field label into course_field table
CourseManager::create_course_extra_field(
@ -1123,31 +1127,31 @@ class Rest extends WebService
// Save the external system's id into user_field_value table.
CourseManager::update_course_extra_field_value(
$course_code,
$courseCode,
$originalCourseIdName,
$originalCourseIdValue
);
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'];
if (is_array($extraList) && count($extraList) > 0) {
foreach ($extraList as $extra) {
$extraFieldName = $extra['field_name'];
$extraFieldValue = $extra['field_value'];
// Save new fieldlabel into course_field table.
CourseManager::create_course_extra_field(
$extra_field_name,
$extraFieldName,
1,
$extra_field_name,
$extraFieldName,
''
);
// Save the external system's id into course_field_value table.
CourseManager::update_course_extra_field_value(
$course_code,
$extra_field_name,
$extra_field_value
$courseCode,
$extraFieldName,
$extraFieldValue
);
}
}
$results[] = $course_code;
$results[] = $courseCode;
}
return $results;

Write Preview
Loading…
Cancel
Save