Fixed tests, applied new auto reply.

db/migrate/20160503000002_update_trigger.rb

@@ -13,7 +13,7 @@ class UpdateTrigger < ActiveRecord::Migration
         },
         'ticket.state_id' => {
           'operator' => 'is not',
-          'value' => '4',
+          'value' => Ticket::State.lookup(name: 'closed').id,
         },
         'article.type_id' => {
           'operator' => 'is',
@@ -23,6 +23,10 @@ class UpdateTrigger < ActiveRecord::Migration
             Ticket::Article::Type.lookup(name: 'web').id,
           ],
         },
+        'article.sender_id' => {
+          'operator' => 'is',
+          'value' => Ticket::Article::Sender.lookup(name: 'Customer').id,
+        },
       },
       perform: {
         'notification.email' => {

db/migrate/20160505000001_update_trigger2.rb

@@ -0,0 +1,33 @@
+class UpdateTrigger2 < ActiveRecord::Migration
+  def up
+
+    # return if it's a new setup
+    return if !Setting.find_by(name: 'system_init_done')
+    trigger = Trigger.find_by(name: 'auto reply (on new tickets)')
+    return if !trigger
+    trigger.condition = {
+      'ticket.action' => {
+        'operator' => 'is',
+        'value' => 'create',
+      },
+      'ticket.state_id' => {
+        'operator' => 'is not',
+        'value' => Ticket::State.lookup(name: 'closed').id,
+      },
+      'article.type_id' => {
+        'operator' => 'is',
+        'value' => [
+          Ticket::Article::Type.lookup(name: 'email').id,
+          Ticket::Article::Type.lookup(name: 'phone').id,
+          Ticket::Article::Type.lookup(name: 'web').id,
+        ],
+      },
+      'article.sender_id' => {
+        'operator' => 'is',
+        'value' => Ticket::Article::Sender.lookup(name: 'Customer').id,
+      },
+    }
+    trigger.save
+  end
+
+end

db/seeds.rb

@@ -3865,7 +3865,7 @@ Trigger.create_or_update(
     },
     'ticket.state_id' => {
       'operator' => 'is not',
-      'value' => '4',
+      'value' => Ticket::State.lookup(name: 'closed').id,
     },
     'article.type_id' => {
       'operator' => 'is',
@@ -3875,6 +3875,10 @@ Trigger.create_or_update(
         Ticket::Article::Type.lookup(name: 'web').id,
       ],
     },
+    'article.sender_id' => {
+      'operator' => 'is',
+      'value' => Ticket::Article::Sender.lookup(name: 'Customer').id,
+    },
   },
   perform: {
     'notification.email' => {

test/browser/agent_ticket_actions_level6_test.rb

@@ -121,11 +121,11 @@ class AgentTicketActionLevel6Test < TestCase
       value: 'test 6 - ticket 1-1',
     )
     match_not(
-      css: '.active .ticket-article-item:nth-child(2) .attachments',
+      css: '.active .ticket-article-item:nth-child(3) .attachments',
       value: 'upload2.jpg',
     )
     match(
-      css: '.active .ticket-article-item:nth-child(2) .attachments',
+      css: '.active .ticket-article-item:nth-child(3) .attachments',
       value: 'upload1.txt',
     )
 
@@ -176,11 +176,11 @@ class AgentTicketActionLevel6Test < TestCase
       },
     )
     match(
-      css: '.active .ticket-article-item:nth-child(3) .attachments',
+      css: '.active .ticket-article-item:nth-child(4) .attachments',
       value: 'upload2.jpg',
     )
     match(
-      css: '.active .ticket-article-item:nth-child(3) .attachments',
+      css: '.active .ticket-article-item:nth-child(4) .attachments',
       value: 'upload1.txt',
     )
     #