Merge branch 'develop' of git.znuny.com:zammad/zammad into develop

.rubocop.yml

@@ -226,3 +226,9 @@ Lint/AmbiguousBlockAssociation:
   Exclude:
     - "**/*_spec.rb"
     - "**/*_examples.rb"
+
+# Special exceptions
+
+Style/HashSyntax:
+  Exclude:
+    - "**/*.rake"

Rakefile

@@ -2,6 +2,6 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require File.expand_path('config/application', __dir__)
+require_relative 'config/application'
 
 Zammad::Application.load_tasks

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

@@ -294,7 +294,7 @@ class ConnectionWizard extends App.WizardModal
 
         option = ''
         options = {}
-        if !_.isEmpty data.attributes && !_.isEmpty data.attributes.namingcontexts
+        if !_.isEmpty(data.attributes) && !_.isEmpty(data.attributes.namingcontexts)
           for dn in data.attributes.namingcontexts
             options[dn] = dn
             if option is ''

app/controllers/application_controller.rb

@@ -1,6 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'exceptions'
-
 class ApplicationController < ActionController::Base
   include ApplicationController::HandlesErrors
   include ApplicationController::HandlesDevices

app/controllers/calendar_subscriptions_controller.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2015 Zammad Foundation, http://zammad-foundation.org/
-
-require 'icalendar'
-
 class CalendarSubscriptionsController < ApplicationController
   prepend_before_action { authentication_check( { basic_auth_promt: true, permission: 'user_preferences.calendar' } ) }
 

app/controllers/getting_started_controller.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'resolv'
-
 class GettingStartedController < ApplicationController
 
 =begin

app/controllers/import_zendesk_controller.rb

@@ -1,6 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'zendesk_api'
-
 class ImportZendeskController < ApplicationController
 
   def url_check

app/controllers/integration/ldap_controller.rb

@@ -1,7 +1,7 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'ldap'
+require_dependency 'ldap'
-require 'ldap/user'
+require_dependency 'ldap/user'
-require 'ldap/group'
+require_dependency 'ldap/group'
 
 class Integration::LdapController < ApplicationController
   include Integration::ImportJobBase

app/controllers/integration/sipgate_controller.rb

@@ -1,5 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
 require 'builder'
 
 class Integration::SipgateController < ApplicationController

app/controllers/reports_controller.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'tempfile'
-
 class ReportsController < ApplicationController
   prepend_before_action { authentication_check(permission: 'report') }
 

app/controllers/ticket_overviews_controller.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'ticket/overviews'
-
 class TicketOverviewsController < ApplicationController
   prepend_before_action :authentication_check
 

app/models/activity_stream.rb

@@ -1,7 +1,5 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
 class ActivityStream < ApplicationModel
-  load 'activity_stream/assets.rb'
   include ActivityStream::Assets
 
   self.table_name = 'activity_streams'

app/models/activity_stream/assets.rb

@@ -2,6 +2,7 @@
 
 class ActivityStream
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/channel.rb

@@ -1,7 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 class Channel < ApplicationModel
-  load 'channel/assets.rb'
   include Channel::Assets
 
   belongs_to :group

app/models/channel/assets.rb

@@ -2,6 +2,7 @@
 
 class Channel
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/channel/driver/facebook.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2015 Zammad Foundation, http://zammad-foundation.org/
-
-require 'facebook'
-
 class Channel::Driver::Facebook
 
 =begin
@@ -38,7 +35,7 @@ class Channel::Driver::Facebook
     if !access_token
       raise "No access_token found for fb_object_id: #{fb_object_id}"
     end
-    client = Facebook.new(access_token)
+    client = ::Facebook.new(access_token)
     client.from_article(article)
   end
 
@@ -96,7 +93,7 @@ returns
       page = get_page(page_to_sync_id)
       next if !page
       next if page_to_sync_params['group_id'].blank?
-      page_client = Facebook.new(page['access_token'])
+      page_client = ::Facebook.new(page['access_token'])
 
       posts = page_client.client.get_connection('me', 'feed', fields: 'id,from,to,message,created_time,permalink_url,comments{id,from,to,message,created_time}')
       posts.each do |post|

app/models/channel/driver/imap.rb

