[Rt-commit] rt branch, update-action, updated. 12dd1fb5598bd4ff4eaefc4eb658ef8f5da777f4

[sartak at bestpractical.com]

The branch, update-action has been updated
       via  12dd1fb5598bd4ff4eaefc4eb658ef8f5da777f4 (commit)
      from  9002518242f7187a18d587352ddf4358f252cf0d (commit)

Summary of changes:
 lib/RT/Action/CreateTicket.pm |   41 -----------------------------------------
 lib/RT/Action/TicketAction.pm |   41 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 41 insertions(+), 41 deletions(-)

- Log -----------------------------------------------------------------
commit 12dd1fb5598bd4ff4eaefc4eb658ef8f5da777f4
Author: Shawn M Moore <sartak at bestpractical.com>
Date:   Tue Dec 8 06:52:39 2009 -0500

    Move datetime/duration parameters upwards too

diff --git a/lib/RT/Action/CreateTicket.pm b/lib/RT/Action/CreateTicket.pm
index 1bf9d0f..7f1941a 100644
--- a/lib/RT/Action/CreateTicket.pm
+++ b/lib/RT/Action/CreateTicket.pm
@@ -69,31 +69,6 @@ sub after_set_queue {
         hints => _('(Sends a carbon-copy of this update to a comma-delimited list of administrative email addresses. These people <strong>will</strong> receive future updates.)'),
     );
 
-    $self->add_duration_parameter(
-        name  => 'time_estimated',
-        label => _('Time Estimated'),
-    );
-
-    $self->add_duration_parameter(
-        name  => 'time_worked',
-        label => _('Time Worked'),
-    );
-
-    $self->add_duration_parameter(
-        name  => 'time_left',
-        label => _('Time Left'),
-    );
-
-    $self->add_datetime_parameter(
-        name  => 'starts',
-        label => _('Starts'),
-    );
-
-    $self->add_datetime_parameter(
-        name  => 'due',
-        label => _('Due'),
-    );
-
     $self->add_link_parameter(
         name  => 'depends_on',
         label => _('Depends on'),
@@ -255,22 +230,6 @@ __PACKAGE__->_add_parameter_type(
 );
 
 __PACKAGE__->_add_parameter_type(
-    name     => 'duration',
-    defaults => {
-        render_as      => 'text', # ideally would be Duration
-        display_length => 3,
-    },
-);
-
-__PACKAGE__->_add_parameter_type(
-    name     => 'datetime',
-    defaults => {
-        render_as      => 'DateTime',
-        display_length => 16,
-    },
-);
-
-__PACKAGE__->_add_parameter_type(
     name     => 'link',
     defaults => {
         render_as      => 'text',
diff --git a/lib/RT/Action/TicketAction.pm b/lib/RT/Action/TicketAction.pm
index 3a799e9..34c3149 100644
--- a/lib/RT/Action/TicketAction.pm
+++ b/lib/RT/Action/TicketAction.pm
@@ -16,6 +16,31 @@ sub after_set_queue {
 
     $self->add_ticket_custom_fields($queue);
     $self->add_ticket_transaction_custom_fields($queue);
+
+    $self->add_duration_parameter(
+        name  => 'time_estimated',
+        label => _('Time Estimated'),
+    );
+
+    $self->add_duration_parameter(
+        name  => 'time_worked',
+        label => _('Time Worked'),
+    );
+
+    $self->add_duration_parameter(
+        name  => 'time_left',
+        label => _('Time Left'),
+    );
+
+    $self->add_datetime_parameter(
+        name  => 'starts',
+        label => _('Starts'),
+    );
+
+    $self->add_datetime_parameter(
+        name  => 'due',
+        label => _('Due'),
+    );
 }
 
 sub set_valid_statuses {
@@ -122,5 +147,21 @@ __PACKAGE__->_add_parameter_type(
     name => 'ticket_transaction_custom_field',
 );
 
+__PACKAGE__->_add_parameter_type(
+    name     => 'duration',
+    defaults => {
+        render_as      => 'text', # ideally would be Duration
+        display_length => 3,
+    },
+);
+
+__PACKAGE__->_add_parameter_type(
+    name     => 'datetime',
+    defaults => {
+        render_as      => 'DateTime',
+        display_length => 16,
+    },
+);
+
 1;
 

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