From: moodler <moodler>
Date: Wed, 27 Feb 2008 07:13:09 +0000 (+0000)
Subject: Merged further fix for MDL-13635
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=ffcd3023e14a09fac066ec245755644a1d3192c4;p=moodle.git

Merged further fix for MDL-13635
---

diff --git a/tag/edit.php b/tag/edit.php
index 7dd236edb4..d91aa5c5d2 100644
--- a/tag/edit.php
+++ b/tag/edit.php
@@ -49,8 +49,6 @@ $tagform->set_data($tag);
 // If new data has been sent, update the tag record
 if ($tagnew = $tagform->get_data()) {
 
-    $do_update_name = false;
-
     if (!has_capability('moodle/tag:manage', $systemcontext)) {
         unset($tagnew->name);
         unset($tagnew->rawname);
@@ -59,15 +57,12 @@ if ($tagnew = $tagform->get_data()) {
         $tagnew->name = array_shift(tag_normalize($tagnew->rawname, TAG_CASE_LOWER));
 
         if (!$tagold = tag_get_tag_by_id($tag_id)) {  // For doing checks
-            error('Error updating tag record');
+            error('Error finding tag record');
         }
 
         if ($tagold->name != $tagnew->name) {  // The name has changed, let's make sure it's not another existing tag
             if (tag_get_id($tagnew->name)) {   // Something exists already, so flag an error
                 $errorstring = s($tagnew->rawname).': '.get_string('namesalreadybeeingused', 'tag');
-            
-            } else {
-                $do_update_name = true;
             }
         }
     }
@@ -76,7 +71,7 @@ if ($tagnew = $tagform->get_data()) {
         $tagnew->timemodified = time();
 
         // rename tag if needed
-        if ($do_update_name && !tag_rename($tag_id, $tagnew->rawname)) {
+        if (tag_rename($tag_id, $tagnew->rawname)) {
             error('Error updating tag record');
         }