@@ -1,5 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
 require 'net/imap'
 
 class Channel::Driver::Imap < Channel::EmailParser
@@ -101,7 +100,7 @@ example
     end
 
     Timeout.timeout(timeout) do
-      @imap = Net::IMAP.new(options[:host], port, ssl, nil, false)
+      @imap = ::Net::IMAP.new(options[:host], port, ssl, nil, false)
       if starttls
         @imap.starttls()
       end

app/models/channel/driver/pop3.rb

@@ -62,7 +62,7 @@ returns
 
     Rails.logger.info "fetching pop3 (#{options[:host]}/#{options[:user]} port=#{port},ssl=#{ssl})"
 
-    @pop = Net::POP3.new(options[:host], port)
+    @pop = ::Net::POP3.new(options[:host], port)
     #@pop.set_debug_output $stderr
 
     # on check, reduce open_timeout to have faster probing

app/models/channel/email_build.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'mail'
-
 module Channel::EmailBuild
 
 =begin

app/models/channel/email_parser.rb

@@ -2,9 +2,6 @@
 
 # encoding: utf-8
 
-require 'mail'
-require 'encode'
-
 class Channel::EmailParser
 
 =begin

app/models/chat/session.rb

@@ -2,10 +2,8 @@ class Chat::Session < ApplicationModel
   include HasSearchIndexBackend
   include HasTags
 
-  extend Chat::Session::Search
+  include Chat::Session::Search
-  load 'chat/session/search_index.rb'
   include Chat::Session::SearchIndex
-  load 'chat/session/assets.rb'
   include Chat::Session::Assets
 
   # rubocop:disable Rails/InverseOf

app/models/chat/session/assets.rb

@@ -1,5 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 module Chat::Session::Assets
+  extend ActiveSupport::Concern
 
 =begin
 

app/models/chat/session/search.rb

@@ -1,7 +1,10 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
 class Chat::Session
   module Search
+    extend ActiveSupport::Concern
+
+    # methods defined here are going to extend the class, not the instance of it
+    class_methods do
 
 =begin
 
@@ -79,4 +82,5 @@ returns
         chat_sessions
       end
     end
+  end
 end

app/models/chat/session/search_index.rb

@@ -1,5 +1,9 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 module Chat::Session::SearchIndex
+  extend ActiveSupport::Concern
+
+  # methods defined here are going to extend the class, not the instance of it
+  class_methods do
 
 =begin
 
@@ -32,5 +36,6 @@ returns
 
       attributes
     end
+  end
 
 end

app/models/concerns/can_csv_import.rb

@@ -1,4 +1,5 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
+
 require 'csv'
 
 module CanCsvImport
@@ -58,7 +59,7 @@ returns
         raise Exceptions::UnprocessableEntity, 'Unable to parse empty file/string!'
       end
 
-      rows = CSV.parse(data[:string], data[:parse_params])
+      rows = ::CSV.parse(data[:string], data[:parse_params])
       header = rows.shift
       if header.blank?
         raise Exceptions::UnprocessableEntity, 'Unable to parse file/string without header!'
@@ -301,7 +302,7 @@ returns
         end
         rows_to_add = []
       end
-      CSV.generate(params) do |csv|
+      ::CSV.generate(params) do |csv|
         csv << header
         rows.each do |row|
           csv << row

app/models/history.rb

@@ -1,7 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 class History < ApplicationModel
-  load 'history/assets.rb'
   include History::Assets
 
   self.table_name = 'histories'

app/models/history/assets.rb

@@ -2,6 +2,7 @@
 
 class History
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/job.rb

@@ -4,7 +4,6 @@ class Job < ApplicationModel
   include ChecksClientNotification
   include ChecksConditionValidation
 
-  load 'job/assets.rb'
   include Job::Assets
 
   store     :timeplan

app/models/job/assets.rb

@@ -2,6 +2,7 @@
 
 class Job
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/observer/session.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'history'
-
 class Observer::Session < ActiveRecord::Observer
   observe 'active_record::_session_store::_session'
 

app/models/observer/tag/ticket_history.rb

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'history'
-
 class Observer::Tag::TicketHistory < ActiveRecord::Observer
   observe 'tag'
 

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

