Changeset 1084 for trunk

Timestamp:

12/21/09 17:09:07 (2 years ago)

Author:

bogdan2412

Message:

Script to add author tags to tasks.

Tries and determine each author correctly for tasks with multiple ones and groups together different spellings of names (for example Baltescu Paul and Paul-Dan Baltescu) :P

Also makes task_edit controller update author in ia_task to reflect the tags.

Location:

trunk

Files:

• common/db/tags.php (modified) (2 diffs)
• scripts/migrate-authors-to-task-tags (added)
• www/controllers/task.php (modified) (3 diffs)
• www/views/task_edit.php (modified) (2 diffs)

trunk/common/db/tags.php

@@ -25 +25 @@
         $query .= sprintf(" WHERE %s", implode(" AND ", $where));
     }
+    $query .= " ORDER BY name";
     return db_fetch_all($query);
 }
@@ -47 +48 @@
         FROM ia_%s_tags AS obj_tags
         LEFT JOIN ia_tags AS tags ON obj_tags.tag_id = tags.id
-        WHERE %s_id = %s%s%s",
+        WHERE %s_id = %s%s%s
+        ORDER BY tag_name",
         db_escape($obj), db_escape($obj), db_escape($obj),
         db_quote($obj_id), $where_type, $where_parent

trunk/www/controllers/task.php

@@ -43 +43 @@
 
     // Fill in form values from request, defaults in $task
-    $fields = array('author', 'type', 'source', 'hidden', 'title', 'page_name',
+    $fields = array('type', 'source', 'hidden', 'title', 'page_name',
                     'open_source', 'open_tests', 'test_count', 'test_groups',
                     'evaluator', 'use_ok_files', 'public_tests');
@@ -146 +146 @@
             // FIXME: error handling? Is that even remotely possible in php?
             task_update_parameters($task_id, $new_task_params);
-            task_update($new_task);
 
             if (identity_can('task-tag', $new_task)) {
@@ -160 +159 @@
                 }
             }
+            $new_task["author"] = tag_build_list("task", $new_task["id"], "author");
+            task_update($new_task);
 
             flash("Task-ul a fost modificat cu succes.");

trunk/www/views/task_edit.php

@@ -29 +29 @@
                 'description' => "Utilizatorul care are drepturi de editare ".
                                  "asupra problemei. Poate fi lasat gol.",
-                'type' => 'string',
-        ),
-        'author' => array(
-                'name' => "Autor(i)",
                 'type' => 'string',
         ),
@@ -120 +116 @@
             <?= view_form_field_li($form_fields['user'], 'user') ?>
         <?php } ?>
-        <?= view_form_field_li($form_fields['author'], 'author') ?>
         <?= view_form_field_li($form_fields['source'], 'source') ?>
         <?php if (identity_can('task-change-security', $task)) { ?>