Use iid to edit work

pull/2487/head
jmontoyaa 9 years ago
parent fd941099c5
commit 256ae350fa
  1. 4
      main/work/edit_work.php
  2. 16
      main/work/work.lib.php
  3. 2
      src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php

@ -1,8 +1,6 @@
<?php <?php
/* For licensing terms, see /license.txt */ /* For licensing terms, see /license.txt */
use ChamiloSession as Session;
require_once '../inc/global.inc.php'; require_once '../inc/global.inc.php';
$lib_path = api_get_path(LIBRARY_PATH); $lib_path = api_get_path(LIBRARY_PATH);
@ -109,7 +107,7 @@ if ($form->validate()) {
if ($editCheck) { if ($editCheck) {
updateWork($workId, $params, $courseInfo, $sessionId); updateWork($workData['iid'], $params, $courseInfo, $sessionId);
updatePublicationAssignment($workId, $params, $courseInfo, $groupId); updatePublicationAssignment($workId, $params, $courseInfo, $groupId);
updateDirName($workData, $params['new_dir']); updateDirName($workData, $params['new_dir']);

@ -866,7 +866,7 @@ function updateDirName($work_data, $newPath)
{ {
$course_id = $work_data['c_id']; $course_id = $work_data['c_id'];
$sessionId = intval($work_data['session_id']); $sessionId = intval($work_data['session_id']);
$work_id = intval($work_data['id']); $work_id = intval($work_data['iid']);
$oldPath = $work_data['url']; $oldPath = $work_data['url'];
$originalNewPath = Database::escape_string($newPath); $originalNewPath = Database::escape_string($newPath);
$newPath = Database::escape_string($newPath); $newPath = Database::escape_string($newPath);
@ -874,6 +874,7 @@ function updateDirName($work_data, $newPath)
$newPath = disable_dangerous_file($newPath); $newPath = disable_dangerous_file($newPath);
if ($oldPath == '/'.$newPath) { if ($oldPath == '/'.$newPath) {
return true; return true;
} }
@ -883,8 +884,7 @@ function updateDirName($work_data, $newPath)
title = '".$originalNewPath."' title = '".$originalNewPath."'
WHERE WHERE
c_id = $course_id AND c_id = $course_id AND
id = $work_id AND iid = $work_id";
session_id = " . ($sessionId ? $sessionId : 'NULL');
Database::query($sql); Database::query($sql);
} }
} }
@ -3773,7 +3773,7 @@ function agendaExistsForWork($workId, $courseInfo)
/** /**
* Update work description, qualification, weight, allow_text_assignment * Update work description, qualification, weight, allow_text_assignment
* @param int $workId * @param int $workId (iid)
* @param array $params * @param array $params
* @param array $courseInfo * @param array $courseInfo
* @param int $sessionId * @param int $sessionId
@ -3792,10 +3792,9 @@ function updateWork($workId, $params, $courseInfo, $sessionId = 0)
$workTable, $workTable,
$filteredParams, $filteredParams,
array( array(
'id = ? AND c_id = ? AND session_id = ? ' => array( 'iid = ? AND c_id = ?' => array(
$workId, $workId,
$courseInfo['real_id'], $courseInfo['real_id']
$sessionId ? $sessionId : null
) )
) )
); );
@ -3819,7 +3818,6 @@ function updatePublicationAssignment($workId, $params, $courseInfo, $groupId)
$agendaId = 0; $agendaId = 0;
if (isset($params['add_to_calendar']) && $params['add_to_calendar'] == 1) { if (isset($params['add_to_calendar']) && $params['add_to_calendar'] == 1) {
require_once api_get_path(SYS_CODE_PATH).'resourcelinker/resourcelinker.inc.php';
// Setting today date // Setting today date
$date = $end_date = $time; $date = $end_date = $time;
@ -3915,7 +3913,7 @@ function updatePublicationAssignment($workId, $params, $courseInfo, $groupId)
WHERE WHERE
publication_id = $workId AND publication_id = $workId AND
c_id = $course_id AND c_id = $course_id AND
id = ".$data['id']; iid = ".$data['iid'];
Database::query($sql); Database::query($sql);
} }

@ -3194,7 +3194,7 @@ class CourseRestorer
$sessionId $sessionId
); );
} else { } else {
$workId = $workData['id']; $workId = $workData['iid'];
updateWork( updateWork(
$workId, $workId,
$obj->params, $obj->params,

Loading…
Cancel
Save