@@ -1,7 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'channel/driver/facebook'
-
 class Observer::Ticket::Article::CommunicateFacebook < ActiveRecord::Observer
   observe 'ticket::_article'
 

app/models/observer/ticket/stats_reopen.rb

@@ -1,7 +1,8 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
+require_dependency 'stats/ticket_reopen'
+
 class Observer::Ticket::StatsReopen < ActiveRecord::Observer
-  load 'stats/ticket_reopen.rb'
 
   observe 'ticket'
 

app/models/online_notification.rb

@@ -1,7 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 class OnlineNotification < ApplicationModel
-  load 'online_notification/assets.rb'
   include OnlineNotification::Assets
 
   belongs_to :user

app/models/online_notification/assets.rb

@@ -2,6 +2,7 @@
 
 class OnlineNotification
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/organization.rb

@@ -7,12 +7,10 @@ class Organization < ApplicationModel
   include HasHistory
   include HasSearchIndexBackend
   include CanCsvImport
-  include Organization::ChecksAccess
 
-  load 'organization/assets.rb'
+  include Organization::ChecksAccess
   include Organization::Assets
-  extend Organization::Search
+  include Organization::Search
-  load 'organization/search_index.rb'
   include Organization::SearchIndex
 
   # rubocop:disable Rails/InverseOf

app/models/organization/assets.rb

@@ -2,6 +2,7 @@
 
 class Organization
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/organization/search.rb

@@ -2,6 +2,10 @@
 
 class Organization
   module Search
+    extend ActiveSupport::Concern
+
+    # methods defined here are going to extend the class, not the instance of it
+    class_methods do
 
 =begin
 
@@ -103,4 +107,5 @@ returns
         organizations
       end
     end
+  end
 end

app/models/organization/search_index.rb

@@ -2,6 +2,7 @@
 
 class Organization
   module SearchIndex
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/overview.rb

@@ -6,7 +6,6 @@ class Overview < ApplicationModel
   include ChecksConditionValidation
   include CanSeed
 
-  load 'overview/assets.rb'
   include Overview::Assets
 
   has_and_belongs_to_many :roles, after_add: :cache_update, after_remove: :cache_update, class_name: 'Role'

app/models/overview/assets.rb

@@ -2,6 +2,7 @@
 
 class Overview
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/recent_view.rb

@@ -1,7 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 class RecentView < ApplicationModel
-  load 'recent_view/assets.rb'
   include RecentView::Assets
 
   # rubocop:disable Rails/InverseOf

app/models/recent_view/assets.rb

@@ -2,6 +2,7 @@
 
 class RecentView
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/role.rb

@@ -6,7 +6,6 @@ class Role < ApplicationModel
   include ChecksLatestChangeObserved
   include HasGroups
 
-  load 'role/assets.rb'
   include Role::Assets
 
   has_and_belongs_to_many :users, after_add: :cache_update, after_remove: :cache_update

app/models/role/assets.rb

@@ -2,6 +2,7 @@
 
 class Role
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/scheduler.rb

@@ -1,6 +1,7 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 class Scheduler < ApplicationModel
+  extend ::Mixin::StartFinishLogger
 
   # rubocop:disable Style/ClassVars
   @@jobs_started = {}
@@ -120,10 +121,15 @@ class Scheduler < ApplicationModel
       raise 'This method should only get called when Scheduler.threads are initialized. Use `force: true` to start anyway.'
     end
 
+    log_start_finish(:info, 'Cleanup of left over locked delayed jobs') do
+
       Delayed::Job.all.each do |job|
+        log_start_finish(:info, "Checking left over delayed job #{job.inspect}") do
           cleanup_delayed(job)
         end
       end
+    end
+  end
 
   # Checks if the given job can be rescheduled or destroys it. Logs the action as warn.
   # Works only for locked jobs. Jobs that are not locked are ignored and
@@ -172,7 +178,7 @@ class Scheduler < ApplicationModel
       job.destroy
     end
 
-    Rails.logger.warn "#{action} locked delayed job: #{job_name}"
+    logger.warn "#{action} locked delayed job: #{job_name}"
   end
 
   def self.start_job(job)

app/models/sla.rb

@@ -4,7 +4,6 @@ class Sla < ApplicationModel
   include ChecksClientNotification
   include ChecksConditionValidation
 
