[Et-mgmt-commits-list] [SCM] virt-factory branch, master now at 04e37e44b7fc016231e7b73d481e5e0ff0b589b0

[Adrian Likins]

Hello,

This is an automated email from the git hooks/update script, it was
generated because a ref change was pushed to the repository.

Updating branch, master,
       via  04e37e44b7fc016231e7b73d481e5e0ff0b589b0 (commit)
      from  ec58ed323488495b22b0a4810e2f71d930becc48 (commit)

- Log -----------------------------------------------------------------
commit 04e37e44b7fc016231e7b73d481e5e0ff0b589b0
Author: Adrian Likins <alikins at grimlock.devel.redhat.com>
Date:   Mon Jun 4 15:00:49 2007 -0400

    were missing an constructor for the validator
    
    used "filter" which shadows keyword, changed to filter_fields
-----------------------------------------------------------------------

Diffstat:
 service/modules/task.py |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/service/modules/task.py b/service/modules/task.py
index 1a5add4..c0c4fd8 100755
--- a/service/modules/task.py
+++ b/service/modules/task.py
@@ -79,14 +79,15 @@ class Task(web_svc.AuthWebSvc):
         """
         required = ('id')
         optional = ('state')
-        filter = ('id', 'user_id', 'action_type', 'machine_id', 'deployment_id')
+        filter_fields = ('id', 'user_id', 'action_type', 'machine_id', 'deployment_id')
+        validator = FieldValidator(args)
         validator.verify_required(required)
         validator.verity_enum('state', VALID_TASK_STATES)
         validator.verity_enum('action_type', VALID_TASK_STATES)
         session = db.open_session()
         try:
             task = db.Task.get(session, args['id'])
-            task.update(args, filter)
+            task.update(args, filter_fields)
             session.save(task)
             session.flush()
             return success()

hooks/update
---
Git Source Code Management System
hooks/update refs/heads/master \
  ec58ed323488495b22b0a4810e2f71d930becc48 \
  04e37e44b7fc016231e7b73d481e5e0ff0b589b0