r4119 updateTag(): dismiss, now obsolete
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 13 Jan 2011 10:31:09 +0000 (10:31 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 13 Jan 2011 10:31:09 +0000 (10:31 +0000)
commitUpdateTag(): sayonara

inc/database.php
inc/navigation.php
inc/ophandlers.php

index ddca0873775efd1786729acaa4062a0c5f6024b4..ec11ac8423fc503cc10d22542ed3b442e9e59bb7 100644 (file)
@@ -2963,20 +2963,6 @@ function getTagList ()
        return $ret;
 }
 
-function commitUpdateTag ($tag_id, $tag_name, $parent_id)
-{
-       return usePreparedExecuteBlade
-       (
-               'UPDATE TagTree SET tag=?, parent_id=? WHERE id=?',
-               array
-               (
-                       $tag_name,
-                       $parent_id == 0 ? NULL : $parent_id,
-                       $tag_id,
-               )
-       );
-}
-
 // Drop the whole chain stored.
 function destroyTagsForEntity ($entity_realm, $entity_id)
 {
index 8705677404b60b8ad7a03bd9199e7251894dd0d8..621c9ffcf7c718f7462c2161243773ff4c4ed8d1 100644 (file)
@@ -488,7 +488,7 @@ $tabhandler['tagtree']['default'] = 'renderTagTree';
 $tabhandler['tagtree']['edit'] = 'renderTagTreeEditor';
 $ophandler['tagtree']['edit']['createTag'] = 'tableHandler';
 $ophandler['tagtree']['edit']['destroyTag'] = 'tableHandler';
-$ophandler['tagtree']['edit']['updateTag'] = 'updateTag';
+$ophandler['tagtree']['edit']['updateTag'] = 'tableHandler';
 
 $page['myaccount']['title'] = 'My account';
 $page['myaccount']['parent'] = 'config';
index 89f242e8a6ac94ca7ca38d1c117916c20cf68b61..52af2758b3b86cb10ec091381882918d6c553c0b 100644 (file)
@@ -230,7 +230,21 @@ $opspec_list['tagtree-edit-destroyTag'] = array
        'action' => 'DELETE',
        'arglist' => array
        (
-               array ('url_argname' => 'tag_id', 'table_colname' => 'id', 'assertion' => 'tag'),
+               array ('url_argname' => 'tag_id', 'table_colname' => 'id', 'assertion' => 'uint'),
+       ),
+);
+$opspec_list['tagtree-edit-updateTag'] = array
+(
+       'table' => 'TagTree',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'tag_name', 'table_colname' => 'tag', 'assertion' => 'tag'),
+               array ('url_argname' => 'parent_id', 'assertion' => 'uint0', 'if_empty' => 'NULL'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'tag_id', 'table_colname' => 'id', 'assertion' => 'uint'),
        ),
 );
 $opspec_list['vlandomain-vlanlist-add'] = array
@@ -1701,22 +1715,6 @@ function saveEntityTags ()
                return buildRedirectURL (__FUNCTION__, 'OK', array ($n_succeeds));
 }
 
-$msgcode['updateTag']['OK'] = 7;
-$msgcode['updateTag']['ERR2'] = 109;
-function updateTag ()
-{
-       assertUIntArg ('tag_id');
-       assertUIntArg ('parent_id', TRUE);
-       genericAssertion ('tag_name', 'tag');
-       if (($parent_id = $_REQUEST['parent_id']) <= 0)
-               $parent_id = 'NULL';
-       if (FALSE !== commitUpdateTag ($_REQUEST['tag_id'], $tagname, $parent_id))
-               return buildRedirectURL (__FUNCTION__, 'OK', array ($tagname));
-       // Use old name in the message, cause update failed.
-       global $taglist;
-       return buildRedirectURL (__FUNCTION__, 'ERR2', array ($taglist[$_REQUEST['tag_id']]['tag']));
-}
-
 $msgcode['rollTags']['OK'] = 67;
 $msgcode['rollTags']['ERR'] = 149;
 function rollTags ()