[Rt-commit] rt branch, 3.999-trunk, updated. b5b929461bd6f6af6da409b2037cb129c3382fcb

[sunnavy at bestpractical.com]

The branch, 3.999-trunk has been updated
       via  b5b929461bd6f6af6da409b2037cb129c3382fcb (commit)
      from  ec701eb3b4fe63f6895c4768ace3b49f1981f2a2 (commit)

Summary of changes:
 TODO                   |    5 -----
 lib/RT/Model/Ticket.pm |    1 -
 2 files changed, 0 insertions(+), 6 deletions(-)

- Log -----------------------------------------------------------------
commit b5b929461bd6f6af6da409b2037cb129c3382fcb
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Tue Nov 17 12:25:11 2009 +0800

    fix transaction type for status: type field should be "set"

diff --git a/TODO b/TODO
index 34b4a98..1d68aa2 100644
--- a/TODO
+++ b/TODO
@@ -3,11 +3,6 @@ things we ripped out that need to be added back
     external auth
     autocreate of users
 
-RT has some inconsistency in recording of Status changes,
-some time it could be (Type = Status, Field = Status) or
-(Type = Set, Field = Status). I think it should be fixed
-to the latter.
-
 RT::load_config and other in RT package should be replaced with RT->load_config
 
 
diff --git a/lib/RT/Model/Ticket.pm b/lib/RT/Model/Ticket.pm
index 937092d..af52fa1 100755
--- a/lib/RT/Model/Ticket.pm
+++ b/lib/RT/Model/Ticket.pm
@@ -2088,7 +2088,6 @@ sub set_status {
         value            => $args{status},
         time_taken       => 0,
         check_acl        => 0,
-        transaction_type => 'status'
     );
 
     return ( $val, $msg );

-----------------------------------------------------------------------