diff --git a/app/assets/javascripts/app/controllers/ticket_zoom.coffee b/app/assets/javascripts/app/controllers/ticket_zoom.coffee index ef81b5640..c61a5242d 100644 --- a/app/assets/javascripts/app/controllers/ticket_zoom.coffee +++ b/app/assets/javascripts/app/controllers/ticket_zoom.coffee @@ -645,7 +645,7 @@ class App.TicketZoom extends App.Controller state = App.TicketState.findByAttribute('id', @ticket.state_id) return if state && state.default_create is true - # prevent multiple changes for the default follow up state + # prevent multiple changes for the default follow-up state @isDefaultFollowUpStateSet = true # get state @@ -952,7 +952,7 @@ class App.TicketZoom extends App.Controller # reset task @taskReset() - # reset default follow up state + # reset default follow-up state @resetDefaultFollowUpState() # reset/delete uploaded attachments diff --git a/app/assets/javascripts/app/models/group.coffee b/app/assets/javascripts/app/models/group.coffee index e0720453c..58bf8a23d 100644 --- a/app/assets/javascripts/app/models/group.coffee +++ b/app/assets/javascripts/app/models/group.coffee @@ -6,8 +6,8 @@ class App.Group extends App.Model @configure_attributes = [ { name: 'name', display: 'Name', tag: 'input', type: 'text', limit: 100, null: false }, { name: 'assignment_timeout', display: 'Assignment Timeout', tag: 'input', note: 'Assignment timeout in minutes if assigned agent is not working on it. Ticket will be shown as unassigend.', type: 'text', limit: 100, null: true }, - { name: 'follow_up_possible', display: 'Follow up possible',tag: 'select', default: 'yes', options: { yes: 'yes', 'new_ticket': 'do not reopen Ticket but create new Ticket' }, null: false, note: 'Follow up for closed ticket possible or not.', translate: true }, - { name: 'follow_up_assignment', display: 'Assign Follow Ups', tag: 'select', default: 'yes', options: { true: 'yes', false: 'no' }, null: false, note: 'Assign follow up to latest agent again.', translate: true }, + { name: 'follow_up_possible', display: 'Follow-up possible',tag: 'select', default: 'yes', options: { yes: 'yes', 'new_ticket': 'do not reopen Ticket but create new Ticket' }, null: false, note: 'Follow-up for closed ticket possible or not.', translate: true }, + { name: 'follow_up_assignment', display: 'Assign Follow-Ups', tag: 'select', default: 'yes', options: { true: 'yes', false: 'no' }, null: false, note: 'Assign follow-up to latest agent again.', translate: true }, { name: 'email_address_id', display: 'Email', tag: 'select', multiple: false, null: true, relation: 'EmailAddress', nulloption: true, do_not_log: true }, { name: 'signature_id', display: 'Signature', tag: 'select', multiple: false, null: true, relation: 'Signature', nulloption: true, do_not_log: true }, { name: 'note', display: 'Note', tag: 'textarea', note: 'Notes are visible to agents only, never to customers.', limit: 250, null: true }, diff --git a/app/controllers/integration/check_mk_controller.rb b/app/controllers/integration/check_mk_controller.rb index 559503a17..e95cfaa52 100644 --- a/app/controllers/integration/check_mk_controller.rb +++ b/app/controllers/integration/check_mk_controller.rb @@ -20,7 +20,7 @@ class Integration::CheckMkController < ApplicationController # check if ticket with host is open customer = User.lookup(id: 1) - # follow up detection by meta data + # follow-up detection by meta data integration = 'check_mk' open_states = Ticket::State.by_category(:open) ticket_ids = Ticket.where(state: open_states).order(created_at: :desc).limit(5000).pluck(:id) diff --git a/app/controllers/tickets_controller.rb b/app/controllers/tickets_controller.rb index d27754d1f..d90295973 100644 --- a/app/controllers/tickets_controller.rb +++ b/app/controllers/tickets_controller.rb @@ -675,7 +675,7 @@ class TicketsController < ApplicationController return true if ticket.group.follow_up_possible != 'new_ticket' # check if the setting for follow_up_possible is disabled return true if ticket.state.name != 'closed' # check if the ticket state is already closed - raise Exceptions::UnprocessableEntity, 'Cannot follow up on a closed ticket. Please create a new ticket.' + raise Exceptions::UnprocessableEntity, 'Cannot follow-up on a closed ticket. Please create a new ticket.' end def ticket_all(ticket) diff --git a/app/models/channel/filter/bounce_follow_up_check.rb b/app/models/channel/filter/bounce_follow_up_check.rb index 62819acc5..0aa26ec1f 100644 --- a/app/models/channel/filter/bounce_follow_up_check.rb +++ b/app/models/channel/filter/bounce_follow_up_check.rb @@ -21,7 +21,7 @@ module Channel::Filter::BounceFollowUpCheck article = Ticket::Article.where(message_id_md5: message_id_md5).order('created_at DESC, id DESC').limit(1).first next if !article - Rails.logger.debug { "Follow up for '##{article.ticket.number}' in bounce email." } + Rails.logger.debug { "Follow-up for '##{article.ticket.number}' in bounce email." } mail[ 'x-zammad-ticket-id'.to_sym ] = article.ticket_id mail[ 'x-zammad-is-auto-response'.to_sym ] = true diff --git a/app/models/channel/filter/follow_up_check.rb b/app/models/channel/filter/follow_up_check.rb index 26181a28c..139c93ed0 100644 --- a/app/models/channel/filter/follow_up_check.rb +++ b/app/models/channel/filter/follow_up_check.rb @@ -9,7 +9,7 @@ module Channel::Filter::FollowUpCheck # get ticket# from subject ticket = Ticket::Number.check(mail[:subject]) if ticket - Rails.logger.debug { "Follow up for '##{ticket.number}' in subject." } + Rails.logger.debug { "Follow-up for '##{ticket.number}' in subject." } mail['x-zammad-ticket-id'.to_sym] = ticket.id return true end @@ -20,7 +20,7 @@ module Channel::Filter::FollowUpCheck if setting.include?('body') ticket = Ticket::Number.check(mail[:body].html2text) if ticket - Rails.logger.debug { "Follow up for '##{ticket.number}' in body." } + Rails.logger.debug { "Follow-up for '##{ticket.number}' in body." } mail['x-zammad-ticket-id'.to_sym] = ticket.id return true end @@ -34,7 +34,7 @@ module Channel::Filter::FollowUpCheck ticket = Ticket::Number.check(attachment[:data].html2text) next if !ticket - Rails.logger.debug { "Follow up for '##{ticket.number}' in attachment." } + Rails.logger.debug { "Follow-up for '##{ticket.number}' in attachment." } mail['x-zammad-ticket-id'.to_sym] = ticket.id return true end @@ -61,7 +61,7 @@ module Channel::Filter::FollowUpCheck article = Ticket::Article.where(message_id_md5: message_id_md5).order('created_at DESC, id DESC').limit(1).first next if !article - Rails.logger.debug { "Follow up for '##{article.ticket.number}' in references." } + Rails.logger.debug { "Follow-up for '##{article.ticket.number}' in references." } mail['x-zammad-ticket-id'.to_sym] = article.ticket_id return true end @@ -102,7 +102,7 @@ module Channel::Filter::FollowUpCheck # if subject is different, it's no followup next if subject_to_check != article_first.subject - Rails.logger.debug { "Follow up for '##{article.ticket.number}' in references with same subject as inital article." } + Rails.logger.debug { "Follow-up for '##{article.ticket.number}' in references with same subject as inital article." } mail['x-zammad-ticket-id'.to_sym] = article_first.ticket_id return true end diff --git a/app/models/channel/filter/follow_up_possible_check.rb b/app/models/channel/filter/follow_up_possible_check.rb index 959c882ee..b64043585 100644 --- a/app/models/channel/filter/follow_up_possible_check.rb +++ b/app/models/channel/filter/follow_up_possible_check.rb @@ -10,7 +10,7 @@ module Channel::Filter::FollowUpPossibleCheck return true if !ticket return true if ticket.state.state_type.name !~ /^(closed|merged|removed)/i - # in case of closed tickets, remove follow up information + # in case of closed tickets, remove follow-up information case ticket.group.follow_up_possible when 'new_ticket' mail[:subject] = ticket.subject_clean(mail[:subject]) diff --git a/app/models/channel/filter/monitoring_base.rb b/app/models/channel/filter/monitoring_base.rb index 9d1f52b40..ea5911ae8 100644 --- a/app/models/channel/filter/monitoring_base.rb +++ b/app/models/channel/filter/monitoring_base.rb @@ -80,7 +80,7 @@ class Channel::Filter::MonitoringBase end end - # follow up detection by meta data + # follow-up detection by meta data open_states = Ticket::State.by_category(:open) ticket_ids = Ticket.where(state: open_states).order(created_at: :desc).limit(5000).pluck(:id) ticket_ids.each do |ticket_id| diff --git a/app/models/observer/ticket/article_changes.rb b/app/models/observer/ticket/article_changes.rb index 8f337e1cb..01c8c80e0 100644 --- a/app/models/observer/ticket/article_changes.rb +++ b/app/models/observer/ticket/article_changes.rb @@ -111,7 +111,7 @@ class Observer::Ticket::ArticleChanges < ActiveRecord::Observer ticket = record.ticket if sender.name == 'Customer' - # in case, update last_contact_customer_at on any customer follow up + # in case, update last_contact_customer_at on any customer follow-up if Setting.get('ticket_last_contact_behaviour') == 'based_on_customer_reaction' # set last_contact_at customer diff --git a/db/migrate/20170116000002_fixed_typos_622.rb b/db/migrate/20170116000002_fixed_typos_622.rb index d0b224b26..438fd606d 100644 --- a/db/migrate/20170116000002_fixed_typos_622.rb +++ b/db/migrate/20170116000002_fixed_typos_622.rb @@ -16,7 +16,7 @@ class FixedTypos622 < ActiveRecord::Migration[4.2] 'password_min_2_lower_2_upper_characters' => 'Password needs to contain 2 lower and 2 upper characters.', 'password_need_digit' => 'Password needs to have at least one digit.', 'ticket_subject_size' => 'Max size of the subject in an email reply.', - 'postmaster_follow_up_search_in' => 'In default the follow up check is done via the subject of an email. With this setting you can add more fields where the follow up check is executed.', + 'postmaster_follow_up_search_in' => 'In default the follow-up check is done via the subject of an email. With this setting you can add more fields where the follow-up check is executed.', } setting_map.each do |key, description| diff --git a/db/migrate/20170130000001_follow_up_possible_check_643.rb b/db/migrate/20170130000001_follow_up_possible_check_643.rb index 80b8110c9..87d266552 100644 --- a/db/migrate/20170130000001_follow_up_possible_check_643.rb +++ b/db/migrate/20170130000001_follow_up_possible_check_643.rb @@ -8,7 +8,7 @@ class FollowUpPossibleCheck643 < ActiveRecord::Migration[4.2] title: 'Define postmaster filter.', name: '0200_postmaster_filter_follow_up_possible_check', area: 'Postmaster::PreFilter', - description: 'Define postmaster filter to check if follow ups get created (based on admin settings).', + description: 'Define postmaster filter to check if follow-ups get created (based on admin settings).', options: {}, state: 'Channel::Filter::FollowUpPossibleCheck', frontend: false @@ -18,7 +18,7 @@ class FollowUpPossibleCheck643 < ActiveRecord::Migration[4.2] force: true, object: 'Group', name: 'follow_up_possible', - display: 'Follow up possible', + display: 'Follow-up possible', data_type: 'select', data_option: { default: 'yes', @@ -27,7 +27,7 @@ class FollowUpPossibleCheck643 < ActiveRecord::Migration[4.2] new_ticket: 'do not reopen Ticket but create new Ticket' }, null: false, - note: 'Follow up for closed ticket possible or not.', + note: 'Follow-up for closed ticket possible or not.', translate: true }, editable: false, diff --git a/db/migrate/20170516000001_trigger_recipient_update.rb b/db/migrate/20170516000001_trigger_recipient_update.rb index 07b3ecee6..2df46112b 100644 --- a/db/migrate/20170516000001_trigger_recipient_update.rb +++ b/db/migrate/20170516000001_trigger_recipient_update.rb @@ -4,7 +4,7 @@ class TriggerRecipientUpdate < ActiveRecord::Migration[4.2] # return if it's a new setup return if !Setting.find_by(name: 'system_init_done') - ['auto reply (on new tickets)', 'auto reply (on follow up of tickets)'].each do |name| + ['auto reply (on new tickets)', 'auto reply (on follow-up of tickets)'].each do |name| trigger = Trigger.find_by(name: name) next if trigger.blank? diff --git a/db/seeds/object_manager_attributes.rb b/db/seeds/object_manager_attributes.rb index 21e6e6e7e..8eee44333 100644 --- a/db/seeds/object_manager_attributes.rb +++ b/db/seeds/object_manager_attributes.rb @@ -1495,7 +1495,7 @@ ObjectManager::Attribute.add( force: true, object: 'Group', name: 'follow_up_possible', - display: 'Follow up possible', + display: 'Follow-up possible', data_type: 'select', data_option: { default: 'yes', @@ -1504,7 +1504,7 @@ ObjectManager::Attribute.add( new_ticket: 'do not reopen Ticket but create new Ticket' }, null: false, - note: 'Follow up for closed ticket possible or not.', + note: 'Follow-up for closed ticket possible or not.', translate: true }, editable: false, @@ -1531,7 +1531,7 @@ ObjectManager::Attribute.add( force: true, object: 'Group', name: 'follow_up_assignment', - display: 'Assign Follow Ups', + display: 'Assign Follow-Ups', data_type: 'select', data_option: { default: 'yes', @@ -1540,7 +1540,7 @@ ObjectManager::Attribute.add( false: 'no', }, null: false, - note: 'Assign follow up to latest agent again.', + note: 'Assign follow-up to latest agent again.', translate: true }, editable: false, diff --git a/db/seeds/settings.rb b/db/seeds/settings.rb index 3b4d6aff2..c5d08b69b 100644 --- a/db/seeds/settings.rb +++ b/db/seeds/settings.rb @@ -2515,9 +2515,9 @@ Setting.create_if_not_exists( name: 'postmaster_follow_up_search_in', tag: 'checkbox', options: { - 'references' => 'References - Search for follow up also in In-Reply-To or References headers.', - 'body' => 'Body - Search for follow up also in mail body.', - 'attachment' => 'Attachment - Search for follow up also in attachments.', + 'references' => 'References - Search for follow-up also in In-Reply-To or References headers.', + 'body' => 'Body - Search for follow-up also in mail body.', + 'attachment' => 'Attachment - Search for follow-up also in attachments.', }, }, ], @@ -3309,7 +3309,7 @@ Setting.create_if_not_exists( title: 'Defines postmaster filter.', name: '0200_postmaster_filter_follow_up_possible_check', area: 'Postmaster::PreFilter', - description: 'Define postmaster filter to check if follow ups get created (based on admin settings).', + description: 'Define postmaster filter to check if follow-ups get created (based on admin settings).', options: {}, state: 'Channel::Filter::FollowUpPossibleCheck', frontend: false diff --git a/db/seeds/triggers.rb b/db/seeds/triggers.rb index becee4a4d..10ff3b73b 100644 --- a/db/seeds/triggers.rb +++ b/db/seeds/triggers.rb @@ -42,7 +42,7 @@ Trigger.create_or_update( updated_by_id: 1, ) Trigger.create_or_update( - name: 'auto reply (on follow up of tickets)', + name: 'auto reply (on follow-up of tickets)', condition: { 'ticket.action' => { 'operator' => 'is', @@ -63,7 +63,7 @@ Trigger.create_or_update( }, perform: { 'notification.email' => { - 'body' => '
Your follow up for (#{config.ticket_hook}##{ticket.number}) has been received and will be reviewed by our support staff.
+ 'body' => '
Your follow-up for (#{config.ticket_hook}##{ticket.number}) has been received and will be reviewed by our support staff.
To provide additional information, please reply to this email or click on the following link:
#{config.http_type}://#{config.fqdn}/#ticket/zoom/#{ticket.id}
@@ -673,7 +673,7 @@ class TicketTriggerRecursiveDisabledTest < ActiveSupport::TestCase
Zammad, your customer support system
', 'recipient' => 'ticket_customer', - 'subject' => 'Thanks for your follow up (#{ticket.title})', + 'subject' => 'Thanks for your follow-up (#{ticket.title})', }, }, active: true, @@ -793,7 +793,7 @@ class TicketTriggerRecursiveDisabledTest < ActiveSupport::TestCase assert_equal(6, ticket_p.articles.count, 'ticket_p.articles verify') article_p = ticket_p.articles.last - assert_match('Thanks for your follow up (aaäöüßad asd)', article_p.subject) + assert_match('Thanks for your follow-up (aaäöüßad asd)', article_p.subject) assert_match('ZammadYour follow up for (#{config.ticket_hook}##{ticket.number}) has been received and will be reviewed by our support staff.
+ 'body' => '
Your follow-up for (#{config.ticket_hook}##{ticket.number}) has been received and will be reviewed by our support staff.
To provide additional information, please reply to this email or click on the following link:
#{config.http_type}://#{config.fqdn}/#ticket/zoom/#{ticket.id}
@@ -687,7 +687,7 @@ class TicketTriggerTest < ActiveSupport::TestCase
Zammad, your customer support system
', 'recipient' => 'ticket_customer', - 'subject' => 'Thanks for your follow up (#{ticket.title})', + 'subject' => 'Thanks for your follow-up (#{ticket.title})', }, }, active: true, @@ -807,7 +807,7 @@ class TicketTriggerTest < ActiveSupport::TestCase assert_equal(6, ticket_p.articles.count, 'ticket_p.articles verify') article_p = ticket_p.articles.last - assert_match('Thanks for your follow up (aaäöüßad asd)', article_p.subject) + assert_match('Thanks for your follow-up (aaäöüßad asd)', article_p.subject) assert_match('Zammad