r2909 - saveEntityTags(): don't import global vars, which we don't use here
authorDenis Ovsienko <infrastation@yandex.ru>
Fri, 26 Jun 2009 09:47:32 +0000 (09:47 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Fri, 26 Jun 2009 09:47:32 +0000 (09:47 +0000)
inc/ophandlers.php

index 038a0dfecf55be09c173e01c599d181aadaf722b..b04da452b7af48a0c7912e2feb963ec454488473 100644 (file)
@@ -1375,7 +1375,7 @@ $msgcode['saveEntityTags']['ERR2'] = 187;
 // Filter out implicit tags before storing the new tag set.
 function saveEntityTags ()
 {
-       global $explicit_tags, $implicit_tags, $page, $pageno, $etype_by_pageno;
+       global $page, $pageno, $etype_by_pageno;
        if (!isset ($etype_by_pageno[$pageno]) or !isset ($page[$pageno]['bypass']))
                return buildRedirectURL (__FUNCTION__, 'ERR2', array (__FUNCTION__));
        $realm = $etype_by_pageno[$pageno];
@@ -1386,6 +1386,8 @@ function saveEntityTags ()
        // Build a chain from the submitted data, minimize it,
        // then wipe existing records and store the new set instead.
        destroyTagsForEntity ($realm, $entity_id);
+       // TODO: these actions are very close to what rebuildTagChainForEntity() does,
+       // so why not use it?
        $newchain = getExplicitTagsOnly (buildTagChainFromIds ($taglist));
        $n_succeeds = $n_errors = 0;
        foreach ($newchain as $taginfo)