Temporal fix for #1365 when renaming works (student publication)

main/work/work.lib.php

@@ -558,6 +558,12 @@ function display_student_publications_list($work_dir, $sub_course_dir, $currentC
 					$display_edit_form = true;
 
 					if ($form_folder -> validate()) {
+						$values = $form_folder -> exportValues();
+						$values = $values['my_group'];
+						$dir_name = replace_dangerous_char($values['dir_name']);
+						$dir_name = disable_dangerous_file($dir_name);
+						if (is_work_exist_by_url('/'.$dir_name) == false) {
+						
 							$TABLEAGENDA = Database::get_course_table(TABLE_AGENDA);
 							if ($there_is_a_end_date || $there_is_a_expire_date) {
 								if ($row['view_properties'] == '1') {
@@ -597,12 +603,7 @@ function display_student_publications_list($work_dir, $sub_course_dir, $currentC
 								$resource_name = $_POST['dir_name'];
 								add_resource_to_course_gradebook(api_get_course_id(), 3, $row['id'], Database::escape_string($resource_name), (float)$_POST['weight']['weight'], (float)$_POST['qualification']['qualification'], Database::escape_string($_POST['description']), time(), 1, api_get_session_id());
 							}
-						Display::display_confirmation_message(get_lang('FolderEdited'));
 						
-						$values = $form_folder -> exportValues();
-						$values = $values['my_group'];
-						$dir_name = replace_dangerous_char($values['dir_name']);
-						$dir_name = disable_dangerous_file($dir_name);
 							update_dir_name($mydir, $dir_name);
 							$mydir = $my_sub_dir.$dir_name;
 							$dir = $dir_name;
@@ -621,6 +622,11 @@ function display_student_publications_list($work_dir, $sub_course_dir, $currentC
 										WHERE id='".$calendar_id[0]."'";
 								Database::query($sql);
 							}							
+							Display::display_confirmation_message(get_lang('FolderEdited'));
+						
+						} else {
+							Display::display_confirmation_message(get_lang('FileExists'));
+						}
 					}
 				}
 			}
@@ -1562,3 +1568,20 @@ function send_email_on_homework_creation($course_id) {
 		}
 	}
 }
+
+function is_work_exist_by_url($url) {
+	$work_table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
+	$url = Database::escape_string($url);
+	$sql = "SELECT id FROM $work_table WHERE url='$url'";
+	$result = Database::query($sql);
+	if (Database::num_rows($result)> 0) {
+		$row = Database::fetch_row($result);
+		if (empty($row)) {
+			return false;
+		} else {
+			return true;
+		}
+	} else {
+		return false;
+	}
+}