Updated CHANGELING.

# Conflicts:
#	README.md

.pkgr.yml

@@ -27,7 +27,6 @@ before:
   - uname -a
   - ruby -v
   - env
-  - echo "gem 'mysql2'" >> Gemfile.local
   - "cat Gemfile.lock"
   - contrib/cleanup.sh
 env:

CHANGELOG.md

@@ -0,0 +1,89 @@
+# Change Log
+
+## [1.1.2](https://github.com/zammad/zammad/tree/1.1.2) (2017-01-16)
+[Full Changelog](https://github.com/zammad/zammad/compare/1.1.1...1.1.2)
+
+**Fixed bugs:**
+
+- New ticket attributes are not shown for customer [\#576](https://github.com/zammad/zammad/issues/576) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Agent - Group assignment shows "\<br\>" [\#566](https://github.com/zammad/zammad/issues/566) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Renaming standard roles breaks ticket views [\#499](https://github.com/zammad/zammad/issues/499) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Overview is not show it organization is used as row and first ticket has no organization \(app is not responding anymore\) [\#554](https://github.com/zammad/zammad/issues/554) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Layout broken if outbound channel has a error message [\#547](https://github.com/zammad/zammad/issues/547) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Error message at creation of new overview with not existing specific customer [\#545](https://github.com/zammad/zammad/issues/545) [[bug](https://github.com/zammad/zammad/labels/bug)]
+
+## [1.1.1](https://github.com/zammad/zammad/tree/1.1.1) (2016-12-14)
+[Full Changelog](https://github.com/zammad/zammad/compare/1.1.0...1.1.1)
+
+**Implemented enhancements:**
+
+- Improved ticket number generator settings \(just show necessary settings based on generator selection\) [\#427](https://github.com/zammad/zammad/issues/427) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+- Provide a Debian Package [\#209](https://github.com/zammad/zammad/issues/209) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+
+**Fixed bugs:**
+
+- "Ticket will escalate soon" in online notifications is not translated. [\#408](https://github.com/zammad/zammad/issues/408) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Can't process email / Error creating ticket with changed Ticket Number Format [\#413](https://github.com/zammad/zammad/issues/413) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Unable to set owner + customer in postmaster/email filter [\#419](https://github.com/zammad/zammad/issues/419) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Switching storage provider for attachments in UI has no effect [\#428](https://github.com/zammad/zammad/issues/428) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Unable to execute more then one trigger \(e. g. adding 2 tags with 2 triggers\) at one ticket at same update cycle [\#441](https://github.com/zammad/zammad/issues/441) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Customer should not have the internal attribute to set in customer interface. [\#437](https://github.com/zammad/zammad/issues/437) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Unable to create job/scheduler if many time options are selected [\#432](https://github.com/zammad/zammad/issues/432) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- New overview with article conditions not working [\#448](https://github.com/zammad/zammad/issues/448) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Debian 8 mysql2.so LoadError [\#415](https://github.com/zammad/zammad/issues/415) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Missing quotation marks on sender address [\#402](https://github.com/zammad/zammad/issues/402) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Search result won't get updated after focus is lost and new search is started [\#466](https://github.com/zammad/zammad/issues/466) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- "Password has been changed" email contains unresolved placeholders [\#468](https://github.com/zammad/zammad/issues/468) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Token without expiration timestamp expires "01/01/1970"  [\#478](https://github.com/zammad/zammad/issues/478) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- "Get latest Translations" fails\(?\) without error [\#502](https://github.com/zammad/zammad/issues/502) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Pending event time after 22:59 jumps back to 22:59 [\#329](https://github.com/zammad/zammad/issues/329) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Max times for escalation 99 hours [\#418](https://github.com/zammad/zammad/issues/418) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Object manager allows method names as attributes [\#433](https://github.com/zammad/zammad/issues/433) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- bugfix: added missing join table [\#541](https://github.com/zammad/zammad/pull/541) [[bug](https://github.com/zammad/zammad/labels/bug)] ([TeaMoe](https://github.com/TeaMoe))
+- New users won't get initial user group [\#249](https://githuvb.com/zammad/zammad/issues/249) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Unable to c&p images in Zammad Text-Editor with Firefox \(FF\) and Safari [\#505](https://github.com/zammad/zammad/issues/505) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Fix spelling mistakes [\#470](https://github.com/zammad/zammad/pull/470) [[bug](https://github.com/zammad/zammad/labels/bug)] ([corny](https://github.com/corny))
+
+## [1.1.0](https://github.com/zammad/zammad/tree/1.1.0) (2016-11-14)
+[Full Changelog](https://github.com/zammad/zammad/compare/1.0.1...1.1.0)
+
+**Implemented enhancements:**
+
+- Missing translation/edit abilities \(Notification mails, contact form\) [\#388](https://github.com/zammad/zammad/issues/388) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+- Automatically join users to organizations based on the email address [\#356](https://github.com/zammad/zammad/issues/356) [[feature](https://github.com/zammad/zammad/labels/feature)]
+- Create emails via fetchmail / procmail [\#326](https://github.com/zammad/zammad/issues/326) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+- Display origin url from tweet/facebook post [\#317](https://github.com/zammad/zammad/issues/317) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+- Vagrant version [\#253](https://github.com/zammad/zammad/issues/253) [[feature](https://github.com/zammad/zammad/labels/feature)]
+- Page posts from facebook and tweets from twitter as tickets [\#251](https://github.com/zammad/zammad/issues/251) [[enhancement](https://github.com/zammad/zammad/labels/enhancement)]
+
+**Fixed bugs:**
+
+- Some windows in the config area too wide in firefox [\#392](https://github.com/zammad/zammad/issues/392) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- \[Critical\] Short display of "Setup a new system" on public site while restarting Zammad! [\#389](https://github.com/zammad/zammad/issues/389) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Mailresponse to customer not working at all after editing a "Trigger" event text [\#385](https://github.com/zammad/zammad/issues/385) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Click on "Reports" leads to Error 500 [\#384](https://github.com/zammad/zammad/issues/384) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- TicketSelector in Overview or SLA not working with filter for ticket customer or owner [\#383](https://github.com/zammad/zammad/issues/383) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Changelog generator breaks packager.io builds [\#379](https://github.com/zammad/zammad/issues/379) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- script/scheduler.rb crashes after recreation of Zammad database [\#374](https://github.com/zammad/zammad/issues/374) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- ticket search cache issue [\#349](https://github.com/zammad/zammad/issues/349) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- IMAP mail fetching stops because of broken spam email \(invalid Content-Transfer-Encoding header\) [\#348](https://github.com/zammad/zammad/issues/348) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- IMAP mail fetching stops working on spam mail and invalid email format [\#345](https://github.com/zammad/zammad/issues/345) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Unable to create tickets [\#323](https://github.com/zammad/zammad/issues/323) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Admin -\> Channels -\> Email -\> Filters not shown / no name shown [\#313](https://github.com/zammad/zammad/issues/313) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- In object manager, blank line in object deleted all values for selections [\#312](https://github.com/zammad/zammad/issues/312) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- triggers do not work? [\#306](https://github.com/zammad/zammad/issues/306) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Answers from google mail shows complete quote [\#286](https://github.com/zammad/zammad/issues/286) [[minor bug](https://github.com/zammad/zammad/labels/minor%20bug)]
+- Avatar in Chat not showing - wrong url [\#275](https://github.com/zammad/zammad/issues/275) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- chat not working / http 404 [\#274](https://github.com/zammad/zammad/issues/274) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Zammad Docker container for rpi/ARM [\#271](https://github.com/zammad/zammad/issues/271) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Zendesk: unknown article type fails import. [\#270](https://github.com/zammad/zammad/issues/270) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- No space at comment on ticket bulk action  [\#268](https://github.com/zammad/zammad/issues/268) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- After adding a new attribute - zammad won't start [\#248](https://github.com/zammad/zammad/issues/248) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Images/Avatars blank [\#217](https://github.com/zammad/zammad/issues/217) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Docker/Ubuntu Not starting postgresql-9.6 service [\#213](https://github.com/zammad/zammad/issues/213) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- Can´t choose Organization for Customer [\#211](https://github.com/zammad/zammad/issues/211) [[bug](https://github.com/zammad/zammad/labels/bug)]
+- CentOS 7 HTTPS Error 404 - Not Found [\#210](https://github.com/zammad/zammad/issues/210) [[bug](https://github.com/zammad/zammad/labels/bug)]
+
+
+
+\* *This Change Log was automatically generated by [github_changelog_generator](https://github.com/skywinder/Github-Changelog-Generator)*

app/assets/javascripts/app/controllers/_application_controller_form.coffee

@@ -435,7 +435,7 @@ class App.ControllerForm extends App.Controller
         else
           value = parseInt(value)
       if param[item.name] isnt undefined
-        if typeof param[item.name] is 'string'
+        if typeof param[item.name] is 'string' || typeof param[item.name] is 'boolean' || typeof param[item.name] is 'number'
           param[item.name] = [param[item.name], value]
         else
           param[item.name].push value

app/assets/javascripts/app/controllers/_integration/sipgate_io.coffee

@@ -6,6 +6,9 @@ class Index extends App.ControllerIntegrationBase
     ['This service shows you contacts of incoming calls and a caller list in realtime.']
     ['Also caller id of outbound calls can be changed.']
   ]
+  events:
+    'click .js-select': 'selectAll'
+    'change .js-switch input': 'switch'
 
   render: =>
     super

app/assets/javascripts/app/controllers/_settings/area.coffee

@@ -35,10 +35,11 @@ class App.SettingsArea extends App.Controller
 
     elements = []
     for setting in settings
-      if setting.preferences.controller && App[setting.preferences.controller]
-        item = new App[setting.preferences.controller](setting: setting)
-      else
-        item = new App.SettingsAreaItem(setting: setting)
-      elements.push item.el
+      if setting.preferences.hidden isnt true
+        if setting.preferences.controller && App[setting.preferences.controller]
+          item = new App[setting.preferences.controller](setting: setting)
+        else
+          item = new App.SettingsAreaItem(setting: setting)
+        elements.push item.el
 
     @html elements

app/assets/javascripts/app/controllers/_settings/area_item.coffee

@@ -1,4 +1,5 @@
 class App.SettingsAreaItem extends App.Controller
+  template: 'settings/item'
   events:
     'submit form': 'update'
 
@@ -21,8 +22,12 @@ class App.SettingsAreaItem extends App.Controller
     # form
     @configure_attributes = @setting.options['form']
 
+    for attribute in @configure_attributes
+      if attribute.tag is 'boolean'
+        attribute.translate = true
+
     # item
-    @html App.view('settings/item')(
+    @html App.view(@template)(
       setting: @setting
     )
 

app/assets/javascripts/app/controllers/_settings/area_storage_provider.coffee

@@ -0,0 +1,2 @@
+class App.SettingsAreaStorageProvider extends App.SettingsAreaItem
+  template: 'settings/storage_provider'

app/assets/javascripts/app/controllers/_settings/area_ticket_hook_position.coffee

@@ -0,0 +1,2 @@
+class App.SettingsAreaTicketHookPosition extends App.SettingsAreaItem
+  template: 'settings/ticket_hook_position'

app/assets/javascripts/app/controllers/_settings/area_ticket_number.coffee

@@ -0,0 +1,132 @@
+class App.SettingsAreaTicketNumber extends App.Controller
+  events:
+    'submit form': 'update'
+
+  constructor: ->
+    super
+    @render()
+
+  render: =>
+
+    # defaults
+    directValue = 0
+    for item in @setting.options['form']
+      directValue += 1
+    if directValue > 1
+      for item in @setting.options['form']
+        item['default'] = @setting.state_current.value[item.name]
+    else
+      item['default'] = @setting.state_current.value
+
+    @map =
+      'Ticket::Number::Increment': 'ticket_number_increment'
+      'Ticket::Number::Date': 'ticket_number_date'
+
+    # form
+    @configure_attributes = @setting.options['form']
+
+    # item
+    @html App.view('settings/ticket_number')(
+      setting: @setting
+    )
+
+    togglePreferences = (params, attribute, @attributes, classname, form) =>
+      return if attribute.name isnt 'ticket_number'
+      @showPreferences(params.ticket_number)
+
+    updatePreview = (params, attribute) =>
+      paramsParent = @formParam(@$('.js-form'))
+      number = "#{App.Config.get('ticket_hook')}???"
+      if paramsParent.ticket_number is 'Ticket::Number::Increment'
+        paramsItem = @paramsPreferences('Ticket::Number::Increment')
+        number     = "#{App.Config.get('ticket_hook')}#{App.Config.get('system_id')}"
+        counter    = '1'
+        if paramsItem.min_size
+          minSize = parseInt(paramsItem.min_size)
+          if paramsItem.checksum
+            minSize -= 1
+          if minSize > 1
+            for itemCounter in [2 .. minSize]
+              counter = "0#{counter}"
+        number += counter
+        if paramsItem.checksum
+          number += '9'
+      else if paramsParent.ticket_number is 'Ticket::Number::Date'
+        paramsItem   = @paramsPreferences('Ticket::Number::Date')
+        current      = new Date()
+        currentDay   = current.getDate()
+        currentMonth = current.getMonth() + 1
+        currentYear  = current.getFullYear()
+
+        number = "#{App.Config.get('ticket_hook')}#{currentYear}#{currentMonth}#{currentDay}#{App.Config.get('system_id')}001"
+        if paramsItem.checksum
+          number += '9'
+
+      @$('.js-preview').text(number)
+
+    new App.ControllerForm(
+      el: @el.find('.js-form'),
+      model: { configure_attributes: @configure_attributes, className: '' }
+      autofocus: false
+      handlers: [togglePreferences, updatePreview]
+    )
+
+    # preferences
+    preferences_settings = @setting.preferences.settings_included || ['ticket_number_increment', 'ticket_number_date']
+    for preferences_setting in preferences_settings
+      setting = App.Setting.findByAttribute('name', preferences_setting)
+      value = App.Setting.get(preferences_setting)
+      el = $(App.view("settings/#{preferences_setting}")(
+        setting: setting
+      ))
+      new App.ControllerForm(
+        el: el.find('.js-formItem'),
+        model: { configure_attributes: setting.options['form'], className: '' }
+        autofocus: false
+        params: value
+        handlers: [updatePreview]
+      )
+      @$('.js-formPreferences').append(el)
+
+    # show current preferences
+    @showPreferences(item['default'])
+
+  showPreferences: (name) =>
+    @$('.js-formPreferencesItem').addClass('hidden')
+    @$(".js-formPreferencesItem[data-backend=\"#{name}\"]").removeClass('hidden')
+
+  paramsPreferences: (name) =>
+    @formParam(@$(".js-formPreferencesItem[data-backend=\"#{name}\"] form"))
+
+  update: (e) =>
+    e.preventDefault()
+    @formDisable(@$('.js-form'))
+    params = @formParam(@$('.js-form'))
+    if params.ticket_number
+      paramsItem = @paramsPreferences(params.ticket_number)
+      setting_name = @map[params.ticket_number]
+      if setting_name && paramsItem
+        App.Setting.set(setting_name, paramsItem)
+
+    @setting['state_current'] = {value: params.ticket_number}
+    ui = @
+    @setting.save(
+      done: =>
+        ui.formEnable(e)
+        App.Event.trigger 'notify', {
+          type:    'success'
+          msg:     App.i18n.translateContent('Update successful!')
+          timeout: 2000
+        }
+
+        # rerender ui || get new collections and session data
+        App.Setting.preferencesPost(@setting)
+
+      fail: (settings, details) ->
+        ui.formEnable(e)
+        App.Event.trigger 'notify', {
+          type:    'error'
+          msg:     App.i18n.translateContent(details.error_human || details.error || 'Unable to update object!')
+          timeout: 2000
+        }
+    )

app/assets/javascripts/app/controllers/_ui_element/object_manager_attribute.coffee

@@ -11,7 +11,6 @@ class App.UiElement.object_manager_attribute extends App.UiElement.ApplicationUi
 
     updateDataMap = (localParams, localAttribute, localAttributes, localClassname, localForm, localA) =>
       localItem = localForm.closest('.js-data')
-      console.log('updateDataMap', attribute, params)
       element = $(App.view("object_manager/attribute/#{localParams.data_type}")(
         attribute: attribute
         params: params
@@ -49,7 +48,7 @@ class App.UiElement.object_manager_attribute extends App.UiElement.ApplicationUi
     objects =
       Ticket:
         Customer:
-          create:
+          create_bottom:
             shown: true
             required: false
           edit:

app/assets/javascripts/app/controllers/_ui_element/postmaster_set.coffee

@@ -16,8 +16,11 @@ class App.UiElement.postmaster_set
             relation: 'TicketState'
           },
           {
-            value:    'x-zammad-ticket-customer'
+            value:    'x-zammad-ticket-customer_id'
             name:     'Customer'
+            relation: 'User'
+            tag:      'user_autocompletion'
+            disableCreateUser: true,
           },
           {
 
@@ -26,10 +29,11 @@ class App.UiElement.postmaster_set
             relation: 'Group'
           },
           {
-            value:    'x-zammad-ticket-owner'
+            value:    'x-zammad-ticket-owner_id'
             name:     'Owner'
-            relation: 'User',
-            tag:      'user_autocompletion',
+            relation: 'User'
+            tag:      'user_autocompletion'
+            disableCreateUser: true,
           },
           {
             value:    'x-zammad-ignore'

app/assets/javascripts/app/controllers/_ui_element/ticket_selector.coffee

@@ -368,9 +368,11 @@ class App.UiElement.ticket_selector
       if config.relation is 'User'
         config.multiple = false
         config.nulloption = false
+        config.guess = false
       if config.relation is 'Organization'
         config.multiple = false
         config.nulloption = false
+        config.guess = false
       if config.tag is 'checkbox'
         config.tag = 'select'
       tagSearch = "#{config.tag}_search"

app/assets/javascripts/app/controllers/_ui_element/user_permission.coffee

@@ -102,12 +102,20 @@ class App.UiElement.user_permission
       if !checked
         if rolesWithGroupPlugin[role_id] is 'group'
           item.find('.js-groupList').addClass('hidden')
+
+          # select groups if only one is available
+          if hideGroups
+            item.find('.js-groupList [name=group_ids]').prop('checked', false)
         return
 
       # if role with groups plugin is selected, show group selection
       if rolesWithGroupPlugin[role_id] is 'group'
         item.find('.js-groupList:not(.js-groupListHide)').removeClass('hidden')
 
+        # select groups if only one is available
+        if hideGroups
+          item.find('.js-groupList [name=group_ids]').prop('checked', true)
+
       for trigger in triggers
         trigger.trigger('change')
     )

app/assets/javascripts/app/controllers/customer_ticket_create.coffee

@@ -68,17 +68,18 @@ class Index extends App.ControllerContent
       params:     defaults
       noFieldset: true
     )
-    #new App.ControllerForm(
-    #  el:       @el.find('.ticket-form-bottom')
-    #  form_id:  @form_id
-    #  model:    App.Ticket
-    #  screen:   'create_bottom'#@article_attributes['screen']
-    #  handlers: [
-    #    formChanges
-    #  ]
-    #  filter:    @formMeta.filter
-    #  params:    defaults
-    #)
+    if !_.isEmpty(App.Ticket.attributesGet('create_bottom', false, true))
+      new App.ControllerForm(
+        el:       @el.find('.ticket-form-bottom')
+        form_id:  @form_id
+        model:    App.Ticket
+        screen:   'create_bottom'
+        handlers: [
+          @ticketFormChanges
+        ]
+        filter:    @formMeta.filter
+        params:    defaults
+      )
 
     new App.ControllerDrox(
       el:   @el.find('.sidebar')

app/assets/javascripts/app/controllers/ticket_overview.coffee

@@ -424,11 +424,13 @@ class Table extends App.Controller
         attribute.link = object.uiUrl()
         value
       callbackUserPopover = (value, object, attribute, attributes, refObject) ->
+        return value if !refObject
         attribute.class = 'user-popover'
         attribute.data =
           id: refObject.id
         value
       callbackOrganizationPopover = (value, object, attribute, attributes, refObject) ->
+        return value if !refObject
         attribute.class = 'organization-popover'
         attribute.data =
           id: refObject.id

app/assets/javascripts/app/controllers/ticket_zoom/article_actions.coffee

@@ -123,6 +123,13 @@ class App.TicketZoomArticleActions extends App.Controller
         icon: 'reply'
         href: '#'
       }
+    if article.sender.name is 'Agent' && article.type.name is 'phone'
+      actions.push {
+        name: 'reply'
+        type: 'emailReply'
+        icon: 'reply'
+        href: '#'
+      }
     if article.type.name is 'twitter status'
       actions.push {
         name: 'reply'
@@ -284,6 +291,7 @@ class App.TicketZoomArticleActions extends App.Controller
     # get reference article
     article_id = $(e.target).parents('[data-id]').data('id')
     article    = App.TicketArticle.fullLocal(article_id)
+    ticket     = App.Ticket.fullLocal(article.ticket_id)
     type       = App.TicketArticleType.find(article.type_id)
     customer   = App.User.find(article.created_by_id)
 
@@ -304,15 +312,31 @@ class App.TicketZoomArticleActions extends App.Controller
 
     if type.name is 'email' || type.name is 'phone' || type.name is 'web'
 
-      if article.sender.name is 'Agent'
-        articleNew.to = article.to
-      else
-        articleNew.to = article.from
+      if type.name is 'phone'
+
+        # inbound call
+        if article.sender.name is 'Agent'
+          articleNew.to = article.to
+
+        # outbound call
+        else
+          articleNew.to = article.to
 
         # if sender is customer but in article.from is no email, try to get
         # customers email via customer user
         if articleNew.to && !articleNew.to.match(/@/)
-          articleNew.to = article.created_by.email
+          articleNew.to = ticket.customer.email
+
+      else
+        if article.sender.name is 'Agent'
+          articleNew.to = article.to
+        else
+          articleNew.to = article.from
+
+          # if sender is customer but in article.from is no email, try to get
+          # customers email via customer user
+          if articleNew.to && !articleNew.to.match(/@/)
+            articleNew.to = article.created_by.email
 
       # filter for uniq recipients
       recipientAddresses = {}

app/assets/javascripts/app/controllers/ticket_zoom/article_new.coffee

@@ -48,6 +48,7 @@ class App.TicketZoomArticleNew extends App.Controller
         possibleArticleType['email'] = true
 
     # gets referenced in @setArticleType
+    @internalSelector = true
     @type = @defaults['type'] || 'note'
     @articleTypes = []
     if possibleArticleType.note
@@ -115,6 +116,9 @@ class App.TicketZoomArticleNew extends App.Controller
         },
       ]
 
+    if @permissionCheck('ticket.customer')
+      @internalSelector = false
+
     @textareaHeight =
       open:   148
       closed: 20
@@ -194,11 +198,12 @@ class App.TicketZoomArticleNew extends App.Controller
     ticket = App.Ticket.fullLocal(@ticket_id)
 
     @html App.view('ticket_zoom/article_new')(
-      ticket:       ticket
-      articleTypes: @articleTypes
-      article:      @defaults
-      form_id:      @form_id
-      isCustomer:   @permissionCheck('ticket.customer')
+      ticket:           ticket
+      articleTypes:     @articleTypes
+      article:          @defaults
+      form_id:          @form_id
+      isCustomer:       @permissionCheck('ticket.customer')
+      internalSelector: @internalSelector
     )
     @setArticleType(@type)
 

app/assets/javascripts/app/controllers/translation.coffee

@@ -164,14 +164,16 @@ class Index extends App.ControllerSubContent
       type:        'GET'
       url:         "#{@apiPath}/translations/sync/#{locale.locale}"
       processData: false
-      complete: (data, status, xhr) =>
+      success: =>
         loader.update(locale.name, false)
         locale = locales.shift()
         if _.isEmpty(locales)
           hide()
           return
         @_syncChanges(locale, locales, loader, hide)
-      )
+      error: ->
+        hide()
+    )
 
   updateOnServer: (params, event) =>
 

app/assets/javascripts/app/controllers/widget/ticket_list.coffee

@@ -16,11 +16,13 @@ class App.TicketList extends App.Controller
       attribute.link = object.uiUrl()
       value
     callbackUserPopover = (value, object, attribute, attributes, refObject) ->
+      return value if !refObject
       attribute.class = 'user-popover'
       attribute.data =
         id: refObject.id
       value
     callbackOrganizationPopover = (value, object, attribute, attributes, refObject) ->
+      return value if !refObject
       attribute.class = 'organization-popover'
       attribute.data =
         id: refObject.id

app/assets/javascripts/app/lib/app_post/global_search.coffee

@@ -9,7 +9,8 @@ class App.GlobalSearch
 class _globalSearchSingleton extends Spine.Module
 
   constructor: ->
-    @searchResultCache = {}
+    @searchResultCache = undefined
+    @searchResultCacheByKey = {}
     @apiPath = App.Config.get('api_path')
 
   execute: (params) ->
@@ -20,8 +21,8 @@ class _globalSearchSingleton extends Spine.Module
 
     # use cache for search result
     currentTime = new Date
-    if @searchResultCache[cacheKey] && @searchResultCache[cacheKey].time > currentTime.setSeconds(currentTime.getSeconds() - 20)
-      render(@searchResultCache[cacheKey].result)
+    if @searchResultCacheByKey[cacheKey] && @searchResultCacheByKey[cacheKey].time > currentTime.setSeconds(currentTime.getSeconds() - 20)
+      @renderTry(render, @searchResultCacheByKey[cacheKey].result, cacheKey)
       return
 
     App.Ajax.request(
@@ -31,7 +32,7 @@ class _globalSearchSingleton extends Spine.Module
       data:
         query: query
         limit: limit
-      processData: true,
+      processData: true
       success: (data, status, xhr) =>
         App.Collection.loadAssets(data.assets)
         result = {}
@@ -48,19 +49,21 @@ class _globalSearchSingleton extends Spine.Module
           else
             App.Log.error('_globalSearchSingleton', "No such model App.#{item.type}")
 
-        diff = false
-        if @searchResultCache[cacheKey]
-          diff = difference(@searchResultCache[cacheKey].resultRaw, data.result)
-
-        # cache search result
-        @searchResultCache[cacheKey] =
-          result: result
-          resultRaw: data.result
-          limit: limit
-          time: new Date
-
-        # if result hasn't changed, do not rerender
-        return if diff isnt false && _.isEmpty(diff)
-
-        render(result)
+        @renderTry(render, result, cacheKey)
     )
+
+  renderTry: (render, result, cacheKey) =>
+
+    # if result hasn't changed, do not rerender
+    diff = false
+    if @searchResultCache
+      diff = difference(@searchResultCache, result)
+    return if diff isnt false && _.isEmpty(diff)
+
+    # cache search result
+    @searchResultCache = result
+    @searchResultCacheByKey[cacheKey] =
+      result: result
+      time: new Date
+
+    render(result)

app/assets/javascripts/app/lib/app_post/i18n.coffee

@@ -288,9 +288,11 @@ class _i18nSingleton extends Spine.Module
     @_notTranslated[locale][key] = true
 
   date: (time, offset) =>
+    return time if !time
     @convert(time, offset, @mapTime['date'] || @dateFormat)
 
   timestamp: (time, offset) =>
+    return time if !time
     @convert(time, offset, @mapTime['timestamp'] || @timestampFormat)
 
   convert: (time, offset, format) ->

app/assets/javascripts/app/lib/app_post/utils.coffee

@@ -206,7 +206,7 @@ class App.Utils
 
     # Replace all x tags with the type of replacementTag
     html.find('textarea').each( ->
-      outer = @outerHTML;
+      outer = @outerHTML
 
       # Replace opening tag
       regex = new RegExp('<' + @tagName, 'i')

app/assets/javascripts/app/lib/bootstrap/bootstrap-timepicker.js

@@ -485,13 +485,21 @@
     },
 
     highlightHour: function() {
+      var time = this.$element.val(),
+        timeArray = time.replace(/[^0-9\:]/g, '').split(':'),
+        hour = timeArray[0] ? timeArray[0].toString() : timeArray.toString(),
+        hour_max_length = hour.toString().length
       this.highlightedUnit = 'hour';
-      this.setSelectionRange(0, 2);
+      this.setSelectionRange(0, hour_max_length);
     },
 
     highlightMinute: function() {
+      var time = this.$element.val(),
+        timeArray = time.replace(/[^0-9\:]/g, '').split(':'),
+        hour = timeArray[0] ? timeArray[0].toString() : timeArray.toString(),
+        hour_max_length = hour.toString().length
       this.highlightedUnit = 'minute';
-      this.setSelectionRange(3, 5);
+      this.setSelectionRange(hour_max_length+1, hour_max_length+3);
     },
 
     highlightSecond: function() {
@@ -790,7 +798,8 @@
 
         hour = timeArray[0] ? timeArray[0].toString() : timeArray.toString();
 
-        if(this.explicitMode && hour.length > 2 && (hour.length % 2) !== 0 ) {
+        var hour_max_length = this.maxHours.toString().length
+        if(this.explicitMode && hour.length > hour_max_length && (hour.length % 2) !== 0 ) {
           this.clear();
           return;
         }
@@ -804,9 +813,9 @@
           hour = hour.slice(0, -2);
         }
 
-        if (hour.length > 2) {
-          minute = hour.slice(-2);
-          hour = hour.slice(0, -2);
+        if (hour.length > hour_max_length) {
+          minute = hour.slice(-hour_max_length);
+          hour = hour.slice(0, -hour_max_length);
         }
 
         if (minute.length > 2) {
@@ -838,9 +847,9 @@
           minute = 59;
         }
 
-        if (hour >= this.maxHours) {
+        if (hour > this.maxHours) {
           // No day/date handling.
-          hour = this.maxHours - 1;
+          hour = this.maxHours;
         }
 
         if (this.showMeridian) {
@@ -859,7 +868,7 @@
         } else if (hour < 12 && timeMode === 2) {
           hour += 12;
         } else {
-          if (hour >= this.maxHours) {
+          if (hour > this.maxHours) {
             hour = this.maxHours - 1;
           } else if ((hour < 0) || (hour === 12 && timeMode === 1)){
             hour = 0;

app/assets/javascripts/app/models/_application_model.coffee

@@ -184,7 +184,7 @@ class App.Model extends Spine.Model
 
   ###
 
-  @attributesGet: (screen = undefined, attributes = false) ->
+  @attributesGet: (screen = undefined, attributes = false, noDefaultAttributes = false) ->
     if !attributes
       attributes = clone(App[ @.className ].configure_attributes, true)
     else
@@ -203,7 +203,7 @@ class App.Model extends Spine.Model
           attributesNew[ attribute.name ] = attribute
 
     # if no screen is given or no attribute has this screen - use default attributes
-    if !screen || _.isEmpty(attributesNew)
+    if (!screen || _.isEmpty(attributesNew)) && !noDefaultAttributes
       for attribute in attributes
         attributesNew[ attribute.name ] = attribute
 

app/assets/javascripts/app/views/channel/email_account_overview.jst.eco

@@ -108,7 +108,7 @@
         <% if !_.isEmpty(channel.last_log_out): %>
         <div class="alert alert--danger">
           <%= channel.last_log_out %>
-        <div>
+        </div>
         <% end %>
 
         <hr>

app/assets/javascripts/app/views/customer_ticket_create.jst.eco

@@ -3,7 +3,7 @@
     <div class="newTicket">
       <div class="box box--newTicket">
         <div class="page-header">
-          <h1><%- @T( 'New Ticket' ) %></h1>
+          <h1><%- @T('New Ticket') %></h1>
         </div>
 
         <div class="page-content">
@@ -18,7 +18,6 @@
               <div class="ticket-form-bottom"></div>
             </div>
 
-
             <div class="form-controls">
               <a class="btn btn--text btn--subtle js-cancel" href="#/"><%- @T( 'Cancel & Go Back' ) %></a>
               <button type="submit" class="btn btn--success js-submit align-right"><%- @T( 'Create' ) %></button>

app/assets/javascripts/app/views/generic/user_permission.jst.eco

@@ -15,7 +15,7 @@
           <input type="checkbox" value="<%= group.id %>" name="group_ids" <% if @groupsSelected[group.id]: %>checked<% end %>/>
           <%- @Icon('checkbox', 'icon-unchecked') %>
           <%- @Icon('checkbox-checked', 'icon-checked') %>
-          <span class="label-text"><%= group.displayName() %> <% if group.note: %>- <span class="help-text"><%= group.note %></span><% end %></span>
+          <span class="label-text"><%= group.displayName() %> <% if group.note: %>- <span class="help-text"><%- group.note %></span><% end %></span>
         </label>
       <% end %>
     </div>

app/assets/javascripts/app/views/integration/sipgate.jst.eco

@@ -1,5 +1,27 @@
 <form>
 
+  <h2>sipgate.io <%- @T('Settings') %></h2>
+
+  <p><%- @T('You need to configure the Zammad endpoints in the Sipgate web interface') %>:<p>
+
+  <div class="settings-entry">
+    <table class="settings-list" style="width: 100%;">
+      <thead>
+        <tr>
+          <th width="20%"><%- @T('Type') %>
+          <th width="80%"><%- @T('URL') %>
+      </thead>
+      <tbody>
+        <tr>
+          <td class="settings-list-row-control"><%- @T('Inbound') %>
+          <td class="settings-list-control-cell"><input type="url" class="form-control form-control--small js-select" readonly value="<%- @C('http_type') %>://<%- @C('fqdn') %>/api/v1/sipgate/in">
+        <tr>
+          <td class="settings-list-row-control"><%- @T('Outbound') %>
+          <td class="settings-list-control-cell"><input type="url" class="form-control form-control--small js-select" readonly value="<%- @C('http_type') %>://<%- @C('fqdn') %>/api/v1/sipgate/out">
+      </tbody>
+    </table>
+  </div>
+
   <h2><%- @T('Inbound') %></h2>
 
   <p><%- @T('Blocked caller ids based on sender caller id.') %>

app/assets/javascripts/app/views/settings/item.jst.eco

@@ -1,6 +1,6 @@
 <form class="settings-entry" id="<%= @setting.name %>">
   <h2><%- @T(@setting.title) %></h2>
-  <p class="help-text"><%- @RichText(@setting.description) %></p>
+  <p class="help-text"><%- @T(@setting.description) %></p>
   <div class="horizontal end">
     <div class="form-item flex"></div>
     <button type="submit" class="btn btn--primary"><%- @T('Submit') %></button>

app/assets/javascripts/app/views/settings/storage_provider.jst.eco

@@ -0,0 +1,22 @@
+<form class="settings-entry" id="<%= @setting.name %>">
+  <h2><%- @T(@setting.title) %></h2>
+  <p class="help-text"><%- @T('You can switch between the backend for new attachments even on a system that is already in production without any loss of data.') %></p>
+  <p class="help-text"><%- @T('If you want to move already stored attachments from one backend to another, you need to execute the following via console.') %></p>
+  </p>
+  <p class="help-text"><%- @T('Move all from "%s" to "%s"', 'filesystem', 'database') %>:</p>
+  </p>
+  <code>
+  rails&gt; Store::File.move('File', 'DB')
+  </code>
+  <p class="help-text"><%- @T('Move all from "%s" to "%s"', 'database', 'filesystem') %>:</p>
+  </p>
+  <code>
+  rails&gt; Store::File.move('DB', 'File')
+  </code>
+  <br>
+  <br>
+  <div class="horizontal end">
+    <div class="form-item flex"></div>
+    <button type="submit" class="btn btn--primary"><%- @T('Submit') %></button>
+  </div>
+</form>

app/assets/javascripts/app/views/settings/ticket_hook_position.jst.eco

@@ -0,0 +1,14 @@
+<form class="settings-entry" id="<%= @setting.name %>">
+  <h2><%- @T(@setting.title) %></h2>
+  <p class="help-text">
+  <%- @T('The format of the subject.') %>
+  <ul>
+    <li><%- @T('|Right| means |Some Subject [Ticket#12345]|') %>
+    <li><%- @T('|Left| means |[Ticket#12345] Some Subject|') %>
+    <li><%- @T('|None| means |Some Subject| (without ticket number). In the last case you should enable "postmaster___follow___up___search___in" to recognize followups based on email headers and/or body.') %>
+  </ul>
+  <div class="horizontal end">
+    <div class="js-form form-item flex"></div>
+    <button type="submit" class="btn btn--primary"><%- @T('Submit') %></button>
+  </div>
+</form>

app/assets/javascripts/app/views/settings/ticket_number.jst.eco

@@ -0,0 +1,19 @@
+<div class="settings-entry" id="<%= @setting.name %>">
+  <h2><%- @T(@setting.title) %></h2>
+  <p class="help-text">
+  <%- @T('Selects the ticket number generator module.') %>
+  <ul>
+    <li><%- @T('|Increment| increments the ticket number, the SystemID and the counter are used with "SystemID.Counter" format (e.g. 1010138, 1010139).') %>
+    <li><%- @T('With |Date| the ticket numbers will be generated by the current date, the SystemID and the counter. The format looks like "Year.Month.Day.SystemID.Counter" (e.g. 201206231010138, 201206231010139).') %>
+  </ul>
+  <form class="js-form"></form>
+  <br>
+  <div class="js-formPreferences"></div>
+  <br>
+  <label class="formGroup-label"><%- @T('Preview') %></label>
+  <div class="js-preview"></div>
+  <br>
+  <form class="horizontal end">
+    <button type="submit" class="btn btn--primary"><%- @T('Submit') %></button>
+  </form>
+</div>

app/assets/javascripts/app/views/settings/ticket_number_date.jst.eco

@@ -0,0 +1,6 @@
+<div class="js-date js-formPreferencesItem hidden" data-backend="Ticket::Number::Date">
+  <label class="formGroup-label"><%- @T('Options') %></label>
+  <div><%- @T('With Checksum the counter will be appended with a checksum at the end. The format looks like "%s" (e. g. %s).', 'Year.Month.Day.SystemID.Counter.CheckSum', '2012070110101520, 2012070110101535') %></div>
+  <br>
+  <form class="js-formItem"></form>
+</div>

app/assets/javascripts/app/views/settings/ticket_number_increment.jst.eco

@@ -0,0 +1,6 @@
+<div class="js-increment js-formPreferencesItem hidden" data-backend="Ticket::Number::Increment">
+  <label class="formGroup-label"><%- @T('Options') %></label>
+  <div><%- @T('With Checksum the counter will be appended with a checksum at the end. The format looks like "%s" (e. g. %s).', 'SystemID.Counter.CheckSum', '10101384, 10101392') %></div>
+  <br>
+  <form class="js-formItem"></form>
+</div>

app/assets/javascripts/app/views/ticket_zoom/article_new.jst.eco

@@ -21,6 +21,7 @@
         <% end %>
       </div>
     </div>
+    <% if @internalSelector: %>
     <div class="editControls-item is-hidden js-toggleVisibility js-selectInternalPublic">
       <div class="editControls-iconHolder">
         <div class="editControls-icon icon-internal" title="<%- @Ti("set to public") %>">
@@ -31,6 +32,7 @@
         </div>
       </div>
     </div>
+    <% end %>
   </div>
   <div class="article-content bubble-gap">
     <div class="internal-border">

app/assets/stylesheets/zammad.scss

@@ -1594,7 +1594,6 @@ input[type=time] {
 
 input.timeframe,
 input.time {
-  width: 4.5ch;
   box-sizing: content-box;
   padding: 0 6px;
   line-height: 42px;
@@ -1605,6 +1604,13 @@ input.time {
   }
 }
 
+input.timeframe {
+  width: 5.5ch;
+}
+input.time {
+  width: 4.5ch;
+}
+
 input.time.time--12 {
   width: 7.5ch;
 }

app/models/activity_stream.rb

@@ -50,11 +50,7 @@ add a new activity entry for an object
 
     # resturn if old entry is really fresh
     if result
-      activity_record_delay = if ENV['ZAMMAD_ACTIVITY_RECORD_DELAY']
-                                ENV['ZAMMAD_ACTIVITY_RECORD_DELAY'].to_i.seconds
-                              else
-                                90.seconds
-                              end
+      activity_record_delay = 90.seconds
       return result if result.created_at.to_i >= ( data[:created_at].to_i - activity_record_delay )
     end
 
@@ -97,13 +93,10 @@ return all activity entries of an user
 =end
 
   def self.list(user, limit)
-    # do not return an activity stream for custoers
+    # do not return an activity stream for customers
     return [] if !user.permissions?('ticket.agent') && !user.permissions?('admin')
 
-    permission_ids = []
-    user.roles.each { |role|
-      permission_ids = permission_ids.concat(role.permission_ids)
-    }
+    permission_ids = user.permissions_with_child_ids
     group_ids = user.group_ids
 
     stream = if group_ids.empty?

app/models/channel/driver/smtp.rb

@@ -49,16 +49,21 @@ class Channel::Driver::Smtp
       options[:openssl_verify_mode] = 'none'
     end
     mail = Channel::EmailBuild.build(attr, notification)
-    mail.delivery_method :smtp, {
+    smtp_params = {
       openssl_verify_mode: options[:openssl_verify_mode],
       address: options[:host],
       port: options[:port],
       domain: options[:domain],
-      user_name: options[:user],
-      password: options[:password],
       enable_starttls_auto: options[:enable_starttls_auto],
-      authentication: options[:authentication],
     }
+
+    # add authentication only if needed
+    if options[:user] && !options[:user].empty?
+      smtp_params[:user_name] = options[:user]
+      smtp_params[:password] = options[:password]
+      smtp_params[:authentication] = options[:authentication]
+    end
+    mail.delivery_method :smtp, smtp_params
     mail.deliver
   end
 end

app/models/channel/email_build.rb

@@ -126,6 +126,21 @@ module Channel::EmailBuild
     mail
   end
 
+=begin
+
+  quoted_in_one_line = Channel::EmailBuild.recipient_line('Somebody @ "Company"', 'some.body@example.com')
+
+returnes
+
+  '"Somebody @ \"Company\"" <some.body@example.com>'
+
+=end
+
+  def self.recipient_line(realname, email)
+    return "#{realname} <#{email}>" if realname =~ /^[A-z]+$/i
+    "\"#{realname.gsub('"', '\"')}\" <#{email}>"
+  end
+
 =begin
 
 Check if string is a complete html document. If not, add head and css styles.

app/models/channel/filter/database.rb

@@ -3,10 +3,10 @@
 # process all database filter
 module Channel::Filter::Database
 
-  def self.run( _channel, mail )
+  def self.run(_channel, mail)
 
     # process postmaster filter
-    filters = PostmasterFilter.where( active: true, channel: 'email' ).order(:name, :created_at)
+    filters = PostmasterFilter.where(active: true, channel: 'email').order(:name, :created_at)
     filters.each { |filter|
       Rails.logger.info " process filter #{filter.name} ..."
       all_matches_ok = true

app/models/object_manager/attribute.rb

@@ -663,6 +663,14 @@ to send no browser reload event, pass false
       raise 'At least one letters is needed'
     elsif name =~ /^(destroy|true|false|integer|select|drop|create|alter|index|table|varchar|blob|date|datetime|timestamp)$/
       raise "#{name} is a reserved word, please choose a different one"
+
+    # do not allow model method names as attributes
+    else
+      model = Kernel.const_get(object_lookup.name)
+      record = model.new
+      if record.respond_to?(name.to_sym) && !record.attributes.key?(name)
+        raise "#{name} is a reserved word, please choose a different one"
+      end
     end
     true
   end

app/models/observer/ticket/article/fillup_from_email.rb

@@ -46,13 +46,13 @@ class Observer::Ticket::Article::FillupFromEmail < ActiveRecord::Observer
     if !email_address
       raise "No email address found for group '#{ticket.group.name}'"
     end
-    system_sender = "#{email_address.realname} <#{email_address.email}>"
     if record.created_by_id != 1 && Setting.get('ticket_define_email_from') == 'AgentNameSystemAddressName'
-      seperator = Setting.get('ticket_define_email_from_seperator')
-      sender    = User.find(record.created_by_id)
-      record.from = "#{sender.firstname} #{sender.lastname} #{seperator} #{system_sender}"
+      seperator   = Setting.get('ticket_define_email_from_seperator')
+      sender      = User.find(record.created_by_id)
+      realname    = "#{sender.firstname} #{sender.lastname} #{seperator} #{email_address.realname}"
+      record.from = Channel::EmailBuild.recipient_line(realname, email_address.email)
     else
-      record.from = system_sender
+      record.from = Channel::EmailBuild.recipient_line(email_address.realname, email_address.email)
     end
   end
 end

app/models/observer/ticket/article_changes.rb

@@ -30,6 +30,20 @@ class Observer::Ticket::ArticleChanges < ActiveRecord::Observer
     record.ticket.save
   end
 
+  def after_destroy(record)
+    changed = false
+    if article_count_update(record)
+      changed = true
+    end
+
+    # save ticket
+    if !changed
+      record.ticket.touch
+      return
+    end
+    record.ticket.save
+  end
+
   # get article count
   def article_count_update(record)
     current_count = record.ticket.article_count

app/models/observer/transaction.rb

@@ -29,6 +29,7 @@ class Observer::Transaction < ActiveRecord::Observer
     sync_backends = []
     Setting.where(area: 'Transaction::Backend::Sync').order(:name).each { |setting|
       backend = Setting.get(setting.name)
+      next if params[:disable] && params[:disable].include?(backend)
       sync_backends.push Kernel.const_get(backend)
     }
 

app/models/role.rb

@@ -52,7 +52,7 @@ get signup roles
 
   Role.signup_roles
 
-returnes
+returns
 
   [role1, role2, ...]
 
@@ -68,7 +68,7 @@ get signup role ids
 
   Role.signup_role_ids
 
-returnes
+returns
 
   [role1, role2, ...]
 

app/models/ticket.rb

@@ -844,6 +844,22 @@ result
     references
   end
 
+=begin
+
+get all articles of a ticket in correct order (overwrite active record default method)
+
+  artilces = ticket.articles
+
+result
+
+  [article1, articl2]
+
+=end
+
+  def articles
+    Ticket::Article.where(ticket_id: id).order(:created_at, :id)
+  end
+
   private
 
   def check_generate

app/models/ticket/number.rb

@@ -20,7 +20,7 @@ returns
     # generate number
     49_999.times {
       number = adapter.generate
-      ticket = Ticket.find_by( number: number )
+      ticket = Ticket.find_by(number: number)
       return number if !ticket
     }
     raise "Can't generate new ticket number!"

app/models/ticket/number/date.rb

@@ -43,6 +43,14 @@ module Ticket::Number::Date
     # vehikel number. The modulus to 10 of this sum is substracted from
     # 10. See: http://www.pruefziffernberechnung.de/F/Fahrzeugnummer.shtml
     # (german)
+
+    # fix for https://github.com/zammad/zammad/issues/413 - can be removed later
+    if config.class == FalseClass || config.class == TrueClass
+      config = {
+        checksum: config
+      }
+    end
+
     if config[:checksum]
       chksum = 0
       mult   = 1

app/models/ticket/overviews.rb

@@ -16,32 +16,31 @@ returns
 =end
 
   def self.all(data)
+    current_user = data[:current_user]
 
     # get customer overviews
-    if data[:current_user].role?('Customer')
-      role_id = Role.lookup(name: 'Customer').id
-      overviews = if data[:current_user].organization_id && data[:current_user].organization.shared
-                    Overview.where(role_id: role_id, active: true).order(:prio)
+    if current_user.permissions?('ticket.customer')
+      overviews = if current_user.organization_id && current_user.organization.shared
+                    Overview.where(role_id: current_user.role_ids, active: true).order(:prio)
                   else
-                    Overview.where(role_id: role_id, organization_shared: false, active: true).order(:prio)
+                    Overview.where(role_id: current_user.role_ids, organization_shared: false, active: true).order(:prio)
                   end
       overviews_list = []
       overviews.each { |overview|
         user_ids = overview.user_ids
-        next if !user_ids.empty? && !user_ids.include?(data[:current_user].id)
+        next if !user_ids.empty? && !user_ids.include?(current_user.id)
         overviews_list.push overview
       }
       return overviews_list
     end
 
     # get agent overviews
-    return if !data[:current_user].role?('Agent')
-    role_id = Role.lookup(name: 'Agent').id
-    overviews = Overview.where(role_id: role_id, active: true).order(:prio)
+    return [] if !current_user.permissions?('ticket.agent')
+    overviews = Overview.where(role_id: current_user.role_ids, active: true).order(:prio)
     overviews_list = []
     overviews.each { |overview|
       user_ids = overview.user_ids
-      next if !user_ids.empty? && !user_ids.include?(data[:current_user].id)
+      next if !user_ids.empty? && !user_ids.include?(current_user.id)
       overviews_list.push overview
     }
     overviews_list
@@ -86,17 +85,41 @@ returns
     overviews = Ticket::Overviews.all(
       current_user: user,
     )
+    return [] if overviews.blank?
 
     # get only tickets with permissions
     access_condition = Ticket.access_condition(user)
 
+    ticket_attributes = Ticket.new.attributes
     list = []
     overviews.each { |overview|
       query_condition, bind_condition, tables = Ticket.selector2sql(overview.condition, user)
 
-      order_by = "#{overview.order[:by]} #{overview.order[:direction]}"
+      # validate direction
+      raise "Invalid order direction '#{overview.order[:direction]}'" if overview.order[:direction] && overview.order[:direction] !~ /^(ASC|DESC)$/i
+
+      # check if order by exists
+      order_by = overview.order[:by]
+      if !ticket_attributes.key?(order_by)
+        order_by = if ticket_attributes.key?("#{order_by}_id")
+                     "#{order_by}_id"
+                   else
+                     'created_at'
+                   end
+      end
+      order_by = "tickets.#{order_by} #{overview.order[:direction]}"
+
+      # check if group by exists
       if overview.group_by && !overview.group_by.empty?
-        order_by = "#{overview.group_by}_id, #{order_by}"
+        group_by = overview.group_by
+        if !ticket_attributes.key?(group_by)
+          group_by = if ticket_attributes.key?("#{group_by}_id")
+                       "#{group_by}_id"
+                     end
+        end
+        if group_by
+          order_by = "tickets.#{group_by}, #{order_by}"
+        end
       end
 
       ticket_result = Ticket.select('id, updated_at')
@@ -115,9 +138,10 @@ returns
         }
         tickets.push ticket_item
       }
-      count = Ticket.where(access_condition).where(query_condition, *bind_condition).count()
+      count = Ticket.where(access_condition).where(query_condition, *bind_condition).joins(tables).count()
       item = {
         overview: {
+          name: overview.name,
           id: overview.id,
           view: overview.link,
           updated_at: overview.updated_at,

app/models/ticket/search.rb

@@ -162,8 +162,9 @@ returns
                           .order('tickets.created_at DESC')
                           .limit(limit)
     else
-      query_condition, bind_condition = selector2sql(condition)
+      query_condition, bind_condition, tables = selector2sql(condition)
       tickets_all = Ticket.select('DISTINCT(tickets.id), tickets.created_at')
+                          .joins(tables)
                           .where(access_condition)
                           .where(query_condition, *bind_condition)
                           .order('tickets.created_at DESC')

app/models/transaction.rb

@@ -13,7 +13,10 @@ class Transaction
       if options[:interface_handle]
         ApplicationHandleInfo.current = original_interface_handle
       end
-      Observer::Transaction.commit(disable_notification: options[:disable_notification])
+      Observer::Transaction.commit(
+        disable_notification: options[:disable_notification],
+        disable: options[:disable],
+      )
       PushMessages.finish
     end
   end

app/models/transaction/background_job.rb

@@ -24,7 +24,9 @@ class Transaction::BackgroundJob
 
   def perform
     Setting.where(area: 'Transaction::Backend::Async').order(:name).each { |setting|
-      backend = Kernel.const_get(Setting.get(setting.name))
+      backend = Setting.get(setting.name)
+      next if @params[:disable] && @params[:disable].include?(backend)
+      backend = Kernel.const_get(backend)
       Observer::Transaction.execute_singel_backend(backend, @item, @params)
     }
   end

app/models/transaction/trigger.rb

@@ -29,7 +29,7 @@ class Transaction::Trigger
 
     return if @item[:object] != 'Ticket'
 
-    triggers = Trigger.where(active: true)
+    triggers = Trigger.where(active: true).order('LOWER(name)')
     return if triggers.empty?
 
     ticket = Ticket.lookup(id: @item[:object_id])
@@ -39,86 +39,94 @@ class Transaction::Trigger
     end
 
     original_user_id = UserInfo.current_user_id
-    UserInfo.current_user_id = 1
 
-    triggers.each { |trigger|
-      condition = trigger.condition
+    Transaction.execute(reset_user_id: true, disable: ['Transaction::Trigger']) do
+      triggers.each { |trigger|
+        condition = trigger.condition
 
-      # check action
-      if condition['ticket.action']
-        next if condition['ticket.action']['operator'] == 'is' && condition['ticket.action']['value'] != @item[:type]
-        next if condition['ticket.action']['operator'] != 'is' && condition['ticket.action']['value'] == @item[:type]
-        condition.delete('ticket.action')
-      end
-
-      # check "has changed" options
-      has_changed_condition_exists = false
-      has_changed = false
-      condition.each do |key, value|
-        next if !value
-        next if !value['operator']
-        next if !value['operator']['has changed']
-        has_changed_condition_exists = true
-
-        # next if has changed? && !@item[:changes][attribute]
-        (object_name, attribute) = key.split('.', 2)
-
-        # remove condition item, because it has changed
-        if @item[:changes][attribute]
-          has_changed = true
-          condition.delete(key)
-          next
+        # check action
+        if condition['ticket.action']
+          next if condition['ticket.action']['operator'] == 'is' && condition['ticket.action']['value'] != @item[:type]
+          next if condition['ticket.action']['operator'] != 'is' && condition['ticket.action']['value'] == @item[:type]
+          condition.delete('ticket.action')
         end
-        break
-      end
 
-      next if has_changed_condition_exists && !has_changed
+        # check action
+        if condition['article.action']
+          next if !article
+          condition.delete('article.action')
+        end
 
-      # check if selector is matching
-      condition['ticket.id'] = {
-        operator: 'is',
-        value: ticket.id,
-      }
-      if article
-        condition['article.id'] = {
-          operator: 'is',
-          value: article.id,
-        }
-      end
+        # check "has changed" options
+        has_changed_condition_exists = false
+        has_changed = false
+        condition.each do |key, value|
+          next if !value
+          next if !value['operator']
+          next if !value['operator']['has changed']
+          has_changed_condition_exists = true
 
-      ticket_count, tickets = Ticket.selectors(condition, 1)
-      next if ticket_count.zero?
-      next if tickets.first.id != ticket.id
+          # next if has changed? && !@item[:changes][attribute]
+          (object_name, attribute) = key.split('.', 2)
 
-      # check if min one article attribute is used
-      article_selector = false
-      trigger.condition.each do |key, _value|
-        (object_name, attribute) = key.split('.', 2)
-        next if object_name != 'article'
-        next if attribute == 'id'
-        article_selector = true
-      end
-
-      # check in min one attribute has changed
-      if @item[:type] == 'update' && !article_selector
-        match = false
-        if has_changed_condition_exists && has_changed
-          match = true
-        else
-          trigger.condition.each do |key, _value|
-            (object_name, attribute) = key.split('.', 2)
-            next if object_name != 'ticket'
-            next if !@item[:changes][attribute]
-            match = true
-            break
+          # remove condition item, because it has changed
+          if @item[:changes][attribute]
+            has_changed = true
+            condition.delete(key)
+            next
           end
+          break
         end
-        next if !match
-      end
-      Transaction.execute do
+
+        next if has_changed_condition_exists && !has_changed
+
+        # check if selector is matching
+        condition['ticket.id'] = {
+          operator: 'is',
+          value: ticket.id,
+        }
+
+        # check if min one article attribute is used
+        article_selector = false
+        trigger.condition.each do |key, _value|
+          (object_name, attribute) = key.split('.', 2)
+          next if object_name != 'article'
+          next if attribute == 'id'
+          article_selector = true
+        end
+
+        next if article_selector && !article
+        if article_selector
+          condition['article.id'] = {
+            operator: 'is',
+            value: article.id,
+          }
+        end
+
+        ticket_count, tickets = Ticket.selectors(condition, 1)
+        next if ticket_count.zero?
+        next if tickets.first.id != ticket.id
+
+        # check in min one attribute has changed
+        if @item[:type] == 'update' && !article_selector
+          match = false
+          if has_changed_condition_exists && has_changed
+            match = true
+          else
+            trigger.condition.each do |key, _value|
+              (object_name, attribute) = key.split('.', 2)
+              next if object_name != 'ticket'
+              next if !@item[:changes][attribute]
+              match = true
+              break
+            end
+          end
+          next if !match
+        end
+
         ticket.perform_changes(trigger.perform, 'trigger', @item)
-      end
-    }
+      }
+    end
     UserInfo.current_user_id = original_user_id
   end
 

app/models/user.rb

@@ -394,6 +394,32 @@ returns
 
 =begin
 
+returns all accessable permission ids of user
+
+  user = User.find(123)
+  user.permissions_with_child_ids
+
+returns
+
+  [permission1_id, permission2_id, permission3_id]
+
+=end
+
+  def permissions_with_child_ids
+    where = ''
+    where_bind = [true]
+    permissions.each { |permission_name, _value|
+      where += ' OR ' if where != ''
+      where += 'permissions.name = ? OR permissions.name LIKE ?'
+      where_bind.push permission_name
+      where_bind.push "#{permission_name}.%"
+    }
+    return [] if where == ''
+    Object.const_get('Permission').where("permissions.active = ? AND (#{where})", *where_bind).pluck(:id)
+  end
+
+=begin
+
 get all users with permission
 
   users = User.with_permissions('admin.session')

app/views/mailer/password_change/en.html.erb

@@ -1,9 +1,9 @@
-Your #{product_name} password has been changed
+Your #{config.product_name} password has been changed
 
 <p>Hi #{user.firstname},</p>
 <br>
-<p>The password for your #{product_name} account <b>#{user.login}</b> has been changed recently.</p>
+<p>The password for your #{config.product_name} account <b>#{user.login}</b> has been changed recently.</p>
 <br>
 <p>This activity is not known to you? If not, contact your system administrator.</p>
 <br>
-<p>Your #{product_name} Team</p>
+<p>Your #{config.product_name} Team</p>

db/migrate/20120101000001_create_base.rb

@@ -577,7 +577,7 @@ class CreateBase < ActiveRecord::Migration
       t.string  :key,                   limit: 250, null: true
       t.integer :related_o_id,                      null: true
       t.integer :related_stats_store_object_id,     null: true
-      t.string  :data,                 limit: 2500, null: true
+      t.string  :data,                 limit: 5000, null: true
       t.integer :created_by_id,                     null: false
       t.timestamps limit: 3, null: false
     end

db/migrate/20120101000010_create_ticket.rb

@@ -186,18 +186,18 @@ class CreateTicket < ActiveRecord::Migration
     add_index :ticket_counters, [:generator], unique: true
 
     create_table :overviews do |t|
-      t.references :role,                                    null: false
-      t.column :name,                 :string,  limit: 250,  null: false
-      t.column :link,                 :string,  limit: 250,  null: false
-      t.column :prio,                 :integer,              null: false
-      t.column :condition,            :string,  limit: 2500, null: false
-      t.column :order,                :string,  limit: 2500, null: false
-      t.column :group_by,             :string,  limit: 250,  null: true
-      t.column :organization_shared,  :boolean,              null: false, default: false
-      t.column :view,                 :string,  limit: 1000, null: false
-      t.column :active,               :boolean,              null: false, default: true
-      t.column :updated_by_id,        :integer,              null: false
-      t.column :created_by_id,        :integer,              null: false
+      t.references :role,                                      null: false
+      t.column :name,                 :string,  limit: 250,    null: false
+      t.column :link,                 :string,  limit: 250,    null: false
+      t.column :prio,                 :integer,                null: false
+      t.column :condition,            :text, limit: 500.kilobytes + 1, null: false
+      t.column :order,                :string,  limit: 2500,   null: false
+      t.column :group_by,             :string,  limit: 250,    null: true
+      t.column :organization_shared,  :boolean,                null: false, default: false
+      t.column :view,                 :string,  limit: 1000,   null: false
+      t.column :active,               :boolean,                null: false, default: true
+      t.column :updated_by_id,        :integer,                null: false
+      t.column :created_by_id,        :integer,                null: false
       t.timestamps limit: 3, null: false
     end
     add_index :overviews, [:name]
@@ -217,34 +217,34 @@ class CreateTicket < ActiveRecord::Migration
     add_index :overviews_groups, [:group_id]
 
     create_table :triggers do |t|
-      t.column :name,                 :string, limit: 250,  null: false
-      t.column :condition,            :string, limit: 2500, null: false
-      t.column :perform,              :string, limit: 2500, null: false
-      t.column :disable_notification, :boolean,             null: false, default: true
-      t.column :note,                 :string, limit: 250,  null: true
-      t.column :active,               :boolean,             null: false, default: true
-      t.column :updated_by_id,        :integer,             null: false
-      t.column :created_by_id,        :integer,             null: false
+      t.column :name,                 :string, limit: 250,    null: false
+      t.column :condition,            :text, limit: 500.kilobytes + 1, null: false
+      t.column :perform,              :text, limit: 500.kilobytes + 1, null: false
+      t.column :disable_notification, :boolean,               null: false, default: true
+      t.column :note,                 :string, limit: 250,    null: true
+      t.column :active,               :boolean,               null: false, default: true
+      t.column :updated_by_id,        :integer,               null: false
+      t.column :created_by_id,        :integer,               null: false
       t.timestamps limit: 3, null: false
     end
     add_index :triggers, [:name], unique: true
 
     create_table :jobs do |t|
-      t.column :name,                 :string,  limit: 250,  null: false
-      t.column :timeplan,             :string,  limit: 1000, null: false
-      t.column :condition,            :string,  limit: 2500, null: false
-      t.column :perform,              :string,  limit: 2500, null: false
-      t.column :disable_notification, :boolean,              null: false, default: true
-      t.column :last_run_at,          :timestamp, limit: 3,  null: true
-      t.column :next_run_at,          :timestamp, limit: 3,  null: true
-      t.column :running,              :boolean,              null: false, default: false
-      t.column :processed,            :integer,              null: false, default: 0
-      t.column :matching,             :integer,              null: false
-      t.column :pid,                  :string,  limit: 250,  null: true
-      t.column :note,                 :string,  limit: 250,  null: true
-      t.column :active,               :boolean,              null: false, default: false
-      t.column :updated_by_id,        :integer,              null: false
-      t.column :created_by_id,        :integer,              null: false
+      t.column :name,                 :string,  limit: 250,    null: false
+      t.column :timeplan,             :string,  limit: 2500,   null: false
+      t.column :condition,            :text, limit: 500.kilobytes + 1, null: false
+      t.column :perform,              :text, limit: 500.kilobytes + 1, null: false
+      t.column :disable_notification, :boolean,                null: false, default: true
+      t.column :last_run_at,          :timestamp, limit: 3,    null: true
+      t.column :next_run_at,          :timestamp, limit: 3,    null: true
+      t.column :running,              :boolean,                null: false, default: false
+      t.column :processed,            :integer,                null: false, default: 0
+      t.column :matching,             :integer,                null: false
+      t.column :pid,                  :string,  limit: 250,    null: true
+      t.column :note,                 :string,  limit: 250,    null: true
+      t.column :active,               :boolean,                null: false, default: false
+      t.column :updated_by_id,        :integer,                null: false
+      t.column :created_by_id,        :integer,                null: false
       t.timestamps limit: 3, null: false
     end
     add_index :jobs, [:name], unique: true
@@ -285,14 +285,14 @@ class CreateTicket < ActiveRecord::Migration
     add_index :links, [:link_object_source_id, :link_object_source_value, :link_object_target_id, :link_object_target_value, :link_type_id], unique: true, name: 'links_uniq_total'
 
     create_table :postmaster_filters do |t|
-      t.column :name,           :string, limit: 250,  null: false
-      t.column :channel,        :string, limit: 250,  null: false
-      t.column :match,          :string, limit: 5000, null: false
-      t.column :perform,        :string, limit: 5000, null: false
-      t.column :active,         :boolean,             null: false, default: true
-      t.column :note,           :string, limit: 250,  null: true
-      t.column :updated_by_id,  :integer,             null: false
-      t.column :created_by_id,  :integer,             null: false
+      t.column :name,           :string, limit: 250,    null: false
+      t.column :channel,        :string, limit: 250,    null: false
+      t.column :match,          :text, limit: 500.kilobytes + 1, null: false
+      t.column :perform,        :text, limit: 500.kilobytes + 1, null: false
+      t.column :active,         :boolean,               null: false, default: true
+      t.column :note,           :string, limit: 250,    null: true
+      t.column :updated_by_id,  :integer,               null: false
+      t.column :created_by_id,  :integer,               null: false
       t.timestamps limit: 3, null: false
     end
     add_index :postmaster_filters, [:channel]
@@ -359,7 +359,7 @@ class CreateTicket < ActiveRecord::Migration
       t.column :first_response_time,  :integer,                 null: true
       t.column :update_time,          :integer,                 null: true
       t.column :solution_time,        :integer,                 null: true
-      t.column :condition,            :string, limit: 5000,     null: true
+      t.column :condition,            :text, limit: 500.kilobytes + 1, null: true
       t.column :updated_by_id,        :integer,                 null: false
       t.column :created_by_id,        :integer,                 null: false
       t.timestamps limit: 3, null: false
@@ -367,12 +367,12 @@ class CreateTicket < ActiveRecord::Migration
     add_index :slas, [:name], unique: true
 
     create_table :macros do |t|
-      t.string  :name,                   limit: 250,  null: true
-      t.string  :perform,                limit: 5000, null: false
-      t.boolean :active,                              null: false, default: true
-      t.string  :note,                   limit: 250,  null: true
-      t.integer :updated_by_id,                       null: false
-      t.integer :created_by_id,                       null: false
+      t.string  :name,                   limit: 250,    null: true
+      t.text    :perform,                limit: 500.kilobytes + 1, null: false
+      t.boolean :active,                                null: false, default: true
+      t.string  :note,                   limit: 250,    null: true
+      t.integer :updated_by_id,                         null: false
+      t.integer :created_by_id,                         null: false
       t.timestamps limit: 3, null: false
     end
     add_index :macros, [:name], unique: true
@@ -436,11 +436,11 @@ class CreateTicket < ActiveRecord::Migration
     add_index :chat_agents, [:created_by_id], unique: true
 
     create_table :report_profiles do |t|
-      t.column :name,           :string, limit: 150,   null: true
-      t.column :condition,      :string, limit: 6000,  null: true
-      t.column :active,         :boolean,              null: false, default: true
-      t.column :updated_by_id,  :integer,              null: false
-      t.column :created_by_id,  :integer,              null: false
+      t.column :name,           :string, limit: 150,    null: true
+      t.column :condition,      :text, limit: 500.kilobytes + 1, null: true
+      t.column :active,         :boolean,               null: false, default: true
+      t.column :updated_by_id,  :integer,               null: false
+      t.column :created_by_id,  :integer,               null: false
       t.timestamps limit: 3, null: false
     end
     add_index :report_profiles, [:name], unique: true

db/migrate/20161117000001_job_unable_to_create_issue_432.rb

@@ -0,0 +1,44 @@
+class JobUnableToCreateIssue432 < ActiveRecord::Migration
+  def up
+    # return if it's a new setup
+    return if !Setting.find_by(name: 'system_init_done')
+
+    ActiveRecord::Migration.change_table :jobs do |t|
+      t.change :timeplan, :string, limit: 2500
+      t.change :condition, :text, limit: 500.kilobytes + 1
+      t.change :perform, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :triggers do |t|
+      t.change :condition, :text, limit: 500.kilobytes + 1
+      t.change :perform, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :overviews do |t|
+      t.change :condition, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :report_profiles do |t|
+      t.change :condition, :text, limit: 500.kilobytes + 1
+    end
+    ActiveRecord::Migration.change_table :slas do |t|
+      t.change :condition, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :macros do |t|
+      t.change :perform, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :postmaster_filters do |t|
+      t.change :match, :text, limit: 500.kilobytes + 1
+      t.change :perform, :text, limit: 500.kilobytes + 1
+    end
+
+    ActiveRecord::Migration.change_table :stats_stores do |t|
+      t.change :data, :string, limit: 5000
+    end
+
+    Cache.clear
+
+  end
+end

db/migrate/20161117000002_ticket_number_generator_issue_427.rb

@@ -0,0 +1,55 @@
+class TicketNumberGeneratorIssue427 < ActiveRecord::Migration
+  def up
+    # return if it's a new setup
+    return if !Setting.find_by(name: 'system_init_done')
+
+    setting = Setting.find_by(name: 'ticket_number')
+    setting.preferences = {
+      settings_included: %w(ticket_number_increment ticket_number_date),
+      controller: 'SettingsAreaTicketNumber',
+      permission: ['admin.ticket'],
+    }
+    setting.save!
+    setting = Setting.find_by(name: 'ticket_number_increment')
+    setting.preferences = {
+      permission: ['admin.ticket'],
+      hidden: true,
+    }
+    setting.save!
+    setting = Setting.find_by(name: 'ticket_number_date')
+    setting.preferences = {
+      permission: ['admin.ticket'],
+      hidden: true,
+    }
+
+    # just to make sure that value is saved correctly - https://github.com/zammad/zammad/issues/413
+    if setting.state_current['value'] == true || setting.state_current['value'] == false
+      setting.state_current['value'] = { 'checksum' => setting.state_current['value'] }
+    end
+    setting.save!
+
+    setting = Setting.find_by(name: 'ticket_hook_position')
+    setting.preferences = {
+      controller: 'SettingsAreaTicketHookPosition',
+      permission: ['admin.ticket'],
+    }
+    setting.options = {
+      form: [
+        {
+          display: '',
+          null: true,
+          name: 'ticket_hook_position',
+          tag: 'select',
+          translate: true,
+          options: {
+            'left'  => 'left',
+            'right' => 'right',
+            'none'  => 'none',
+          },
+        },
+      ],
+    }
+    setting.save!
+
+  end
+end

db/migrate/20161117000003_store_config_name_update_issue_428.rb

@@ -0,0 +1,30 @@
+class StoreConfigNameUpdateIssue428 < ActiveRecord::Migration
+  def up
+    # return if it's a new setup
+    return if !Setting.find_by(name: 'system_init_done')
+    setting = Setting.find_by(name: 'storage')
+    return if !Setting
+    setting.name = 'storage_provider'
+    setting.options = {
+      form: [
+        {
+          display: '',
+          null: true,
+          name: 'storage_provider',
+          tag: 'select',
+          tranlate: true,
+          options: {
+            'DB' => 'Database',
+            'File' => 'Filesystem',
+          },
+        },
+      ],
+    }
+    setting.preferences = {
+      controller: 'SettingsAreaStorageProvider',
+      online_service_disable: true,
+      permission: ['admin.system'],
+    }
+    setting.save!
+  end
+end

db/seeds.rb

@@ -259,7 +259,7 @@ Setting.create_if_not_exists(
 
 Setting.create_if_not_exists(
   title: 'Storage Mechanism',
-  name: 'storage',
+  name: 'storage_provider',
   area: 'System::Storage',
   description: '"Database" stores all attachments in the database (not recommended for storing large amounts of data). "Filesystem" stores the data on the filesystem. You can switch between the modules even on a system that is already in production without any loss of data.',
   options: {
@@ -267,17 +267,19 @@ Setting.create_if_not_exists(
       {
         display: '',
         null: true,
-        name: 'storage',
+        name: 'storage_provider',
         tag: 'select',
+        tranlate: true,
         options: {
           'DB' => 'Database',
-          'FS' => 'Filesystem',
+          'File' => 'Filesystem',
         },
       },
     ],
   },
   state: 'DB',
   preferences: {
+    controller: 'SettingsAreaStorageProvider',
     online_service_disable: true,
     permission: ['admin.system'],
   },
@@ -1137,10 +1139,10 @@ Setting.create_if_not_exists(
   title: 'Ticket Hook Position',
   name: 'ticket_hook_position',
   area: 'Ticket::Base',
-  description: "@T('The format of the subject.')
-* @T('**Right** means **Some Subject [Ticket#12345]**')
-* @T('**Left** means **[Ticket#12345] Some Subject**')
-* @T('**None** means **Some Subject** (without ticket number). In the last case you should enable *postmaster_follow_up_search_in* to recognize followups based on email headers and/or body.')",
+  description: "The format of the subject.
+* **Right** means **Some Subject [Ticket#12345]**
+* **Left** means **[Ticket#12345] Some Subject**
+* **None** means **Some Subject** (without ticket number). In the last case you should enable *postmaster_follow_up_search_in* to recognize followups based on email headers and/or body.",
   options: {
     form: [
       {
@@ -1148,6 +1150,7 @@ Setting.create_if_not_exists(
         null: true,
         name: 'ticket_hook_position',
         tag: 'select',
+        translate: true,
         options: {
           'left'  => 'left',
           'right' => 'right',
@@ -1158,6 +1161,7 @@ Setting.create_if_not_exists(
   },
   state: 'right',
   preferences: {
+    controller: 'SettingsAreaTicketHookPosition',
     permission: ['admin.ticket'],
   },
   frontend: false
@@ -1166,11 +1170,9 @@ Setting.create_if_not_exists(
   title: 'Ticket Number Format',
   name: 'ticket_number',
   area: 'Ticket::Number',
-  description: "@T('Selects the ticket number generator module.')
-* @T('**Increment** increments the ticket number, the SystemID and the counter are used with SystemID.Counter format (e.g. 1010138, 1010139).')
-* @T('With **Date** the ticket numbers will be generated by the current date, the SystemID and the counter. The format looks like Year.Month.Day.SystemID.counter (e.g. 201206231010138, 201206231010139).')
-
-@T('With param 'Checksum => true' the counter will be appended as checksum to the string. The format looks like SystemID.Counter.CheckSum (e. g. 10101384, 10101392) or Year.Month.Day.SystemID.Counter.CheckSum (e.g. 2012070110101520, 2012070110101535).')",
+  description: "Selects the ticket number generator module.
+* **Increment** increments the ticket number, the SystemID and the counter are used with SystemID.Counter format (e.g. 1010138, 1010139).
+* With **Date** the ticket numbers will be generated by the current date, the SystemID and the counter. The format looks like Year.Month.Day.SystemID.counter (e.g. 201206231010138, 201206231010139).",
   options: {
     form: [
       {
@@ -1178,6 +1180,7 @@ Setting.create_if_not_exists(
         null: true,
         name: 'ticket_number',
         tag: 'select',
+        translate: true,
         options: {
           'Ticket::Number::Increment' => 'Increment (SystemID.Counter)',
           'Ticket::Number::Date'      => 'Date (Year.Month.Day.SystemID.Counter)',
@@ -1187,6 +1190,8 @@ Setting.create_if_not_exists(
   },
   state: 'Ticket::Number::Increment',
   preferences: {
+    settings_included: %w(ticket_number_increment ticket_number_date),
+    controller: 'SettingsAreaTicketNumber',
     permission: ['admin.ticket'],
   },
   frontend: false
@@ -1244,6 +1249,7 @@ Setting.create_if_not_exists(
   },
   preferences: {
     permission: ['admin.ticket'],
+    hidden: true,
   },
   frontend: false
 )
@@ -1267,10 +1273,11 @@ Setting.create_if_not_exists(
     ],
   },
   state: {
-    checksum: false,
+    checksum: false
   },
   preferences: {
     permission: ['admin.ticket'],
+    hidden: true,
   },
   frontend: false
 )

lib/search_index_backend.rb

@@ -339,8 +339,6 @@ get count of tickets and tickets which match on selector
   end
 
   def self.selector2query(selector, _current_user, aggs_interval, limit)
-    filter_must = []
-    filter_must_not = []
     query_must = []
     query_must_not = []
     if selector && !selector.empty?
@@ -355,9 +353,9 @@ get count of tickets and tickets which match on selector
           t[:term][key_tmp] = data['value']
         end
         if data['operator'] == 'is'
-          filter_must.push t
+          query_must.push t
         elsif data['operator'] == 'is not'
-          filter_must_not.push t
+          query_must_not.push t
         elsif data['operator'] == 'contains'
           query_must.push t
         elsif data['operator'] == 'contains not'
@@ -391,43 +389,18 @@ get count of tickets and tickets which match on selector
         from: aggs_interval[:from],
         to: aggs_interval[:to],
       }
-      filter_must.push r
+      query_must.push r
     end
 
-    if !query_must.empty? || !query_must_not.empty?
-      if !data[:query][:filtered]
-        data[:query][:filtered] = {}
-      end
-      if !data[:query][:filtered][:query]
-        data[:query][:filtered][:query] = {}
-      end
-      if !data[:query][:filtered][:query][:bool]
-        data[:query][:filtered][:query][:bool] = {}
-      end
+    if !data[:query][:bool]
+      data[:query][:bool] = {}
     end
+
     if !query_must.empty?
-      data[:query][:filtered][:query][:bool][:must] = query_must
+      data[:query][:bool][:must] = query_must
     end
     if !query_must_not.empty?
-      data[:query][:filtered][:query][:bool][:must_not] = query_must_not
-    end
-
-    if !filter_must.empty? || !filter_must.empty?
-      if !data[:query][:filtered]
-        data[:query][:filtered] = {}
-      end
-      if !data[:query][:filtered][:filter]
-        data[:query][:filtered][:filter] = {}
-      end
-      if !data[:query][:filtered][:filter][:bool]
-        data[:query][:filtered][:filter][:bool] = {}
-      end
-    end
-    if !filter_must.empty?
-      data[:query][:filtered][:filter][:bool][:must] = filter_must
-    end
-    if !filter_must_not.empty?
-      data[:query][:filtered][:filter][:bool][:must_not] = filter_must_not
+      data[:query][:bool][:must_not] = query_must_not
     end
 
     # add sort

public/assets/tests/core.js

@@ -312,6 +312,27 @@ test('i18n', function() {
   var timestamp = App.i18n.translateTimestamp('2012-11-06T21:07:24Z', offset);
   equal(timestamp, '06.11.2012 21:07', 'de-de - timestamp translated correctly')
 
+  timestamp = App.i18n.translateTimestamp('', offset);
+  equal(timestamp, '', 'de-de - timestamp translated correctly')
+
+  timestamp = App.i18n.translateTimestamp(null, offset);
+  equal(timestamp, null, 'de-de - timestamp translated correctly')
+
+  timestamp = App.i18n.translateTimestamp(undefined, offset);
+  equal(timestamp, undefined, 'de-de - timestamp translated correctly')
+
+  var date = App.i18n.translateDate('2012-11-06', 0)
+  equal(date, '06.11.2012', 'de-de - date translated correctly')
+
+  date = App.i18n.translateDate('', 0)
+  equal(date, '', 'de-de - date translated correctly')
+
+  date = App.i18n.translateDate(null, 0)
+  equal(date, null, 'de-de - date translated correctly')
+
+  date = App.i18n.translateDate(undefined, 0)
+  equal(date, undefined, 'de-de - date translated correctly')
+
   // en
   App.i18n.set('en-us')
   translated = App.i18n.translateContent('yes')
@@ -368,6 +389,27 @@ test('i18n', function() {
   timestamp = App.i18n.translateTimestamp('2012-11-06T21:07:24Z', offset)
   equal(timestamp, '11/06/2012 21:07', 'en - timestamp translated correctly')
 
+  timestamp = App.i18n.translateTimestamp('', offset);
+  equal(timestamp, '', 'en - timestamp translated correctly')
+
+  timestamp = App.i18n.translateTimestamp(null, offset);
+  equal(timestamp, null, 'en - timestamp translated correctly')
+
+  timestamp = App.i18n.translateTimestamp(undefined, offset);
+  equal(timestamp, undefined, 'en - timestamp translated correctly')
+
+  date = App.i18n.translateDate('2012-11-06', 0)
+  equal(date, '11/06/2012', 'en - date translated correctly')
+
+  date = App.i18n.translateDate('', 0)
+  equal(date, '', 'en - date translated correctly')
+
+  date = App.i18n.translateDate(null, 0)
+  equal(date, null, 'en - date translated correctly')
+
+  date = App.i18n.translateDate(undefined, 0)
+  equal(date, undefined, 'en - date translated correctly')
+
   // locale alias test
   // de
   App.i18n.set('de')

public/assets/tests/form.js

@@ -715,13 +715,14 @@ test("form postmaster filter", function() {
       },
     },
     set: {
-      'x-zammad-ticket-customer': {
-        value: 'customer'
+      'x-zammad-ticket-customer_id': {
+        value: 'customer',
+        value_completion: ''
       },
       'x-zammad-ticket-group_id': {
         value: '1'
       },
-      'x-zammad-ticket-owner': {
+      'x-zammad-ticket-owner_id': {
         value: 'owner',
         value_completion: ''
       },
@@ -757,13 +758,14 @@ test("form postmaster filter", function() {
       }
     },
     set: {
-      'x-zammad-ticket-customer': {
-        value: 'customer'
+      'x-zammad-ticket-customer_id': {
+        value: 'customer',
+        value_completion: ''
       },
       'x-zammad-ticket-group_id': {
         value: '1'
       },
-      'x-zammad-ticket-owner': {
+      'x-zammad-ticket-owner_id': {
         value: 'owner',
         value_completion: ''
       },
@@ -774,7 +776,7 @@ test("form postmaster filter", function() {
   };
   deepEqual(params, test_params, 'form param check')
   el.find('[name="set::x-zammad-ticket-priority_id::value"]').closest('.js-filterElement').find('.js-remove').click()
-  el.find('[name="set::x-zammad-ticket-customer::value"]').closest('.js-filterElement').find('.js-remove').click()
+  el.find('[name="set::x-zammad-ticket-customer_id::value"]').closest('.js-filterElement').find('.js-remove').click()
   App.Delay.set(function() {
       test("form param check after remove click", function() {
         params = App.ControllerForm.params(el)
@@ -792,7 +794,7 @@ test("form postmaster filter", function() {
             }
           },
           set: {
-            'x-zammad-ticket-owner': {
+            'x-zammad-ticket-owner_id': {
               value: 'owner',
               value_completion: ''
             },

test/browser/abb_one_group_test.rb

@@ -57,9 +57,72 @@ class AgentTicketActionLevel0Test < TestCase
       css: '.modal [name="email"]',
       value: "#{agent}@example.com",
     )
-    exists_not(
-      css: '.modal select[name="group_ids"]',
+    exists(
+      displayed: false,
+      css: '.modal [name="group_ids"]',
     )
+    exists(
+      css: '.modal [name="group_ids"]:checked',
+    )
+    click(
+      css: '.modal button.btn.btn--primary',
+      fast: true,
+    )
+    watch_for(
+      css:   'body div.modal',
+      value: 'Sending',
+    )
+    watch_for_disappear(
+      css:   'body div.modal',
+      value: 'Sending',
+    )
+
+    click(css: '#navigation a[href="#dashboard"]')
+    click(css: '.active.content .tab[data-area="first-steps-widgets"]')
+    watch_for(
+      css:   '.active.content',
+      value: 'Configuration',
+    )
+    click(css: '.active.content .js-inviteAgent')
+    modal_ready()
+    set(
+      css: '.modal [name="firstname"]',
+      value: 'Bob2',
+    )
+    set(
+      css: '.modal [name="lastname"]',
+      value: 'Smith2',
+    )
+    set(
+      css: '.modal [name="email"]',
+      value: "#{agent}2@example.com",
+    )
+
+    # disable agent role
+    uncheck(
+      css: '.modal [name="role_ids"][value=2]',
+    )
+
+    exists(
+      displayed: false,
+      css: '.modal [name="group_ids"]',
+    )
+    exists_not(
+      css: '.modal [name="group_ids"]:checked',
+    )
+
+    # enable agent role
+    check(
+      css: '.modal [name="role_ids"][value=2]',
+    )
+
+    exists(
+      css: '.modal [name="group_ids"]',
+    )
+    exists(
+      css: '.modal [name="group_ids"]:checked',
+    )
+
     click(
       css: '.modal button.btn.btn--primary',
       fast: true,

test/browser/customer_ticket_create_test.rb

@@ -28,6 +28,13 @@ class CustomerTicketCreateTest < TestCase
       css: '.newTicket [data-name="body"]',
       value: 'some body 123äöü',
     )
+    exists_not(
+      css: '.newTicket input[name="customer_id"]',
+    )
+    exists_not(
+      css: '.newTicket input[name="priority_id"]',
+    )
+
     click(css: '.newTicket button.js-submit')
     sleep 5
 
@@ -42,7 +49,7 @@ class CustomerTicketCreateTest < TestCase
 
     # update ticket
     set(
-      css: '.active [data-name="body"]',
+      css: '.content.active [data-name="body"]',
       value: 'some body 1234 äöüß',
       no_click: true,
     )
@@ -51,10 +58,10 @@ class CustomerTicketCreateTest < TestCase
       type: 'stayOnTab',
     )
 
-    click(css: '.active .js-submit')
+    click(css: '.content.active .js-submit')
 
     watch_for(
-      css: '.active div.ticket-article',
+      css: '.content.active div.ticket-article',
       value: 'some body 1234 äöüß',
     )
   end

test/browser/translation_test.rb

@@ -2,6 +2,7 @@
 require 'browser_test_helper'
 
 class TranslationTest < TestCase
+
   def test_preferences
     @browser = browser_instance
     login(
@@ -250,4 +251,60 @@ class TranslationTest < TestCase
     )
 
   end
+
+  def test_admin_sync
+    @browser = browser_instance
+    login(
+      username: 'master@example.com',
+      password: 'test',
+      url: browser_url,
+    )
+    tasks_close_all()
+
+    click(css: 'a[href="#current_user"]')
+    click(css: 'a[href="#profile"]')
+    click(css: 'a[href="#profile/language"]')
+    select(
+      css: '.language_item [name="locale"]',
+      value: 'Deutsch',
+    )
+    click(css: '.content.active button[type="submit"]')
+    watch_for(
+      css: 'body',
+      value: 'Sprache',
+    )
+
+    click(css: 'a[href="#manage"]')
+    click(css: 'a[href="#system/translation"]')
+
+    watch_for(
+      css: '.content.active',
+      value: 'Inline Übersetzung',
+    )
+
+    click(css: '.content.active .js-syncChanges')
+    watch_for(
+      css: '.content.active .modal',
+      value: 'Letzte Übersetzung laden',
+    )
+    watch_for_disappear(
+      css: '.content.active .modal',
+      timeout: 5 * 60,
+    )
+
+    click(css: 'a[href="#current_user"]')
+    click(css: 'a[href="#profile"]')
+    click(css: 'a[href="#profile/language"]')
+    select(
+      css: '.language_item [name="locale"]',
+      value: 'English (United States)',
+    )
+    click(css: '.content.active button[type="submit"]')
+    watch_for(
+      css: 'body',
+      value: 'Language',
+    )
+
+  end
+
 end

test/browser_test_helper.rb

@@ -547,6 +547,12 @@ class TestCase < Test::Unit::TestCase
     css: '.some_class',
   )
 
+  exists(
+    displayed: false, # true|false
+    browser: browser1,
+    css: '.some_class',
+  )
+
 =end
 
   def exists(params)
@@ -558,6 +564,16 @@ class TestCase < Test::Unit::TestCase
       screenshot(browser: instance, comment: 'exists_failed')
       raise "#{params[:css]} dosn't exist, but should"
     end
+
+    if params.key?(:displayed)
+      if params[:displayed] == true && !instance.find_elements(css: params[:css])[0].displayed?
+        raise "#{params[:css]} is not displayed, but should"
+      end
+      if params[:displayed] == false && instance.find_elements(css: params[:css])[0].displayed?
+        raise "#{params[:css]} is displayed, but should not"
+      end
+    end
+
     true
   end
 

test/controllers/search_controller_test.rb

@@ -2,7 +2,7 @@
 require 'test_helper'
 
 class SearchControllerTest < ActionDispatch::IntegrationTest
-  setup do
+  def base_data
 
     # clear cache
     Cache.clear
@@ -192,6 +192,7 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   end
 
   test 'settings index with nobody' do
+    base_data
 
     params = {
       query: 'test 1234',
@@ -222,6 +223,7 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   end
 
   test 'settings index with admin' do
+    base_data
 
     credentials = ActionController::HttpAuthentication::Basic.encode_credentials('search-admin@example.com', 'adminpw')
 
@@ -295,6 +297,7 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   end
 
   test 'settings index with agent' do
+    base_data
 
     credentials = ActionController::HttpAuthentication::Basic.encode_credentials('search-agent@example.com', 'agentpw')
 
@@ -368,6 +371,7 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   end
 
   test 'settings index with customer 1' do
+    base_data
 
     credentials = ActionController::HttpAuthentication::Basic.encode_credentials('search-customer1@example.com', 'customer1pw')
 
@@ -413,6 +417,7 @@ class SearchControllerTest < ActionDispatch::IntegrationTest
   end
 
   test 'settings index with customer 2' do
+    base_data
 
     credentials = ActionController::HttpAuthentication::Basic.encode_credentials('search-customer2@example.com', 'customer2pw')
 

test/controllers/tickets_controller_test.rb

@@ -318,7 +318,7 @@ class TicketsControllerTest < ActionDispatch::IntegrationTest
     result = JSON.parse(@response.body)
     assert_equal(Hash, result.class)
     assert_equal(ticket.id, result['ticket_id'])
-    assert_equal('Tickets Agent via Zammad <zammad@localhost>', result['from'])
+    assert_equal('"Tickets Agent via Zammad" <zammad@localhost>', result['from'])
     assert_equal('some subject', result['subject'])
     assert_equal('some body', result['body'])
     assert_equal('text/plain', result['content_type'])
@@ -335,7 +335,7 @@ class TicketsControllerTest < ActionDispatch::IntegrationTest
     result = JSON.parse(@response.body)
     assert_equal(Hash, result.class)
     assert_equal(ticket.id, result['ticket_id'])
-    assert_equal('Tickets Agent via Zammad <zammad@localhost>', result['from'])
+    assert_equal('"Tickets Agent via Zammad" <zammad@localhost>', result['from'])
     assert_equal('new subject', result['subject'])
     assert_equal('some body', result['body'])
     assert_equal('text/plain', result['content_type'])
@@ -444,7 +444,7 @@ class TicketsControllerTest < ActionDispatch::IntegrationTest
     result = JSON.parse(@response.body)
     assert_equal(Hash, result.class)
     assert_equal(ticket.id, result['ticket_id'])
-    assert_equal('Tickets Admin via Zammad <zammad@localhost>', result['from'])
+    assert_equal('"Tickets Admin via Zammad" <zammad@localhost>', result['from'])
     assert_equal('some subject', result['subject'])
     assert_equal('some body', result['body'])
     assert_equal('text/plain', result['content_type'])

test/test_helper.rb

@@ -38,6 +38,9 @@ class ActiveSupport::TestCase
 
     # remove background jobs
     Delayed::Job.destroy_all
+    ActivityStream.destroy_all
+    PostmasterFilter.destroy_all
+    Ticket.destroy_all
 
     # set current user
     UserInfo.current_user_id = nil

test/unit/activity_stream_test.rb

@@ -4,9 +4,8 @@ require 'test_helper'
 class ActivityStreamTest < ActiveSupport::TestCase
   admin_user = nil
   current_user = nil
-  activity_record_delay = nil
   test 'aaa - setup' do
-    role  = Role.lookup(name: 'Admin')
+    roles = Role.where(name: %w(Admin Agent))
     group = Group.lookup(name: 'Users')
     admin_user = User.create_or_update(
       login: 'admin',
@@ -15,392 +14,200 @@ class ActivityStreamTest < ActiveSupport::TestCase
       email: 'bob@example.com',
       password: 'some_pass',
       active: true,
-      role_ids: [role.id],
+      roles: roles,
       group_ids: [group.id],
       updated_by_id: 1,
       created_by_id: 1
     )
     current_user = User.lookup(email: 'nicole.braun@zammad.org')
-
-    activity_record_delay = if ENV['ZAMMAD_ACTIVITY_RECORD_DELAY']
-                              ENV['ZAMMAD_ACTIVITY_RECORD_DELAY'].to_i.seconds
-                            else
-                              90.seconds
-                            end
   end
 
   test 'ticket+user' do
-    tests = [
+    ticket = Ticket.create(
+      group_id: Group.lookup(name: 'Users').id,
+      customer_id: current_user.id,
+      owner_id: User.lookup(login: '-').id,
+      title: 'Unit Test 1 (äöüß)!',
+      state_id: Ticket::State.lookup(name: 'new').id,
+      priority_id: Ticket::Priority.lookup(name: '2 normal').id,
+      updated_by_id: current_user.id,
+      created_by_id: current_user.id,
+    )
+    travel 2.seconds
 
-      # test 1
-      {
-        create: {
-          ticket: {
-            group_id: Group.lookup(name: 'Users').id,
-            customer_id: current_user.id,
-            owner_id: User.lookup(login: '-').id,
-            title: 'Unit Test 1 (äöüß)!',
-            state_id: Ticket::State.lookup(name: 'new').id,
-            priority_id: Ticket::Priority.lookup(name: '2 normal').id,
-            updated_by_id: current_user.id,
-            created_by_id: current_user.id,
-          },
-          article: {
-            updated_by_id: current_user.id,
-            created_by_id: current_user.id,
-            type_id: Ticket::Article::Type.lookup(name: 'phone').id,
-            sender_id: Ticket::Article::Sender.lookup(name: 'Customer').id,
-            from: 'Unit Test <unittest@example.com>',
-            body: 'Unit Test 123',
-            internal: false,
-          },
-        },
-        update: {
-          ticket: {
-            title: 'Unit Test 1 (äöüß) - update!',
-            state_id: Ticket::State.lookup(name: 'open').id,
-            priority_id: Ticket::Priority.lookup(name: '1 low').id,
-          },
-        },
-        update2: {
-          ticket: {
-            title: 'Unit Test 2 (äöüß) - update!',
-            priority_id: Ticket::Priority.lookup(name: '2 normal').id,
-          },
-        },
-        check: [
-          {
-            result: true,
-            object: 'Ticket',
-            type: 'update',
-          },
-          {
-            result: true,
-            object: 'Ticket::Article',
-            type: 'create',
-          },
-          {
-            result: true,
-            object: 'Ticket',
-            type: 'create',
-          },
-          {
-            result: false,
-            object: 'User',
-            type: 'update',
-            o_id: current_user.id,
-          },
-        ]
-      },
-    ]
-    tickets = []
-    tests.each { |test|
+    article = Ticket::Article.create(
+      ticket_id: ticket.id,
+      updated_by_id: current_user.id,
+      created_by_id: current_user.id,
+      type_id: Ticket::Article::Type.lookup(name: 'phone').id,
+      sender_id: Ticket::Article::Sender.lookup(name: 'Customer').id,
+      from: 'Unit Test <unittest@example.com>',
+      body: 'Unit Test 123',
+      internal: false,
+    )
 
-      ticket = Ticket.create(test[:create][:ticket])
-      test[:check][0][:o_id]          = ticket.id
-      test[:check][2][:o_id]          = ticket.id
-      test[:check][2][:created_at]    = ticket.created_at
-      test[:check][2][:created_by_id] = current_user.id
-      sleep 2
+    travel 100.seconds
+    ticket.update_attributes(
+      title: 'Unit Test 1 (äöüß) - update!',
+      state_id: Ticket::State.lookup(name: 'open').id,
+      priority_id: Ticket::Priority.lookup(name: '1 low').id,
+    )
+    updated_at = ticket.updated_at
 
-      test[:create][:article][:ticket_id] = ticket.id
-      article = Ticket::Article.create(test[:create][:article])
-      test[:check][1][:o_id]          = article.id
-      test[:check][1][:created_at]    = article.created_at
-      test[:check][1][:created_by_id] = current_user.id
+    travel 1.second
+    ticket.update_attributes(
+      title: 'Unit Test 2 (äöüß) - update!',
+      priority_id: Ticket::Priority.lookup(name: '2 normal').id,
+    )
 
-      assert_equal(ticket.class.to_s, 'Ticket')
-      assert_equal(article.class.to_s, 'Ticket::Article')
+    # check activity_stream
+    stream = admin_user.activity_stream(4)
+    assert_equal(stream[0]['group_id'], ticket.group_id)
+    assert_equal(stream[0]['o_id'], ticket.id)
+    assert_equal(stream[0]['created_by_id'], current_user.id)
+    assert_equal(stream[0]['created_at'].to_s, updated_at.to_s)
+    assert_equal(stream[0]['object'], 'Ticket')
+    assert_equal(stream[0]['type'], 'update')
+    assert_equal(stream[1]['group_id'], ticket.group_id)
+    assert_equal(stream[1]['o_id'], article.id)
+    assert_equal(stream[1]['created_by_id'], current_user.id)
+    assert_equal(stream[1]['created_at'].to_s, article.created_at.to_s)
+    assert_equal(stream[1]['object'], 'Ticket::Article')
+    assert_equal(stream[1]['type'], 'create')
+    assert_equal(stream[2]['group_id'], ticket.group_id)
+    assert_equal(stream[2]['o_id'], ticket.id)
+    assert_equal(stream[2]['created_by_id'], current_user.id)
+    assert_equal(stream[2]['created_at'].to_s, ticket.created_at.to_s)
+    assert_equal(stream[2]['object'], 'Ticket')
+    assert_equal(stream[2]['type'], 'create')
+    assert_not(stream[3])
 
-      # update ticket
-      if test[:update][:ticket]
-        ticket.update_attributes(test[:update][:ticket])
+    stream = current_user.activity_stream(4)
+    assert(stream.empty?)
 
-        # check updated user
-        test[:check][3][:o_id]          = current_user.id
-        test[:check][3][:created_at]    = ticket.created_at
-        test[:check][3][:created_by_id] = current_user.id
-      end
-      if test[:update2][:ticket]
-        ticket = Ticket.find(ticket.id)
-        ticket.update_attributes(test[:update2][:ticket])
-      end
-      if test[:update][:article]
-        article.update_attributes(test[:update][:article])
-      end
-
-      sleep activity_record_delay + 1
-      if test[:update][:ticket]
-        ticket.update_attributes(test[:update][:ticket])
-      end
-      if test[:update2][:ticket]
-        ticket.update_attributes(test[:update2][:ticket])
-      end
-
-      # remember ticket
-      tickets.push ticket
-
-      # check activity_stream
-      activity_stream_check(admin_user.activity_stream(3), test[:check])
-    }
-
-    # delete tickets
-    tickets.each { |ticket|
-      ticket_id = ticket.id
-      ticket.destroy
-      found = Ticket.where(id: ticket_id).first
-      assert_not(found, 'Ticket destroyed')
-    }
+    # cleanup
+    ticket.destroy
+    travel_back
   end
 
   test 'organization' do
-    tests = [
 
-      # test 1
-      {
-        create: {
-          organization: {
-            name: 'some name',
-            updated_by_id: current_user.id,
-            created_by_id: current_user.id,
-          },
-        },
-        update1: {
-          organization: {
-            name: 'some name (äöüß)',
-          },
-        },
-        update2: {
-          organization: {
-            name: 'some name 2 (äöüß)',
-          },
-        },
-        check: [
-          {
-            result: true,
-            object: 'Organization',
-            type: 'update',
-          },
-          {
-            result: true,
-            object: 'Organization',
-            type: 'create',
-          },
-        ]
-      },
-    ]
-    organizations = []
-    tests.each { |test|
+    organization = Organization.create(
+      name: 'some name',
+      updated_by_id: current_user.id,
+      created_by_id: current_user.id,
+    )
+    travel 100.seconds
+    assert_equal(organization.class, Organization)
 
-      organization = Organization.create(test[:create][:organization])
-      test[:check][0][:o_id]          = organization.id
-      test[:check][0][:created_at]    = organization.created_at
-      test[:check][0][:created_by_id] = current_user.id
-      sleep 2
+    organization.update_attributes(name: 'some name (äöüß)')
+    updated_at = organization.updated_at
 
-      assert_equal(organization.class.to_s, 'Organization')
+    travel 10.seconds
+    organization.update_attributes(name: 'some name 2 (äöüß)')
 
-      if test[:update1][:organization]
-        organization.update_attributes(test[:update1][:organization])
-        test[:check][1][:o_id]          = organization.id
-        test[:check][1][:updated_at]    = organization.updated_at
-        test[:check][1][:created_by_id] = current_user.id
-        sleep activity_record_delay - 1
-      end
+    # check activity_stream
+    stream = admin_user.activity_stream(3)
+    assert_not(stream[0]['group_id'])
+    assert_equal(stream[0]['o_id'], organization.id)
+    assert_equal(stream[0]['created_by_id'], current_user.id)
+    assert_equal(stream[0]['created_at'].to_s, updated_at.to_s)
+    assert_equal(stream[0]['object'], 'Organization')
+    assert_equal(stream[0]['type'], 'update')
+    assert_not(stream[1]['group_id'])
+    assert_equal(stream[1]['o_id'], organization.id)
+    assert_equal(stream[1]['created_by_id'], current_user.id)
+    assert_equal(stream[1]['created_at'].to_s, organization.created_at.to_s)
+    assert_equal(stream[1]['object'], 'Organization')
+    assert_equal(stream[1]['type'], 'create')
+    assert_not(stream[2])
 
-      if test[:update2][:organization]
-        organization.update_attributes(test[:update2][:organization])
-      end
+    stream = current_user.activity_stream(4)
+    assert(stream.empty?)
 
-      # remember organization
-      organizations.push organization
-
-      # check activity_stream
-      activity_stream_check(admin_user.activity_stream(2), test[:check])
-    }
-
-    # delete tickets
-    organizations.each { |organization|
-      organization_id = organization.id
-      organization.destroy
-      found = Organization.where(id: organization_id).first
-      assert( !found, 'Organization destroyed')
-    }
+    # cleanup
+    organization.destroy
+    travel_back
   end
 
   test 'user with update check false' do
-    tests = [
+    user = User.create(
+      login: 'someemail@example.com',
+      email: 'someemail@example.com',
+      firstname: 'Bob Smith II',
+      updated_by_id: current_user.id,
+      created_by_id: current_user.id,
+    )
+    assert_equal(user.class, User)
+    user.update_attributes(
+      firstname: 'Bob U',
+      lastname: 'Smith U',
+    )
 
-      # test 1
-      {
-        create: {
-          user: {
-            login: 'someemail@example.com',
-            email: 'someemail@example.com',
-            firstname: 'Bob Smith II',
-            updated_by_id: current_user.id,
-            created_by_id: current_user.id,
-          },
-        },
-        update1: {
-          user: {
-            firstname: 'Bob U',
-            lastname: 'Smith U',
-          },
-        },
-        check: [
-          {
-            result: true,
-            object: 'User',
-            type: 'create',
-          },
-          {
-            result: false,
-            object: 'User',
-            type: 'update',
-          },
-        ]
-      },
-    ]
-    users = []
-    tests.each { |test|
+    # check activity_stream
+    stream = admin_user.activity_stream(3)
+    assert_not(stream[0]['group_id'])
+    assert_equal(stream[0]['o_id'], user.id)
+    assert_equal(stream[0]['created_by_id'], current_user.id)
+    assert_equal(stream[0]['created_at'].to_s, user.created_at.to_s)
+    assert_equal(stream[0]['object'], 'User')
+    assert_equal(stream[0]['type'], 'create')
+    assert_not(stream[1])
 
-      user = User.create(test[:create][:user])
-      test[:check][0][:o_id]          = user.id
-      test[:check][0][:created_at]    = user.created_at
-      test[:check][0][:created_by_id] = current_user.id
+    stream = current_user.activity_stream(4)
+    assert(stream.empty?)
 
-      assert_equal(user.class.to_s, 'User')
-
-      if test[:update1][:user]
-        user.update_attributes(test[:update1][:user])
-        test[:check][1][:o_id]          = user.id
-        test[:check][1][:updated_at]    = user.updated_at
-        test[:check][1][:created_by_id] = current_user.id
-      end
-
-      # remember organization
-      users.push user
-
-      # check activity_stream
-      activity_stream_check(admin_user.activity_stream(3), test[:check])
-    }
-
-    # delete tickets
-    users.each { |user|
-      user_id = user.id
-      user.destroy
-      found = User.where( id: user_id ).first
-      assert_not(found, 'User destroyed')
-    }
+    # cleanup
+    user.destroy
+    travel_back
   end
 
   test 'user with update check true' do
-    tests = [
 
-      # test 1
-      {
-        create: {
-          user: {
-            login: 'someemail@example.com',
-            email: 'someemail@example.com',
-            firstname: 'Bob Smith II',
-            updated_by_id: current_user.id,
-            created_by_id: current_user.id,
-          },
-        },
-        update1: {
-          user: {
-            firstname: 'Bob U',
-            lastname: 'Smith U',
-          },
-        },
-        update2: {
-          user: {
-            firstname: 'Bob',
-            lastname: 'Smith',
-          },
-        },
-        check: [
-          {
-            result: true,
-            object: 'User',
-            type: 'update',
-          },
-          {
-            result: true,
-            object: 'User',
-            type: 'create',
-          },
-        ]
-      },
-    ]
-    users = []
-    tests.each { |test|
+    user = User.create(
+      login: 'someemail@example.com',
+      email: 'someemail@example.com',
+      firstname: 'Bob Smith II',
+      updated_by_id: current_user.id,
+      created_by_id: current_user.id,
+    )
+    travel 100.seconds
+    assert_equal(user.class, User)
 
-      user = User.create(test[:create][:user])
-      test[:check][0][:o_id]          = user.id
-      test[:check][0][:created_at]    = user.created_at
-      test[:check][0][:created_by_id] = current_user.id
+    user.update_attributes(
+      firstname: 'Bob U',
+      lastname: 'Smith U',
+    )
+    updated_at = user.updated_at
 
-      assert_equal(user.class.to_s, 'User')
+    travel 10.seconds
+    user.update_attributes(
+      firstname: 'Bob',
+      lastname: 'Smith',
+    )
 
-      if test[:update1][:user]
-        user.update_attributes(test[:update1][:user])
-        test[:check][1][:o_id]          = user.id
-        test[:check][1][:updated_at]    = user.updated_at
-        test[:check][1][:created_by_id] = current_user.id
-      end
+    # check activity_stream
+    stream = admin_user.activity_stream(3)
+    assert_not(stream[0]['group_id'])
+    assert_equal(stream[0]['o_id'], user.id)
+    assert_equal(stream[0]['created_by_id'], current_user.id)
+    assert_equal(stream[0]['created_at'].to_s, updated_at.to_s)
+    assert_equal(stream[0]['object'], 'User')
+    assert_equal(stream[0]['type'], 'update')
+    assert_not(stream[1]['group_id'])
+    assert_equal(stream[1]['o_id'], user.id)
+    assert_equal(stream[1]['created_by_id'], current_user.id)
+    assert_equal(stream[1]['created_at'].to_s, user.created_at.to_s)
+    assert_equal(stream[1]['object'], 'User')
+    assert_equal(stream[1]['type'], 'create')
+    assert_not(stream[2])
 
-      # to verify update which need to be logged
-      sleep activity_record_delay + 1
+    stream = current_user.activity_stream(4)
+    assert(stream.empty?)
 
-      if test[:update2][:user]
-        user.update_attributes(test[:update2][:user])
-      end
-
-      # remember organization
-      users.push user
-
-      # check activity_stream
-      activity_stream_check(admin_user.activity_stream(2), test[:check])
-    }
-
-    # delete tickets
-    users.each { |user|
-      user_id = user.id
-      user.destroy
-      found = User.where(id: user_id).first
-      assert(!found, 'User destroyed')
-    }
-  end
-
-  def activity_stream_check(activity_stream_list, checks)
-    #activity_stream_list = activity_stream_list.reverse
-    #puts 'AS ' + activity_stream_list.inspect
-    check_count = 0
-    checks.each { |check_item|
-      check_count += 1
-
-      #puts '+++++++++++'
-      #puts check_item.inspect
-      check_list = 0
-      activity_stream_list.each { |item|
-        check_list += 1
-        next if check_list != check_count
-        #next if match
-        #puts '--------'
-        #puts item.inspect
-        #puts check_item.inspect
-        if check_item[:result]
-          assert_equal(check_item[:object], item['object'])
-          assert_equal(check_item[:type], item['type'])
-          assert_equal(check_item[:o_id], item['o_id'])
-        elsif check_item[:object] == item['object'] && check_item[:type] == item['type'] && check_item[:o_id] == item['o_id']
-          assert(false, "entry should not exist #{item['object']}/#{item['type']}/#{item['o_id']}")
-        end
-      }
-    }
+    # cleanup
+    user.destroy
+    travel_back
   end
 
 end

test/unit/assets_test.rb

@@ -6,7 +6,7 @@ class AssetsTest < ActiveSupport::TestCase
 
     roles  = Role.where(name: %w(Agent Admin))
     groups = Group.all
-    org    = Organization.create_or_update(
+    org1   = Organization.create_or_update(
       name: 'some user org',
       updated_by_id: 1,
       created_by_id: 1,
@@ -21,7 +21,7 @@ class AssetsTest < ActiveSupport::TestCase
       active: true,
       updated_by_id: 1,
       created_by_id: 1,
-      organization_id: org.id,
+      organization_id: org1.id,
       roles: roles,
       groups: groups,
     )
@@ -54,10 +54,10 @@ class AssetsTest < ActiveSupport::TestCase
     user3 = User.find(user3.id)
     assets = user3.assets({})
 
-    org = Organization.find(org.id)
-    attributes = org.attributes_with_associations
+    org1 = Organization.find(org1.id)
+    attributes = org1.attributes_with_associations
     attributes.delete('user_ids')
-    assert( diff(attributes, assets[:Organization][org.id]), 'check assets')
+    assert( diff(attributes, assets[:Organization][org1.id]), 'check assets')
 
     user1 = User.find(user1.id)
     attributes = user1.attributes_with_associations
@@ -84,8 +84,8 @@ class AssetsTest < ActiveSupport::TestCase
     assert( diff(attributes, assets[:User][user3.id]), 'check assets' )
 
     # touch org, check if user1 has changed
-    sleep 2
-    org2 = Organization.find(org.id)
+    travel 2.seconds
+    org2 = Organization.find(org1.id)
     org2.note = "some note...#{rand(9_999_999_999_999)}"
     org2.save
 
@@ -105,7 +105,7 @@ class AssetsTest < ActiveSupport::TestCase
     assets = user3.assets({})
     attributes = org2.attributes_with_associations
     attributes.delete('user_ids')
-    assert( diff(attributes, assets[:Organization][org.id]), 'check assets')
+    assert( diff(attributes, assets[:Organization][org1.id]), 'check assets')
 
     user1 = User.find(user1.id)
     attributes = user1.attributes_with_associations
@@ -130,6 +130,13 @@ class AssetsTest < ActiveSupport::TestCase
     attributes.delete('token_ids')
     attributes.delete('authorization_ids')
     assert( diff(attributes, assets[:User][user3.id]), 'check assets' )
+    travel_back
+
+    user1.destroy
+    user2.destroy
+    user3.destroy
+    org1.destroy
+    org2.destroy
 
   end
 
@@ -148,8 +155,8 @@ class AssetsTest < ActiveSupport::TestCase
       roles: roles,
     )
 
-    roles  = Role.where( name: %w(Customer) )
-    org    = Organization.create_or_update(
+    roles = Role.where( name: %w(Customer) )
+    org   = Organization.create_or_update(
       name: 'some customer org',
       updated_by_id: admin1.id,
       created_by_id: 1,
@@ -232,7 +239,7 @@ class AssetsTest < ActiveSupport::TestCase
     assert_nil( assets[:User][user3.id], 'check assets' )
 
     # touch user 2, check if org has changed
-    sleep 2
+    travel 2.seconds
     user_new_2 = User.find(user2.id)
     user_new_2.lastname = 'assets2'
     user_new_2.save
@@ -261,6 +268,13 @@ class AssetsTest < ActiveSupport::TestCase
     attributes.delete('token_ids')
     attributes.delete('authorization_ids')
     assert( diff(attributes, assets[:User][user_new_2.id]), 'check assets' )
+    travel_back
+
+    user1.destroy
+    user2.destroy
+    user3.destroy
+    org.destroy
+    org_new.destroy
 
   end
 

test/unit/calendar_test.rb

@@ -19,7 +19,7 @@ class CalendarTest < ActiveSupport::TestCase
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
     calendar2 = Calendar.create_or_update(
       name: 'US 2',
       timezone: 'America/Los_Angeles',
@@ -88,6 +88,7 @@ class CalendarTest < ActiveSupport::TestCase
 
     assert_equal(true, calendar2.default)
     assert_equal(false, calendar3.default)
+    travel_back
   end
 
 end

test/unit/chat_test.rb

@@ -208,19 +208,19 @@ class ChatTest < ActiveSupport::TestCase
       content: 'message 1',
       created_by_id: agent1.id,
     )
-    sleep 1
+    travel 1.second
     Chat::Message.create(
       chat_session_id: chat_session6.id,
       content: 'message 2',
       created_by_id: agent1.id,
     )
-    sleep 1
+    travel 1.second
     Chat::Message.create(
       chat_session_id: chat_session6.id,
       content: 'message 3',
       created_by_id: agent1.id,
     )
-    sleep 1
+    travel 1.second
     Chat::Message.create(
       chat_session_id: chat_session6.id,
       content: 'message 4',
@@ -351,7 +351,7 @@ class ChatTest < ActiveSupport::TestCase
     assert_equal(1, agent_state[:seads_available])
     assert_equal(4, agent_state[:seads_total])
     assert_equal(false, agent_state[:active])
-
+    travel_back
   end
 
 end

test/unit/email_build_test.rb

@@ -211,4 +211,20 @@ text
     assert_equal(html_should, html_with_fixes)
   end
 
+  test 'from checks' do
+
+    quoted_in_one_line = Channel::EmailBuild.recipient_line('Somebody @ "Company"', 'some.body@example.com')
+    assert_equal('"Somebody @ \"Company\"" <some.body@example.com>', quoted_in_one_line)
+
+    quoted_in_one_line = Channel::EmailBuild.recipient_line('Somebody', 'some.body@example.com')
+    assert_equal('Somebody <some.body@example.com>', quoted_in_one_line)
+
+    quoted_in_one_line = Channel::EmailBuild.recipient_line('Somebody | Some Org', 'some.body@example.com')
+    assert_equal('"Somebody | Some Org" <some.body@example.com>', quoted_in_one_line)
+
+    quoted_in_one_line = Channel::EmailBuild.recipient_line('Test Master Agent via Support', 'some.body@example.com')
+    assert_equal('"Test Master Agent via Support" <some.body@example.com>', quoted_in_one_line)
+
+  end
+
 end

test/unit/email_process_bounce_test.rb

@@ -27,11 +27,13 @@ class EmailProcessBounceTest < ActiveSupport::TestCase
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
     email_raw_string = IO.binread('test/fixtures/mail33-undelivered-mail-returned-to-sender.box')
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('new', ticket_p.state.name)
+    travel_back
+    ticket.destroy
   end
 
 end

test/unit/email_process_follow_up_test.rb

@@ -84,47 +84,48 @@ no reference "
     setting_orig = Setting.get('postmaster_follow_up_search_in')
     Setting.set('postmaster_follow_up_search_in', %w(body attachment references))
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_subject)
     assert_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_body)
     assert_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_attachment)
     assert_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_references1)
     assert_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_references2)
     assert_equal(ticket.id, ticket_p.id)
 
     Setting.set('postmaster_follow_up_search_in', setting_orig)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_subject)
     assert_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_body)
     assert_not_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_attachment)
     assert_not_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_references1)
     assert_not_equal(ticket.id, ticket_p.id)
 
-    sleep 1
+    travel 1.second
     ticket_p, article_p, user_p = Channel::EmailParser.new.process({}, email_raw_string_references2)
     assert_not_equal(ticket.id, ticket_p.id)
+    travel_back
   end
 
   test 'process with follow up check with two external reference headers' do
@@ -141,7 +142,7 @@ test 123
 "
     ticket_p1, article_p1, user_p1 = Channel::EmailParser.new.process({}, data1)
 
-    sleep 1
+    travel 1.second
 
     data1 = "From: me@example.com
 To: z@example.com
@@ -166,7 +167,7 @@ test 123
     assert_equal(ticket_p2.id, ticket_p3.id)
 
     Setting.set('postmaster_follow_up_search_in', setting_orig)
-
+    travel_back
   end
 
   test 'process with follow up check - with auto responses and no T# in subject_build' do
@@ -193,7 +194,7 @@ test 123
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     # auto response without T# in subject, find follow up by references header
     email_raw_string = "From: bob@example.com
@@ -211,6 +212,7 @@ Some Text"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('open', ticket.state.name)
+    travel_back
   end
 
   test 'process with follow up check - email with more forgein T#\'s in subject' do
@@ -237,7 +239,7 @@ Some Text"
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     system_id           = Setting.get('system_id')
     ticket_hook         = Setting.get('ticket_hook')
@@ -255,6 +257,7 @@ Some Text"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('open', ticket.state.name)
+    travel_back
   end
 
   test 'process with follow up check - ticket initiated by customer without T# in subject and other people in Cc reply to all' do

test/unit/email_process_out_of_office_test.rb

@@ -27,7 +27,7 @@ class EmailProcessOutOfOfficeTest < ActiveSupport::TestCase
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     # exchange out of office example #1
     email_raw_string = "From: me@example.com
@@ -86,7 +86,7 @@ Some Text"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('closed', ticket.state.name)
-
+    travel_back
   end
 
   test 'process with out of office check - zimbra' do
@@ -113,7 +113,7 @@ Some Text"
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     # exchange out of office example #1
     email_raw_string = "From: me@example.com
@@ -164,7 +164,7 @@ Some Text"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('closed', ticket.state.name)
-
+    travel_back
   end
 
   test 'process with out of office check - cloud' do
@@ -191,7 +191,7 @@ Some Text"
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     # exchange out of office example #1
     email_raw_string = "From: me@example.com
@@ -237,7 +237,7 @@ Some Text"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('closed', ticket.state.name)
-
+    travel_back
   end
 
   test 'process with out of office check - gmail' do
@@ -264,7 +264,7 @@ Some Text"
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 1
+    travel 1.second
 
     # gmail out of office example #1
     email_raw_string = "From: me@example.com
@@ -294,6 +294,7 @@ Some Text 2"
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.id, ticket_p.id)
     assert_equal('open', ticket_p.state.name)
+    travel_back
   end
 
 end

test/unit/object_manager_test.rb

@@ -407,6 +407,26 @@ class ObjectManagerTest < ActiveSupport::TestCase
     }
     assert_equal(false, ObjectManager::Attribute.pending_migration?)
 
+    assert_raises(RuntimeError) {
+      attribute18 = ObjectManager::Attribute.add(
+        object: 'Ticket',
+        name: 'delete',
+        display: 'Test 18',
+        data_type: 'input',
+        data_option: {
+          maxlength: 200,
+          type: 'text',
+          null: false,
+        },
+        active: true,
+        screens: {},
+        position: 20,
+        created_by_id: 1,
+        updated_by_id: 1,
+      )
+    }
+    assert_equal(false, ObjectManager::Attribute.pending_migration?)
+
   end
 
   test 'b object manager attribute' do

test/unit/organization_ref_object_touch_test.rb

@@ -81,7 +81,7 @@ class OrganizationRefObjectTouchTest < ActiveSupport::TestCase
     assert_equal(ticket.customer.id, customer1.id)
     assert_equal(ticket.organization.id, organization1.id)
 
-    sleep 4
+    travel 4.seconds
 
     organization1.name = 'Ref Object Update Org 1/1'
     organization1.save
@@ -101,7 +101,7 @@ class OrganizationRefObjectTouchTest < ActiveSupport::TestCase
       assert(false, 'customer1.updated_at has not been updated')
     end
 
-    sleep 4
+    travel 4.seconds
 
     customer2.organization_id = organization1.id
     customer2.save
@@ -123,5 +123,6 @@ class OrganizationRefObjectTouchTest < ActiveSupport::TestCase
 
     delete = ticket.destroy
     assert(delete, 'ticket destroy')
+    travel_back
   end
 end

test/unit/permission_test.rb

@@ -123,4 +123,48 @@ class PermissionTest < ActiveSupport::TestCase
 
   end
 
+  test 'user permission with childs' do
+
+    permission1 = Permission.create_or_update(
+      name: 'admin.permission_child1',
+      note: 'Admin Interface',
+      preferences: {},
+      active: true,
+    )
+    permission2 = Permission.create_or_update(
+      name: 'admin.permission_child2',
+      note: 'Admin Interface',
+      preferences: {},
+      active: false,
+    )
+    role_permission1 = Role.create_or_update(
+      name: 'AdminPermissionChild1',
+      note: 'To configure your permission child1.',
+      preferences: {
+        not: ['Customer'],
+      },
+      default_at_signup: false,
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    role_permission1.permission_grand('admin')
+    user_with_permission1 = User.create_or_update(
+      login: 'setting-permission-child1',
+      firstname: 'Setting',
+      lastname: 'Admin Permission Child1',
+      email: 'setting-admin-permission-child1@example.com',
+      password: 'some_pw',
+      active: true,
+      roles: [role_permission1],
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    assert(user_with_permission1.permissions_with_child_ids.include?(permission1.id))
+    assert_not(user_with_permission1.permissions_with_child_ids.include?(permission2.id))
+    assert(user_with_permission1.permissions_with_child_ids.include?(Permission.find_by(name: 'admin').id))
+
+    # cleanup
+    user_with_permission1.destroy
+    role_permission1.destroy
+  end
 end

test/unit/recent_view_test.rb

@@ -29,11 +29,11 @@ class RecentViewTest < ActiveSupport::TestCase
     RecentView.user_log_destroy(user1)
 
     RecentView.log(ticket1.class.to_s, ticket1.id, user1)
-    sleep 1
+    travel 1.second
     RecentView.log(ticket2.class.to_s, ticket2.id, user1)
-    sleep 1
+    travel 1.second
     RecentView.log(ticket1.class.to_s, ticket1.id, user1)
-    sleep 1
+    travel 1.second
     RecentView.log(ticket1.class.to_s, ticket1.id, user1)
 
     list = RecentView.list(user1)
@@ -54,6 +54,7 @@ class RecentViewTest < ActiveSupport::TestCase
 
     list = RecentView.list(user1)
     assert(!list[0], 'check if recent view list is empty')
+    travel_back
   end
 
   test 'existing tests' do

test/unit/session_basic_test.rb

@@ -19,7 +19,7 @@ class SessionBasicTest < ActiveSupport::TestCase
     assert_equal(false, result, 'check 1 - expired')
 
     # should be expired
-    sleep 2
+    travel 2.seconds
     result = Sessions::CacheIn.expired('last_run_test')
     assert_equal(true, result, 'check 1 - expired')
 
@@ -32,7 +32,7 @@ class SessionBasicTest < ActiveSupport::TestCase
     assert_equal(false, result, 'check 1 - expired')
 
     # ignore expired
-    sleep 2
+    travel 2.seconds
     result = Sessions::CacheIn.get('last_run_test', ignore_expire: true)
     assert_equal(true, result, 'check 1 - ignore_expire')
 
@@ -50,6 +50,7 @@ class SessionBasicTest < ActiveSupport::TestCase
     Sessions::CacheIn.delete('last_run_delete')
     result = Sessions::CacheIn.get('last_run_delete')
     assert_equal(nil, nil, 'check delete')
+    travel_back
   end
 
   test 'c session create / update' do
@@ -124,14 +125,14 @@ class SessionBasicTest < ActiveSupport::TestCase
     # next check should be empty
     result1 = collection_client1.push
     assert(!result1, 'check collections - recall')
-    sleep 1
+    travel 1.second
     result2 = collection_client2.push
     assert(!result2, 'check collections - recall')
 
     # change collection
     group = Group.first
     group.touch
-    sleep 4
+    travel 4.seconds
 
     # get whole collections
     result1 = collection_client1.push
@@ -149,7 +150,7 @@ class SessionBasicTest < ActiveSupport::TestCase
 
     # change collection
     group = Group.create(name: "SomeGroup::#{rand(999_999)}", active: true)
-    sleep 4
+    travel 4.seconds
 
     # get whole collections
     result1 = collection_client1.push
@@ -159,7 +160,7 @@ class SessionBasicTest < ActiveSupport::TestCase
     assert_equal(result1, result2, 'check collections')
 
     # check again after create
-    sleep 4
+    travel 4.seconds
     result1 = collection_client1.push
     assert(!result1, 'check collections - after create - recall')
     result2 = collection_client2.push
@@ -168,7 +169,7 @@ class SessionBasicTest < ActiveSupport::TestCase
 
     # change collection
     group.destroy
-    sleep 4
+    travel 4.seconds
 
     # get whole collections
     result1 = collection_client1.push
@@ -178,12 +179,13 @@ class SessionBasicTest < ActiveSupport::TestCase
     assert_equal(result1, result2, 'check collections')
 
     # check again after destroy
-    sleep 4
+    travel 4.seconds
     result1 = collection_client1.push
     assert(!result1, 'check collections - after destroy - recall')
     result2 = collection_client2.push
     assert(!result2, 'check collections - after destroy - recall')
     assert_equal(result1, result2, 'check collections')
+    travel_back
   end
 
   test 'c activity stream' do
@@ -206,30 +208,39 @@ class SessionBasicTest < ActiveSupport::TestCase
     agent1.roles = roles
     assert(agent1.save, 'create/update agent1')
 
+    # create min. on activity record
+    random_name = "Random:#{rand(9_999_999_999)}"
+    Group.create_or_update(
+      name: random_name,
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
     as_client1 = Sessions::Backend::ActivityStream.new(agent1, {}, false, '123-1', 3)
 
     # get as stream
     result1 = as_client1.push
     assert(result1, 'check as agent1')
-    sleep 1
+    travel 1.second
 
     # next check should be empty
     result1 = as_client1.push
     assert(!result1, 'check as agent1 - recall')
 
     # next check should be empty
-    sleep 4
+    travel 4.seconds
     result1 = as_client1.push
     assert(!result1, 'check as agent1 - recall 2')
 
     agent1.update_attribute(:email, 'activity-stream-agent11@example.com')
     ticket = Ticket.create(title: '12323', group_id: 1, priority_id: 1, state_id: 1, customer_id: 1)
 
-    sleep 4
+    travel 4.seconds
 
     # get as stream
     result1 = as_client1.push
     assert( result1, 'check as agent1 - recall 3')
+    travel_back
   end
 
   test 'c ticket_create' do
@@ -254,11 +265,12 @@ class SessionBasicTest < ActiveSupport::TestCase
 
     Group.create(name: "SomeTicketCreateGroup::#{rand(999_999)}", active: true)
 
-    sleep 4
+    travel 4.seconds
 
     # get as stream
     result1 = ticket_create_client1.push
     assert(result1, 'check ticket_create - recall 3')
+    travel_back
   end
 
 end

test/unit/session_basic_ticket_test.rb

@@ -38,17 +38,17 @@ class SessionBasicTicketTest < ActiveSupport::TestCase
     assert(!result1, 'check ticket_overview_index - recall')
 
     # next check should be empty / no changes
-    sleep 3
+    travel 3.seconds
     result1 = client1.push
     assert(!result1, 'check ticket_overview_index - recall 2')
 
     # create ticket
     ticket = Ticket.create(title: '12323', group_id: 1, priority_id: 1, state_id: 1, customer_id: 1)
-    sleep 3
+    travel 3.seconds
 
     result1 = client1.push
     assert(result1, 'check ticket_overview_index - recall 3')
-    sleep 3
+    travel 3.seconds
 
     # chnage overview
     overviews = Ticket::Overviews.all(
@@ -70,6 +70,7 @@ class SessionBasicTicketTest < ActiveSupport::TestCase
     Sessions::Backend::TicketOverviewList.reset(user.id)
     result1 = client1.push
     assert(result1, 'check ticket_overview_index - recall 7')
+    travel_back
 
   end
 

test/unit/session_collections_test.rb

@@ -67,7 +67,7 @@ class SessionCollectionsTest < ActiveSupport::TestCase
     assert(check_if_collection_exists(result1, :Role), 'check collections - after init')
     assert(check_if_collection_exists(result1, :Signature), 'check collections - after init')
     assert(check_if_collection_exists(result1, :EmailAddress), 'check collections - after init')
-    sleep 1
+    travel 1.second
     result2 = collection_client2.push
     assert(result2, 'check collections')
     assert(check_if_collection_exists(result2, :Group), 'check collections - after init')
@@ -108,7 +108,7 @@ class SessionCollectionsTest < ActiveSupport::TestCase
     # change collection
     group = Group.first
     group.touch
-    sleep 3
+    travel 3.seconds
 
     # get whole collections
     result1 = collection_client1.push
@@ -130,6 +130,7 @@ class SessionCollectionsTest < ActiveSupport::TestCase
     assert(result2.empty?, 'check collections - recall')
     result3 = collection_client3.push
     assert(result3.empty?, 'check collections - recall')
+    travel_back
   end
 
   def check_if_collection_exists(results, collection, attributes = nil)

test/unit/session_enhanced_test.rb

@@ -242,17 +242,17 @@ class SessionEnhancedTest < ActiveSupport::TestCase
       'Group' => true,
       'User'  => nil,
     }
-    check_if_collection_reset_message_exists(client_id1_0, collections, 'init')
-    check_if_collection_reset_message_exists(client_id1_1, collections, 'init')
-    check_if_collection_reset_message_exists(client_id2, collections, 'init')
+    assert_if_collection_reset_message_exists(client_id1_0, collections, 'init')
+    assert_if_collection_reset_message_exists(client_id1_1, collections, 'init')
+    assert_if_collection_reset_message_exists(client_id2, collections, 'init')
 
     collections = {
       'Group' => nil,
       'User'  => nil,
     }
-    check_if_collection_reset_message_exists(client_id1_0, collections, 'init2')
-    check_if_collection_reset_message_exists(client_id1_1, collections, 'init2')
-    check_if_collection_reset_message_exists(client_id2, collections, 'init2')
+    assert_if_collection_reset_message_exists(client_id1_0, collections, 'init2')
+    assert_if_collection_reset_message_exists(client_id1_1, collections, 'init2')
+    assert_if_collection_reset_message_exists(client_id2, collections, 'init2')
 
     sleep 8
 
@@ -260,9 +260,9 @@ class SessionEnhancedTest < ActiveSupport::TestCase
       'Group' => nil,
       'User'  => nil,
     }
-    check_if_collection_reset_message_exists(client_id1_0, collections, 'init3')
-    check_if_collection_reset_message_exists(client_id1_1, collections, 'init3')
-    check_if_collection_reset_message_exists(client_id2, collections, 'init3')
+    assert_if_collection_reset_message_exists(client_id1_0, collections, 'init3')
+    assert_if_collection_reset_message_exists(client_id1_1, collections, 'init3')
+    assert_if_collection_reset_message_exists(client_id2, collections, 'init3')
 
     # change collection
     group = Group.first
@@ -275,9 +275,9 @@ class SessionEnhancedTest < ActiveSupport::TestCase
       'Group' => true,
       'User'  => nil,
     }
-    check_if_collection_reset_message_exists(client_id1_0, collections, 'update')
-    check_if_collection_reset_message_exists(client_id1_1, collections, 'update')
-    check_if_collection_reset_message_exists(client_id2, collections, 'update')
+    assert_if_collection_reset_message_exists(client_id1_0, collections, 'update')
+    assert_if_collection_reset_message_exists(client_id1_1, collections, 'update')
+    assert_if_collection_reset_message_exists(client_id2, collections, 'update')
 
     # check if session still exists after idle cleanup
     sleep 4
@@ -290,7 +290,7 @@ class SessionEnhancedTest < ActiveSupport::TestCase
 
   end
 
-  def check_if_collection_reset_message_exists(client_id, collections_orig, type)
+  def assert_if_collection_reset_message_exists(client_id, collections_orig, type)
     messages = Sessions.queue(client_id)
     #puts "cid: #{client_id}"
     #puts "m: #{messages.inspect}"
@@ -309,7 +309,7 @@ class SessionEnhancedTest < ActiveSupport::TestCase
     }
     #puts "c: #{collections_result.inspect}"
     collections_orig.each { |key, _value|
-      assert_equal( collections_orig[key], collections_result[key], "collection message for #{key} #{type}-check (client_id #{client_id})" )
+      assert_equal(collections_orig[key], collections_result[key], "collection message for #{key} #{type}-check (client_id #{client_id})")
     }
   end
 end

test/unit/tag_test.rb

@@ -184,7 +184,7 @@ class TagTest < ActiveSupport::TestCase
       updated_by_id: 1,
       created_by_id: 1,
     )
-    sleep 2
+    travel 2.seconds
     Tag.tag_add(
       object: 'Ticket',
       o_id: ticket1.id,
@@ -266,7 +266,7 @@ class TagTest < ActiveSupport::TestCase
     assert(tags_ticket2.include?('some tag4'))
 
     # rename tag
-    sleep 2
+    travel 2.seconds
     tag_item3 = Tag::Item.find_by(name: 'some tag3')
     Tag::Item.rename(
       id: tag_item3.id,
@@ -300,7 +300,7 @@ class TagTest < ActiveSupport::TestCase
     assert(tags_ticket2.include?('some tag4'))
 
     # merge tags
-    sleep 2
+    travel 2.seconds
     Tag::Item.rename(
       id: tag_item3.id,
       name: 'some tag2',
@@ -333,7 +333,7 @@ class TagTest < ActiveSupport::TestCase
     assert_not(Tag::Item.find_by(id: tag_item3.id))
 
     # remove tag item
-    sleep 2
+    travel 2.seconds
     tag_item4 = Tag::Item.find_by(name: 'some TAG4')
     Tag::Item.remove(tag_item4.id)
 
@@ -359,6 +359,6 @@ class TagTest < ActiveSupport::TestCase
     assert_not_equal(ticket1_lookup4.updated_at.to_s, ticket1_lookup3.updated_at.to_s)
     ticket2_lookup4 = Ticket.lookup(id: ticket2.id)
     assert_not_equal(ticket2_lookup4.updated_at.to_s, ticket2_lookup3.updated_at.to_s)
-
+    travel_back
   end
 end

test/unit/ticket_overview_test.rb

@@ -8,6 +8,14 @@ class TicketOverviewTest < ActiveSupport::TestCase
   customer1 = nil
   customer2 = nil
   customer3 = nil
+  overview1 = nil
+  overview2 = nil
+  overview3 = nil
+  overview4 = nil
+  overview5 = nil
+  overview6 = nil
+  overview7 = nil
+  overview8 = nil
   test 'aaa - setup' do
 
     # create base
@@ -93,7 +101,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
     Overview.destroy_all
     UserInfo.current_user_id = 1
     overview_role = Role.find_by(name: 'Agent')
-    Overview.create_or_update(
+    overview1 = Overview.create_or_update(
       name: 'My assigned Tickets',
       link: 'my_assigned',
       prio: 1000,
@@ -120,7 +128,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
       },
     )
 
-    Overview.create_or_update(
+    overview2 = Overview.create_or_update(
       name: 'Unassigned & Open',
       link: 'all_unassigned',
       prio: 1010,
@@ -146,7 +154,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
         view_mode_default: 's',
       },
     )
-    Overview.create_or_update(
+    overview3 = Overview.create_or_update(
       name: 'My Tickets 2',
       link: 'my_tickets_2',
       prio: 1020,
@@ -173,9 +181,36 @@ class TicketOverviewTest < ActiveSupport::TestCase
         view_mode_default: 's',
       },
     )
+    overview4 = Overview.create_or_update(
+      name: 'My Tickets only with Note',
+      link: 'my_tickets_onyl_with_note',
+      prio: 1030,
+      role_id: overview_role.id,
+      user_ids: [agent1.id],
+      condition: {
+        'article.type_id' => {
+          operator: 'is',
+          value: Ticket::Article::Type.find_by(name: 'note').id,
+        },
+        'ticket.owner_id' => {
+          operator: 'is',
+          pre_condition: 'current_user.id',
+        },
+      },
+      order: {
+        by: 'created_at',
+        direction: 'ASC',
+      },
+      view: {
+        d: %w(title customer group created_at),
+        s: %w(title customer group created_at),
+        m: %w(number title customer group created_at),
+        view_mode_default: 's',
+      },
+    )
 
     overview_role = Role.find_by(name: 'Customer')
-    Overview.create_or_update(
+    overview5 = Overview.create_or_update(
       name: 'My Tickets',
       link: 'my_tickets',
       prio: 1100,
@@ -201,7 +236,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
         view_mode_default: 's',
       },
     )
-    Overview.create_or_update(
+    overview6 = Overview.create_or_update(
       name: 'My Organization Tickets',
       link: 'my_organization_tickets',
       prio: 1200,
@@ -228,7 +263,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
         view_mode_default: 's',
       },
     )
-    Overview.create_or_update(
+    overview7 = Overview.create_or_update(
       name: 'My Organization Tickets (open)',
       link: 'my_organization_tickets_open',
       prio: 1200,
@@ -258,7 +293,7 @@ class TicketOverviewTest < ActiveSupport::TestCase
     )
 
     overview_role = Role.find_by(name: 'Admin')
-    Overview.create_or_update(
+    overview8 = Overview.create_or_update(
       name: 'Not Shown Admin',
       link: 'not_shown_admin',
       prio: 9900,
@@ -282,14 +317,15 @@ class TicketOverviewTest < ActiveSupport::TestCase
     )
   end
 
-  test 'ticket create' do
+  test 'bbb overiview index' do
 
     result = Ticket::Overviews.all(
       current_user: agent1,
     )
-    assert_equal(2, result.count)
+    assert_equal(3, result.count)
     assert_equal('My assigned Tickets', result[0].name)
     assert_equal('Unassigned & Open', result[1].name)
+    assert_equal('My Tickets only with Note', result[2].name)
 
     result = Ticket::Overviews.all(
       current_user: agent2,
@@ -322,4 +358,504 @@ class TicketOverviewTest < ActiveSupport::TestCase
 
   end
 
+  test 'ccc overiview content' do
+
+    Ticket.destroy_all
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+    assert_equal(result[2][:count], 0)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    ticket1 = Ticket.create(
+      title: 'overview test 1',
+      group: Group.lookup(name: 'OverviewTest'),
+      customer_id: 2,
+      state: Ticket::State.lookup(name: 'new'),
+      priority: Ticket::Priority.lookup(name: '2 normal'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    article1 = Ticket::Article.create(
+      ticket_id: ticket1.id,
+      from: 'some_sender@example.com',
+      to: 'some_recipient@example.com',
+      subject: 'some subject',
+      message_id: 'some@id',
+      body: 'some message... 123',
+      internal: false,
+      sender: Ticket::Article::Sender.find_by(name: 'Customer'),
+      type: Ticket::Article::Type.find_by(name: 'email'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:count], 1)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+    assert_equal(result[2][:count], 0)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    travel 1.second # because of mysql millitime issues
+    ticket2 = Ticket.create(
+      title: 'overview test 2',
+      group: Group.lookup(name: 'OverviewTest'),
+      customer_id: 2,
+      state: Ticket::State.lookup(name: 'new'),
+      priority: Ticket::Priority.lookup(name: '3 high'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    article2 = Ticket::Article.create(
+      ticket_id: ticket2.id,
+      from: 'some_sender@example.com',
+      to: 'some_recipient@example.com',
+      subject: 'some subject',
+      message_id: 'some@id',
+      body: 'some message... 123',
+      internal: false,
+      sender: Ticket::Article::Sender.find_by(name: 'Agent'),
+      type: Ticket::Article::Type.find_by(name: 'note'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:tickets][1][:id], ticket2.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+    assert_equal(result[2][:count], 0)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    ticket2.owner_id = agent1.id
+    ticket2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:count], 1)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    travel 1.second # because of mysql millitime issues
+    ticket3 = Ticket.create(
+      title: 'overview test 3',
+      group: Group.lookup(name: 'OverviewTest'),
+      customer_id: 2,
+      state: Ticket::State.lookup(name: 'new'),
+      priority: Ticket::Priority.lookup(name: '1 low'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    article3 = Ticket::Article.create(
+      ticket_id: ticket3.id,
+      from: 'some_sender@example.com',
+      to: 'some_recipient@example.com',
+      subject: 'some subject',
+      message_id: 'some@id',
+      body: 'some message... 123',
+      internal: false,
+      sender: Ticket::Article::Sender.find_by(name: 'Customer'),
+      type: Ticket::Article::Type.find_by(name: 'email'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+    travel_back
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:tickets][1][:id], ticket3.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    overview2.order = {
+      by: 'created_at',
+      direction: 'DESC',
+    }
+    overview2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket3.id)
+    assert_equal(result[1][:tickets][1][:id], ticket1.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    overview2.order = {
+      by: 'priority_id',
+      direction: 'DESC',
+    }
+    overview2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:tickets][1][:id], ticket3.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    overview2.order = {
+      by: 'priority_id',
+      direction: 'ASC',
+    }
+    overview2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket3.id)
+    assert_equal(result[1][:tickets][1][:id], ticket1.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    overview2.order = {
+      by: 'priority',
+      direction: 'DESC',
+    }
+    overview2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket1.id)
+    assert_equal(result[1][:tickets][1][:id], ticket3.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+    overview2.order = {
+      by: 'priority',
+      direction: 'ASC',
+    }
+    overview2.save!
+
+    result = Ticket::Overviews.index(agent1)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[0][:tickets][0][:id], ticket2.id)
+    assert_equal(result[0][:count], 1)
+    assert_equal(result[0][:tickets].class, Array)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert_not(result[1][:tickets].empty?)
+    assert_equal(result[1][:tickets][0][:id], ticket3.id)
+    assert_equal(result[1][:tickets][1][:id], ticket1.id)
+    assert_equal(result[1][:count], 2)
+    assert_equal(result[2][:overview][:id], overview4.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets only with Note')
+    assert_equal(result[2][:overview][:view], 'my_tickets_onyl_with_note')
+    assert_equal(result[2][:tickets].class, Array)
+    assert_equal(result[2][:tickets][0][:id], ticket2.id)
+    assert_equal(result[2][:count], 1)
+
+    result = Ticket::Overviews.index(agent2)
+    assert_equal(result[0][:overview][:id], overview1.id)
+    assert_equal(result[0][:overview][:name], 'My assigned Tickets')
+    assert_equal(result[0][:overview][:view], 'my_assigned')
+    assert_equal(result[0][:count], 0)
+    assert_equal(result[0][:tickets].class, Array)
+    assert(result[0][:tickets].empty?)
+    assert_equal(result[1][:overview][:id], overview2.id)
+    assert_equal(result[1][:overview][:name], 'Unassigned & Open')
+    assert_equal(result[1][:overview][:view], 'all_unassigned')
+    assert_equal(result[1][:tickets].class, Array)
+    assert(result[1][:tickets].empty?)
+    assert_equal(result[1][:count], 0)
+    assert_equal(result[2][:overview][:id], overview3.id)
+    assert_equal(result[2][:overview][:name], 'My Tickets 2')
+    assert_equal(result[2][:overview][:view], 'my_tickets_2')
+    assert_equal(result[2][:tickets].class, Array)
+    assert(result[2][:tickets].empty?)
+
+  end
+
 end

test/unit/ticket_ref_object_touch_test.rb

@@ -90,7 +90,7 @@ class TicketRefObjectTouchTest < ActiveSupport::TestCase
       assert(false, 'organization1.updated_at has not been updated')
     end
 
-    sleep 4
+    travel 4.seconds
 
     delete = ticket.destroy
     assert(delete, 'ticket destroy')
@@ -109,11 +109,12 @@ class TicketRefObjectTouchTest < ActiveSupport::TestCase
     else
       assert(false, 'organization1.updated_at has not been updated')
     end
+    travel_back
   end
 
   test 'c - check if customer (not organization) has been updated' do
 
-    sleep 6
+    travel 8.seconds
     ticket = Ticket.create(
       title: "some title2\n äöüß",
       group: Group.lookup(name: 'Users'),
@@ -143,7 +144,7 @@ class TicketRefObjectTouchTest < ActiveSupport::TestCase
       assert(true, 'organization1.updated_at has not been updated')
     end
 
-    sleep 4
+    travel 4.seconds
 
     delete = ticket.destroy
     assert(delete, 'ticket destroy')
@@ -162,6 +163,6 @@ class TicketRefObjectTouchTest < ActiveSupport::TestCase
     else
       assert(true, 'organization1.updated_at has not been updated')
     end
-
+    travel_back
   end
 end

test/unit/ticket_selector_test.rb

@@ -100,7 +100,7 @@ class TicketSelectorTest < ActiveSupport::TestCase
     assert(ticket1, 'ticket created')
     assert_equal(ticket1.customer.id, customer1.id)
     assert_equal(ticket1.organization.id, organization1.id)
-    sleep 1
+    travel 1.second
 
     ticket2 = Ticket.create!(
       title: 'some title2',
@@ -116,7 +116,7 @@ class TicketSelectorTest < ActiveSupport::TestCase
     assert(ticket2, 'ticket created')
     assert_equal(ticket2.customer.id, customer2.id)
     assert_equal(ticket2.organization_id, nil)
-    sleep 1
+    travel 1.second
 
     ticket3 = Ticket.create!(
       title: 'some title3',
@@ -133,7 +133,7 @@ class TicketSelectorTest < ActiveSupport::TestCase
     assert(ticket3, 'ticket created')
     assert_equal(ticket3.customer.id, customer2.id)
     assert_equal(ticket3.organization_id, nil)
-    sleep 1
+    travel 1.second
 
     # search not matching
     condition = {
@@ -995,7 +995,7 @@ class TicketSelectorTest < ActiveSupport::TestCase
 
     ticket_count, tickets = Ticket.selectors(condition, 10)
     assert_equal(ticket_count, 0)
-
+    travel_back
   end
 
 end

test/unit/ticket_test.rb

@@ -43,7 +43,7 @@ class TicketTest < ActiveSupport::TestCase
     assert_equal(ticket.close_at, nil, 'ticket.close_at verify - inbound')
 
     # create inbound article #2
-    sleep 2
+    travel 2.seconds
     article_inbound2 = Ticket::Article.create(
       ticket_id: ticket.id,
       from: 'some_sender@example.com',
@@ -91,7 +91,7 @@ class TicketTest < ActiveSupport::TestCase
     assert_equal(ticket.close_at, nil, 'ticket.close_at verify - note')
 
     # create outbound article
-    sleep 2
+    travel 2.seconds
     article_outbound = Ticket::Article.create(
       ticket_id: ticket.id,
       from: 'some_recipient@example.com',
@@ -139,7 +139,7 @@ class TicketTest < ActiveSupport::TestCase
     assert_equal(ticket.close_at, nil, 'ticket.close_at verify - outbound')
 
     # create inbound article #4
-    sleep 2
+    travel 2.seconds
     article_inbound4 = Ticket::Article.create(
       ticket_id: ticket.id,
       from: 'some_sender@example.com',
@@ -189,10 +189,32 @@ class TicketTest < ActiveSupport::TestCase
 
     ticket = Ticket.find(ticket.id)
     assert_equal(ticket.state.name, 'closed', 'state verify')
-    assert_equal(ticket.pending_time, nil )
+    assert_equal(ticket.pending_time, nil)
+
+    # delete article
+    article_note = Ticket::Article.create(
+      ticket_id: ticket.id,
+      from: 'some person',
+      subject: 'some note',
+      body: 'some message',
+      internal: true,
+      sender: Ticket::Article::Sender.find_by(name: 'Agent'),
+      type: Ticket::Article::Type.find_by(name: 'note'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
+    ticket = Ticket.find(ticket.id)
+    assert_equal(ticket.article_count, 7, 'ticket.article_count verify - note')
+
+    article_note.destroy
+
+    ticket = Ticket.find(ticket.id)
+    assert_equal(ticket.article_count, 6, 'ticket.article_count verify - note')
 
     delete = ticket.destroy
     assert(delete, 'ticket destroy')
+    travel_back
   end
 
   test 'ticket latest change' do
@@ -207,7 +229,7 @@ class TicketTest < ActiveSupport::TestCase
     )
     assert_equal(Ticket.latest_change.to_s, ticket1.updated_at.to_s)
 
-    sleep 1
+    travel 1.minute
 
     ticket2 = Ticket.create(
       title: 'latest change 2',
@@ -220,20 +242,21 @@ class TicketTest < ActiveSupport::TestCase
     )
     assert_equal(Ticket.latest_change.to_s, ticket2.updated_at.to_s)
 
-    sleep 1
+    travel 1.minute
 
     ticket1.title = 'latest change 1 - 1'
     ticket1.save
     assert_equal(Ticket.latest_change.to_s, ticket1.updated_at.to_s)
 
-    sleep 1
+    travel 1.minute
 
     ticket1.touch
     assert_equal(Ticket.latest_change.to_s, ticket1.updated_at.to_s)
 
     ticket1.destroy
     assert_equal(Ticket.latest_change.to_s, ticket2.updated_at.to_s)
-
+    ticket2.destroy
+    travel_back
   end
 
   test 'ticket process_pending' do

test/unit/ticket_trigger_test.rb

@@ -4,8 +4,37 @@ require 'test_helper'
 class TicketTriggerTest < ActiveSupport::TestCase
   test '1 basic' do
     trigger1 = Trigger.create_or_update(
+      name: 'aaa loop check',
+      condition: {
+        'article.subject' => {
+          'operator' => 'contains',
+          'value' => 'Thanks for your inquiry',
+        },
+      },
+      perform: {
+        'ticket.tags' => {
+          'operator' => 'add',
+          'value' => 'should_not_loop',
+        },
+        'notification.email' => {
+          'body' => 'some lala',
+          'recipient' => 'ticket_customer',
+          'subject' => 'Thanks for your inquiry - loop check (#{ticket.title})!',
+        },
+      },
+      disable_notification: true,
+      active: true,
+      created_by_id: 1,
+      updated_by_id: 1,
+    )
+
+    trigger2 = Trigger.create_or_update(
       name: 'auto reply',
       condition: {
+        'ticket.action' => {
+          'operator' => 'is',
+          'value' => 'create',
+        },
         'ticket.state_id' => {
           'operator' => 'is',
           'value' => Ticket::State.lookup(name: 'new').id.to_s,
@@ -31,7 +60,54 @@ class TicketTriggerTest < ActiveSupport::TestCase
       updated_by_id: 1,
     )
 
-    trigger2 = Trigger.create_or_update(
+    trigger3 = Trigger.create_or_update(
+      name: 'auto tag 1',
+      condition: {
+        'ticket.action' => {
+          'operator' => 'is',
+          'value' => 'update',
+        },
+        'ticket.state_id' => {
+          'operator' => 'is',
+          'value' => Ticket::State.lookup(name: 'new').id.to_s,
+        }
+      },
+      perform: {
+        'ticket.priority_id' => {
+          'value' => Ticket::Priority.lookup(name: '3 high').id.to_s,
+        },
+        'ticket.tags' => {
+          'operator' => 'remove',
+          'value' => 'kk',
+        },
+      },
+      disable_notification: true,
+      active: true,
+      created_by_id: 1,
+      updated_by_id: 1,
+    )
+
+    trigger4 = Trigger.create_or_update(
+      name: 'auto tag 2',
+      condition: {
+        'ticket.state_id' => {
+          'operator' => 'is',
+          'value' => Ticket::State.lookup(name: 'new').id.to_s,
+        }
+      },
+      perform: {
+        'ticket.tags' => {
+          'operator' => 'add',
+          'value' => 'abc',
+        },
+      },
+      disable_notification: true,
+      active: true,
+      created_by_id: 1,
+      updated_by_id: 1,
+    )
+
+    trigger5 = Trigger.create_or_update(
       name: 'not matching',
       condition: {
         'ticket.state_id' => {
@@ -50,6 +126,31 @@ class TicketTriggerTest < ActiveSupport::TestCase
       updated_by_id: 1,
     )
 
+    trigger6 = Trigger.create_or_update(
+      name: 'zzz last',
+      condition: {
+        'article.subject' => {
+          'operator' => 'contains',
+          'value' => 'some subject 1234',
+        },
+      },
+      perform: {
+        'ticket.tags' => {
+          'operator' => 'add',
+          'value' => 'article_create_trigger',
+        },
+        'notification.email' => {
+          'body' => 'some lala',
+          'recipient' => 'ticket_customer',
+          'subject' => 'Thanks for your inquiry - 1234 check (#{ticket.title})!',
+        },
+      },
+      disable_notification: true,
+      active: true,
+      created_by_id: 1,
+      updated_by_id: 1,
+    )
+
     ticket1 = Ticket.create(
       title: "some <b>title</b>\n äöüß",
       group: Group.lookup(name: 'Users'),
@@ -89,7 +190,7 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_equal('new', ticket1.state.name, 'ticket1.state verify')
     assert_equal('3 high', ticket1.priority.name, 'ticket1.priority verify')
     assert_equal(2, ticket1.articles.count, 'ticket1.articles verify')
-    assert_equal(%w(aa kk), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
+    assert_equal(%w(aa kk abc), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
     article1 = ticket1.articles.last
     assert_match('Zammad <zammad@localhost>', article1.from)
     assert_match('nicole.braun@zammad.org', article1.to)
@@ -108,7 +209,7 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_equal('new', ticket1.state.name, 'ticket1.state verify')
     assert_equal('2 normal', ticket1.priority.name, 'ticket1.priority verify')
     assert_equal(2, ticket1.articles.count, 'ticket1.articles verify')
-    assert_equal(%w(aa kk), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
+    assert_equal(%w(aa kk abc), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
 
     ticket1.state = Ticket::State.lookup(name: 'open')
     ticket1.save
@@ -120,7 +221,7 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_equal('open', ticket1.state.name, 'ticket1.state verify')
     assert_equal('2 normal', ticket1.priority.name, 'ticket1.priority verify')
     assert_equal(2, ticket1.articles.count, 'ticket1.articles verify')
-    assert_equal(%w(aa kk), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
+    assert_equal(%w(aa kk abc), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
 
     ticket1.state = Ticket::State.lookup(name: 'new')
     ticket1.save
@@ -132,14 +233,8 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_equal('Users', ticket1.group.name, 'ticket1.group verify')
     assert_equal('new', ticket1.state.name, 'ticket1.state verify')
     assert_equal('3 high', ticket1.priority.name, 'ticket1.priority verify')
-    assert_equal(3, ticket1.articles.count, 'ticket1.articles verify')
-    assert_equal(%w(aa kk), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
-    article1 = ticket1.articles.last
-    assert_match('Zammad <zammad@localhost>', article1.from)
-    assert_match('nicole.braun@zammad.org', article1.to)
-    assert_match('Thanks for your inquiry (some <b>title</b>  äöüß)!', article1.subject)
-    assert_match('Braun<br>some &lt;b&gt;title&lt;/b&gt;', article1.body)
-    assert_equal('text/html', article1.content_type)
+    assert_equal(2, ticket1.articles.count, 'ticket1.articles verify')
+    assert_equal(%w(aa abc), Tag.tag_list(object: 'Ticket', o_id: ticket1.id))
 
     ticket2 = Ticket.create(
       title: "some title\n äöüß",
@@ -179,11 +274,12 @@ class TicketTriggerTest < ActiveSupport::TestCase
       created_by_id: 1,
     )
     assert(ticket3, 'ticket3 created')
+
     Ticket::Article.create(
       ticket_id: ticket3.id,
       from: 'some_sender@example.com',
       to: 'some_recipient@example.com',
-      subject: 'some subject',
+      subject: 'some subject 1234',
       message_id: 'some@id',
       content_type: 'text/html',
       body: 'some message <b>note</b><br>new line',
@@ -208,9 +304,9 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_equal('Users', ticket3.group.name, 'ticket3.group verify')
     assert_equal('new', ticket3.state.name, 'ticket3.state verify')
     assert_equal('3 high', ticket3.priority.name, 'ticket3.priority verify')
-    assert_equal(2, ticket3.articles.count, 'ticket3.articles verify')
-    assert_equal(%w(aa kk), Tag.tag_list(object: 'Ticket', o_id: ticket3.id))
-    article3 = ticket3.articles.last
+    assert_equal(3, ticket3.articles.count, 'ticket3.articles verify')
+    assert_equal(%w(aa kk abc article_create_trigger), Tag.tag_list(object: 'Ticket', o_id: ticket3.id))
+    article3 = ticket3.articles[1]
     assert_match('Zammad <zammad@localhost>', article3.from)
     assert_match('nicole.braun@zammad.org', article3.to)
     assert_match('Thanks for your inquiry (some <b>title</b>  äöüß3)!', article3.subject)
@@ -218,6 +314,61 @@ class TicketTriggerTest < ActiveSupport::TestCase
     assert_match('&gt; some message note<br>&gt; new line', article3.body)
     assert_no_match('&gt; some message &lt;b&gt;note&lt;/b&gt;<br>&gt; new line', article3.body)
     assert_equal('text/html', article3.content_type)
+    article3 = ticket3.articles[2]
+    assert_match('Zammad <zammad@localhost>', article3.from)
+    assert_match('nicole.braun@zammad.org', article3.to)
+    assert_match('Thanks for your inquiry - 1234 check (some <b>title</b>  äöüß3)!', article3.subject)
+    assert_equal('text/html', article3.content_type)
+
+    Ticket::Article.create(
+      ticket_id: ticket3.id,
+      from: 'some_sender@example.com',
+      to: 'some_recipient@example.com',
+      subject: 'some subject - not 1234',
+      message_id: 'some@id',
+      content_type: 'text/html',
+      body: 'some message <b>note</b><br>new line',
+      internal: false,
+      sender: Ticket::Article::Sender.find_by(name: 'Agent'),
+      type: Ticket::Article::Type.find_by(name: 'note'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
+    Observer::Transaction.commit
+
+    ticket3 = Ticket.lookup(id: ticket3.id)
+    assert_equal('some <b>title</b>  äöüß3', ticket3.title, 'ticket3.title verify')
+    assert_equal('Users', ticket3.group.name, 'ticket3.group verify')
+    assert_equal('new', ticket3.state.name, 'ticket3.state verify')
+    assert_equal('3 high', ticket3.priority.name, 'ticket3.priority verify')
+    assert_equal(4, ticket3.articles.count, 'ticket3.articles verify')
+    assert_equal(%w(aa kk abc article_create_trigger), Tag.tag_list(object: 'Ticket', o_id: ticket3.id))
+
+    Ticket::Article.create(
+      ticket_id: ticket3.id,
+      from: 'some_sender@example.com',
+      to: 'some_recipient@example.com',
+      subject: 'some subject 1234',
+      message_id: 'some@id',
+      content_type: 'text/html',
+      body: 'some message <b>note</b><br>new line',
+      internal: false,
+      sender: Ticket::Article::Sender.find_by(name: 'Agent'),
+      type: Ticket::Article::Type.find_by(name: 'note'),
+      updated_by_id: 1,
+      created_by_id: 1,
+    )
+
+    Observer::Transaction.commit
+
+    ticket3 = Ticket.lookup(id: ticket3.id)
+    assert_equal('some <b>title</b>  äöüß3', ticket3.title, 'ticket3.title verify')
+    assert_equal('Users', ticket3.group.name, 'ticket3.group verify')
+    assert_equal('new', ticket3.state.name, 'ticket3.state verify')
+    assert_equal('3 high', ticket3.priority.name, 'ticket3.priority verify')
+    assert_equal(5, ticket3.articles.count, 'ticket3.articles verify')
+    assert_equal(%w(aa kk abc article_create_trigger), Tag.tag_list(object: 'Ticket', o_id: ticket3.id))
 
     Trigger.destroy_all
   end