-  load 'sla/assets.rb'
   include Sla::Assets
 
   store      :condition

app/models/sla/assets.rb

@@ -2,6 +2,7 @@
 
 class Sla
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/store.rb

@@ -1,9 +1,9 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
-class Store < ApplicationModel
+require_dependency 'store/object'
-  load 'store/object.rb'
+require_dependency 'store/file'
-  load 'store/file.rb'
 
+class Store < ApplicationModel
   # rubocop:disable Rails/InverseOf
   belongs_to :store_object, class_name: 'Store::Object'
   belongs_to :store_file,   class_name: 'Store::File'

app/models/store/file.rb

@@ -1,6 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
-class Store
+class Store < ApplicationModel
   class File < ApplicationModel
     include ApplicationLib
     after_destroy :destroy_provider

app/models/store/object.rb

@@ -1,6 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
-class Store
+class Store < ApplicationModel
   class Object < ApplicationModel
     validates :name, presence: true
   end

app/models/ticket.rb

@@ -15,11 +15,9 @@ class Ticket < ApplicationModel
 
   include Ticket::Escalation
   include Ticket::Subject
-  load 'ticket/assets.rb'
   include Ticket::Assets
-  load 'ticket/search_index.rb'
   include Ticket::SearchIndex
-  extend Ticket::Search
+  include Ticket::Search
 
   store          :preferences
   before_create  :check_generate, :check_defaults, :check_title, :set_default_state, :set_default_priority

app/models/ticket/article.rb

@@ -5,9 +5,8 @@ class Ticket::Article < ApplicationModel
   include HasHistory
   include ChecksHtmlSanitized
   include CanCsvImport
-  include Ticket::Article::ChecksAccess
 
-  load 'ticket/article/assets.rb'
+  include Ticket::Article::ChecksAccess
   include Ticket::Article::Assets
 
   belongs_to :ticket

app/models/ticket/article/assets.rb

@@ -1,6 +1,7 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
 module Ticket::Article::Assets
+  extend ActiveSupport::Concern
 
 =begin
 

app/models/ticket/assets.rb

@@ -1,5 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 module Ticket::Assets
+  extend ActiveSupport::Concern
 
 =begin
 

app/models/ticket/search.rb

@@ -1,5 +1,9 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 module Ticket::Search
+  extend ActiveSupport::Concern
+
+  # methods defined here are going to extend the class, not the instance of it
+  class_methods do
 
 =begin
 
@@ -188,4 +192,5 @@ returns
       end
       tickets
     end
+  end
 end

app/models/ticket/search_index.rb

@@ -1,5 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 module Ticket::SearchIndex
+  extend ActiveSupport::Concern
 
 =begin
 

app/models/transaction/cti_caller_id_detection.rb

@@ -1,6 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'signature_detection'
-
 class Transaction::CtiCallerIdDetection
 
 =begin

app/models/transaction/signature_detection.rb

@@ -1,6 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'signature_detection'
-
 class Transaction::SignatureDetection
 
 =begin
@@ -46,14 +44,14 @@ class Transaction::SignatureDetection
     return if type['name'] != 'email'
 
     # update current signature of user id
-    SignatureDetection.rebuild_user(article.created_by_id)
+    ::SignatureDetection.rebuild_user(article.created_by_id)
 
     # user
     user = User.lookup(id: article.created_by_id)
     return if !user
     return if !user.preferences
     return if !user.preferences[:signature_detection]
