Changeset 1081 for trunk/common

Timestamp:

12/17/09 00:03:05 (2 years ago)

Author:

savin.tiberiu@…

Message:

Made the interface for adding tags to a task.
Reviewed: ​http://reviewboard.infoarena.ro/r/120/

Location:

trunk/common

Files:

• db/tags.php (modified) (3 diffs)
• tags.php (modified) (1 diff)
• task.php (modified) (1 diff)

trunk/common/db/tags.php

@@ -54 +54 @@
 }
 
+// Receives an array of tag_id's
+// Return an array with id's of their parents
+// Each parent id appears only once
+function tag_get_parents($tags) {
+    $query = sprintf("SELECT DISTINCT(`parent`)
+            FROM ia_tags
+            WHERE `id` IN (%s)",
+            implode(",", array_map('db_quote', $tags))
+        );
+    $result = db_fetch_all($query);
+    $ret = Array();
+    foreach ($result as $row) {
+        $ret[] = $row['parent'];
+    }
+    return $ret;
+}
+
 // Get tag id for a certain tag
 function tag_get_id($tag) {
@@ -81 +98 @@
 }
 
+// Get a list of tags from a list of tag ids
+function tag_get_by_ids($tag_ids) {
+    $query = sprintf(
+            "SELECT `id` AS tag_id, `name` AS tag_name,
+                    `parent` AS tag_parent, `type` AS tag_type
+            FROM ia_tags
+            WHERE `id` IN (%s)", implode(', ', array_map('db_quote', $tag_ids))
+        );
+    return db_fetch_all($query);
+}
+
 // Assign numeric id to a given tag name
 function tag_assign_id($tag) {
@@ -170 +198 @@
         db_escape($obj), $where, "%s"
     );
-    if (is_null($type)) {
-        $join = "";
-    } else {
+
+    $join = "";
+    if (!is_null($type)) {
         $join = "LEFT JOIN ia_tags AS tags ON obj_tags.tag_id = tags.id";
         $where .= sprintf(" AND tags.type = %s", db_quote($type));
     }
+
     $subquery = sprintf("SELECT tag_id FROM ia_%s_tags AS obj_tags %s WHERE %s",
         db_escape($obj), $join, $where);

trunk/common/tags.php

@@ -52 +52 @@
 }
 
+// Receives a list of parent tags and children tags
+// For each parent_tag adds an array 'sub_tags' containing
+// an array with all his children tags
+function build_tags_tree($parent_tags, $sub_tags) {
+    log_assert(is_array($parent_tags), "Parent tags is not an array");
+    log_assert(is_array($sub_tags), "Children tags is not an array");
+
+    $parent_tags_key = Array();
+    foreach ($parent_tags as $key => $tag) {
+        $parent_tags[$key]['sub_tags'] = Array();
+        $parent_tags_key[ $tag['tag_id'] ] = $key;
+    }
+
+    foreach ($sub_tags as $tag) {
+        log_assert(isset($parent_tags_key[ $tag['tag_parent'] ]), "Child tag doesn't have a parent");
+        $parent_tag_key = $parent_tags_key[ $tag['tag_parent'] ];
+        $parent_tags[ $parent_tag_key ]['sub_tags'][] = $tag;
+    }
+
+    return $parent_tags;
+}
+
 // Receives a list of tags of a certain type and, optionally, with a
 // certain parent and updates the tag list for the specified object.

trunk/common/task.php

@@ -277 +277 @@
 }
 
+// Receives a list of method and algorithm tag ids and adds links them to task_id
+function task_update_tags($task_id, $method_tags_id, $algorithm_tags_id) {
+    log_assert(is_array($method_tags_id), 'method_tags must be an array');
+    log_assert(is_array($algorithm_tags_id), 'algorithm_tags must be an array');
+    log_assert(is_task_id($task_id), "Invalid task_id");
+
+    tag_clear('task', $task_id, 'method');
+    tag_clear('task', $task_id, 'algorithm');
+
+    foreach ($method_tags_id as $tag_id) {
+        tag_add('task', $task_id, $tag_id);
+    }
+
+    foreach ($algorithm_tags_id as $tag_id) {
+        tag_add('task', $task_id, $tag_id);
+    }
+}
+
 ?>