-    line = SignatureDetection.find_signature_line_by_article(
+    line = ::SignatureDetection.find_signature_line_by_article(
       user,
       article
     )

app/models/trigger.rb

@@ -4,7 +4,6 @@ class Trigger < ApplicationModel
   include ChecksConditionValidation
   include CanSeed
 
-  load 'trigger/assets.rb'
   include Trigger::Assets
 
   store     :condition

app/models/trigger/assets.rb

@@ -2,6 +2,7 @@
 
 class Trigger
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/user.rb

@@ -1,28 +1,4 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-
-require 'digest/md5'
-
-# @model User
-#
-# @property id(required)    [Integer] The identifier for the User.
-# @property login(required) [String]  The login of the User used for authentication.
-# @property firstname       [String]  The firstname of the User.
-# @property lastname        [String]  The lastname of the User.
-# @property email           [String]  The email of the User.
-# @property image           [String]  The Image used as the User avatar (TODO: Image model?).
-# @property web             [String]  The website/URL of the User.
-# @property password        [String]  The password of the User.
-# @property phone           [String]  The phone number of the User.
-# @property fax             [String]  The fax number of the User.
-# @property mobile          [String]  The mobile number of the User.
-# @property department      [String]  The department the User is working at.
-# @property street          [String]  The street the User lives in.
-# @property zip             [Integer] The zip postal code of the User city.
-# @property city            [String]  The city the User lives in.
-# @property country         [String]  The country the User lives in.
-# @property verified        [Boolean] The flag that shows the verified state of the User.
-# @property active          [Boolean] The flag that shows the active state of the User.
-# @property note            [String]  The note or comment stored to the User.
 class User < ApplicationModel
   include HasActivityStreamLog
   include ChecksClientNotification
@@ -31,12 +7,10 @@ class User < ApplicationModel
   include CanCsvImport
   include HasGroups
   include HasRoles
-  include User::ChecksAccess
 
-  load 'user/assets.rb'
+  include User::ChecksAccess
   include User::Assets
-  extend User::Search
+  include User::Search
-  load 'user/search_index.rb'
   include User::SearchIndex
 
   has_and_belongs_to_many :roles,          after_add: %i[cache_update check_notifications], after_remove: :cache_update, before_add: %i[validate_agent_limit_by_role validate_roles], before_remove: :last_admin_check_by_role, class_name: 'Role'

app/models/user/assets.rb

@@ -2,6 +2,7 @@
 
 class User
   module Assets
+    extend ActiveSupport::Concern
 
 =begin
 

app/models/user/search.rb

@@ -2,6 +2,10 @@
 
 class User
   module Search
+    extend ActiveSupport::Concern
+
+    # methods defined here are going to extend the class, not the instance of it
+    class_methods do
 
 =begin
 
@@ -115,4 +119,5 @@ returns
         users
       end
     end
+  end
 end

app/models/user/search_index.rb

@@ -2,6 +2,7 @@
 
 class User
   module SearchIndex
+    extend ActiveSupport::Concern
 
 =begin
 
@@ -67,6 +68,5 @@ returns
 
       attributes
     end
-
   end
 end

lib/auth/ldap.rb

@@ -1,7 +1,7 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
-require 'ldap'
+require_dependency 'ldap'
-require 'ldap/user'
+require_dependency 'ldap/user'
 
 class Auth
   class Ldap < Auth::Base

lib/encode.rb

@@ -1,4 +1,3 @@
-#require 'iconv'
 module Encode
   def self.conv (charset, string)
 

lib/facebook.rb

@@ -1,7 +1,5 @@
 # Copyright (C) 2012-2015 Zammad Foundation, http://zammad-foundation.org/
 
-require 'koala'
-
 class Facebook
 
   attr_accessor :client, :account

lib/import/exchange/folder.rb

@@ -1,4 +1,4 @@
-require 'mixin/rails_logger'
+require_dependency 'mixin/rails_logger'
 
 module Import
   class Exchange

lib/import/ldap.rb

@@ -1,7 +1,7 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
 
-require 'ldap'
+require_dependency 'ldap'
-require 'ldap/group'
+require_dependency 'ldap/group'
 
 module Import
   class Ldap < Import::IntegrationBase

lib/import/otrs.rb

@@ -2,24 +2,24 @@
 # in the importer folder require AND simultaniuos requiring
 # of the same file in different threads so we need to
 # require them ourself
-require 'import/otrs/ticket'
+require_dependency 'import/otrs/ticket'
-require 'import/otrs/ticket_factory'
+require_dependency 'import/otrs/ticket_factory'
-require 'import/otrs/article_customer'
+require_dependency 'import/otrs/article_customer'
-require 'import/otrs/article_customer_factory'
+require_dependency 'import/otrs/article_customer_factory'
-require 'import/otrs/article'
+require_dependency 'import/otrs/article'
-require 'import/otrs/article_factory'
+require_dependency 'import/otrs/article_factory'
-require 'import/otrs/article/attachment_factory'
+require_dependency 'import/otrs/article/attachment_factory'
-require 'import/otrs/history'
+require_dependency 'import/otrs/history'
-require 'import/otrs/history_factory'
+require_dependency 'import/otrs/history_factory'
-require 'import/otrs/history/article'
+require_dependency 'import/otrs/history/article'
-require 'import/otrs/history/move'
+require_dependency 'import/otrs/history/move'
-require 'import/otrs/history/new_ticket'
+require_dependency 'import/otrs/history/new_ticket'
-require 'import/otrs/history/priority_update'
+require_dependency 'import/otrs/history/priority_update'
-require 'import/otrs/history/state_update'
+require_dependency 'import/otrs/history/state_update'
-require 'store'
+require_dependency 'store'
-require 'store/object'
+require_dependency 'store/object'
-require 'store/provider/db'
+require_dependency 'store/provider/db'
-require 'store/provider/file'
+require_dependency 'store/provider/file'
 
 module Import
   module OTRS

lib/import/otrs/dynamic_field/date.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/otrs/dynamic_field'
+require_dependency 'import/otrs/dynamic_field'
 
 module Import
   module OTRS

lib/import/otrs/dynamic_field/date_time.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/otrs/dynamic_field'
+require_dependency 'import/otrs/dynamic_field'
 
 module Import
   module OTRS

lib/import/otrs/ticket.rb

@@ -1,4 +1,4 @@
-require 'ticket'
+require_dependency 'ticket'
 
 module Import
   module OTRS

lib/import/zendesk/object_attribute/checkbox.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/date.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/integer.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/regexp.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/select.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/text.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/import/zendesk/object_attribute/textarea.rb

@@ -1,7 +1,7 @@
 # this require is required (hehe) because of Rails autoloading
 # which causes strange behavior not inheriting correctly
 # from Import::OTRS::DynamicField
-require 'import/zendesk/object_attribute/base'
+require_dependency 'import/zendesk/object_attribute/base'
 
 module Import
   class Zendesk

lib/ldap.rb

@@ -1,6 +1,6 @@
 # Copyright (C) 2012-2016 Zammad Foundation, http://zammad-foundation.org/
-require 'net/ldap'
+require_dependency 'net/ldap'
-require 'net/ldap/entry'
+require_dependency 'net/ldap/entry'
 
 # Class for establishing LDAP connections. A wrapper around Net::LDAP needed for Auth and Sync.
 # ATTENTION: Loads custom 'net/ldap/entry' from 'lib/core_ext' which extends the Net::LDAP::Entry class.

lib/ldap/group.rb

@@ -6,8 +6,8 @@ class Ldap
   #  an ::Ldap instance.
   #
   # @example
-  #  require 'ldap'
+  #  require_dependency 'ldap'
-  #  require 'ldap/group'
+  #  require_dependency 'ldap/group'
   class Group
     include Ldap::FilterLookup
 
@@ -29,8 +29,8 @@ class Ldap
     # @param ldap [Ldap] An optional existing Ldap class instance. Default is a new connection with given configuration.
     #
     # @example
-    #  require 'ldap'
+    #  require_dependency 'ldap'
-    #  require 'ldap/group'
+    #  require_dependency 'ldap/group'
     #  ldap_group = Ldap::Group.new
     #
     # @return [nil]

lib/ldap/user.rb

@@ -6,8 +6,8 @@ class Ldap
   #  an ::Ldap instance.
   #
   # @example
-  #  require 'ldap'
+  #  require_dependency 'ldap'
-  #  require 'ldap/user'
+  #  require_dependency 'ldap/user'
   class User
     include Ldap::FilterLookup
 
@@ -76,8 +76,8 @@ class Ldap
     # @param ldap [Ldap] An optional existing Ldap class instance. Default is a new connection with given configuration.
     #
     # @example
-    #  require 'ldap'
+    #  require_dependency 'ldap'
-    #  require 'ldap/user'
+    #  require_dependency 'ldap/user'
     #  ldap_user = Ldap::User.new
     #
     # @return [nil]

lib/migration_job/ldap_samaccountname_to_uid.rb

@@ -1,5 +1,5 @@
-require 'ldap'
+require_dependency 'ldap'
-require 'ldap/user'
+require_dependency 'ldap/user'
 
 module MigrationJob
   class LdapSamaccountnameToUid

lib/sequencer.rb

@@ -1,5 +1,5 @@
-require 'mixin/rails_logger'
+require_dependency 'mixin/rails_logger'
-require 'mixin/start_finish_logger'
+require_dependency 'mixin/start_finish_logger'
 
 class Sequencer
   include ::Mixin::RailsLogger

lib/sequencer/sequence.rb

@@ -1,4 +1,4 @@
-require 'sequencer/mixin/prefixed_constantize'
+require_dependency 'sequencer/mixin/prefixed_constantize'
 
 class Sequencer
   class Sequence

lib/sequencer/state.rb

@@ -1,5 +1,5 @@
-require 'mixin/rails_logger'
+require_dependency 'mixin/rails_logger'
-require 'mixin/start_finish_logger'
+require_dependency 'mixin/start_finish_logger'
 
 class Sequencer
   class State

lib/sequencer/unit.rb

@@ -1,4 +1,4 @@
-require 'sequencer/mixin/prefixed_constantize'
+require_dependency 'sequencer/mixin/prefixed_constantize'
 
 class Sequencer
   class Unit

lib/sequencer/unit/base.rb

@@ -1,4 +1,4 @@
-require 'mixin/rails_logger'
+require_dependency 'mixin/rails_logger'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/mapping/flat_keys.rb

@@ -5,8 +5,12 @@ class Sequencer
         module Mapping
           class FlatKeys < Sequencer::Unit::Base
             include ::Sequencer::Unit::Import::Common::Mapping::Mixin::ProvideMapped
+            prepend ::Sequencer::Unit::Import::Common::Model::Mixin::Skip::Action
+
+            skip_any_action
 
             uses :resource
+            provides :mapped
 
             def process
               provide_mapped do

lib/sequencer/unit/import/common/model/associations/assign.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/import/common/model/mixin/handle_failure'
+require_dependency 'sequencer/unit/import/common/model/mixin/handle_failure'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/model/associations/extract.rb

@@ -5,6 +5,9 @@ class Sequencer
         module Model
           module Associations
             class Extract < Sequencer::Unit::Base
+              prepend ::Sequencer::Unit::Import::Common::Model::Mixin::Skip::Action
+
+              skip_any_action
 
               uses :model_class, :mapped
               provides :associations

lib/sequencer/unit/import/common/model/lookup/attributes.rb

@@ -8,7 +8,7 @@ class Sequencer
               include ::Sequencer::Unit::Import::Common::Model::Mixin::HandleFailure
               prepend ::Sequencer::Unit::Import::Common::Model::Mixin::Skip::Action
 
-              skip_action :skipped
+              skip_action :skipped, :failed
 
               uses :mapped, :model_class
               provides :instance

lib/sequencer/unit/import/common/model/save.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/import/common/model/mixin/handle_failure'
+require_dependency 'sequencer/unit/import/common/model/mixin/handle_failure'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/model/skip/blank/base.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/common/mixin/dynamic_attribute'
+require_dependency 'sequencer/unit/common/mixin/dynamic_attribute'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/model/skip/missing_mandatory/base.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/common/mixin/dynamic_attribute'
+require_dependency 'sequencer/unit/common/mixin/dynamic_attribute'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/model/statistics/diff/custom_key.rb

@@ -1,5 +1,4 @@
-require 'sequencer/unit/import/common/model/statistics/mixin/action_diff'
+require_dependency 'sequencer/unit/import/common/model/statistics/mixin/action_diff'
-
 class Sequencer
   class Unit
     module Import

lib/sequencer/unit/import/common/model/statistics/diff/model_key.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/import/common/model/statistics/mixin/action_diff'
+require_dependency 'sequencer/unit/import/common/model/statistics/mixin/action_diff'
 
 class Sequencer
   class Unit

lib/sequencer/unit/import/common/model/statistics/diff/top_level.rb

@@ -1,4 +1,4 @@
-require 'sequencer/unit/import/common/model/statistics/mixin/action_diff'
+require_dependency 'sequencer/unit/import/common/model/statistics/mixin/common'
 
 class Sequencer
   class Unit