notification-handler 1.2.5 → 1.2.6

data/app/models/notification_handler/notification.rb

@@ -1,6 +1,8 @@
-class NotificationHandler::Notification < ApplicationRecord
-
-    include NotificationHandler::NotificationLibrary
-    include NotificationHandler::NotificationScopes
-
-end
+# frozen_string_literal: true
+
+module NotificationHandler
+  class Notification < ApplicationRecord
+    include NotificationHandler::NotificationLibrary
+    include NotificationHandler::NotificationScopes
+  end
+end

data/lib/generators/notification_handler/install_generator.rb

@@ -1,46 +1,45 @@
+# frozen_string_literal: true
+
 require 'rails/generators'
 require 'rails/generators/migration'
 
 module NotificationHandler
+  class InstallGenerator < Rails::Generators::Base
+    include Rails::Generators::Migration
+
+    source_root(File.join(File.dirname(__FILE__), '../templates/install'))
+    desc 'Install NotificationHandler'
+
+    def self.next_migration_number(dirname)
+      if ActiveRecord::Base.timestamped_migrations
+        Time.now.utc.strftime('%Y%m%d%H%M%S')
+      else
+        format('%.3d', current_migration_number(dirname) + 1)
+      end
+    end
 
-    class InstallGenerator < Rails::Generators::Base
-
-        include Rails::Generators::Migration
-
-        source_root File.join File.dirname(__FILE__), '../templates/install'
-        desc 'Install NotificationHandler'
-
-        def self.next_migration_number dirname
-            if ActiveRecord::Base.timestamped_migrations
-                Time.now.utc.strftime '%Y%m%d%H%M%S'
-            else
-                "%.3d" % ( current_migration_number(dirname) + 1 )
-            end
-        end
-
-        def create_initializer
-            template 'initializer.rb', 'config/initializers/notification-handler.rb'
-        end
+    def create_initializer
+      template 'initializer.rb', 'config/initializers/notification-handler.rb'
+    end
 
-        def create_notifications_migration_file
-            migration_template 'notifications_migration.rb.erb', 'db/migrate/notification_handler_migration.rb', migration_version: migration_version
-        end
-        def create_notification_model
-            template 'notification_model.rb', 'app/models/notification.rb'
-        end
+    def create_notifications_migration_file
+      migration_template(
+        'notifications_migration.rb.erb',
+        'db/migrate/notification_handler_migration.rb',
+        migration_version: migration_version
+      )
+    end
 
-        def show_readme
-            readme 'README.md'
-        end
+    def create_notification_model
+      template 'notification_model.rb', 'app/models/notification.rb'
+    end
 
-        private
+    private
 
-        def migration_version
-            if Rails.version >= '5.0.0'
-                "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
-            end
-        end
+    def migration_version
+      return unless Rails.version >= '5.0.0'
 
+      "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
     end
-
+  end
 end

data/lib/generators/templates/install/initializer.rb

@@ -1,11 +1,12 @@
-NotificationHandler.configure do |config|
-
-    # Cache amount of unread and read notifications for notification targets
-    # Learn more: https://github.com/jonhue/notifications-rails/tree/master/notification-handler#caching
-    # config.cache = false
-
-    # Groups are a powerful way to bulk-create notifications for multiple objects that don't necessarily have a common class.
-    # Learn more: https://github.com/jonhue/notifications-rails/tree/master/notification-handler#groups
-    # config.define_group :subscribers, User.where(subscriber: true)
-
-end
+# frozen_string_literal: true
+
+NotificationHandler.configure do |config|
+  # Cache amount of unread and read notifications for notification targets.
+  # Learn more: https://github.com/jonhue/notifications-rails/tree/master/notification-handler#caching
+  # config.cache = false
+
+  # Groups are a powerful way to bulk-create notifications for multiple objects
+  # that don't necessarily have a common class.
+  # Learn more: https://github.com/jonhue/notifications-rails/tree/master/notification-handler#groups
+  # config.define_group :subscribers, User.where(subscriber: true)
+end

data/lib/generators/templates/install/notification_model.rb

@@ -1,2 +1,4 @@
-class Notification < NotificationHandler::Notification
-end
+# frozen_string_literal: true
+
+class Notification < NotificationHandler::Notification
+end

data/lib/generators/templates/install/notifications_migration.rb.erb

@@ -1,16 +1,16 @@
-class NotificationHandlerMigration < ActiveRecord::Migration<%= migration_version %>
-    def change
-        create_table :notifications do |t|
-
-            t.references :target, polymorphic: true, index: true
-            t.references :object, polymorphic: true, index: true
-
-            t.boolean :read, default: false, null: false, index: true
-
-            t.text :metadata
-
-            t.timestamps
-
-        end
-    end
-end
+# frozen_string_literal: true
+
+class NotificationHandlerMigration < ActiveRecord::Migration<%= migration_version %>
+  def change
+    create_table :notifications do |t|
+      t.references :target, polymorphic: true, index: true
+      t.references :object, polymorphic: true, index: true
+
+      t.boolean :read, default: false, null: false, index: true
+
+      t.text :metadata
+
+      t.timestamps
+    end
+  end
+end

data/lib/notification-handler.rb

@@ -1,16 +1,16 @@
-module NotificationHandler
-
-    require 'notification_handler/configuration'
+# frozen_string_literal: true
 
-    require 'notification_handler/engine'
+module NotificationHandler
+  require 'notification_handler/configuration'
 
-    autoload :Group, 'notification_handler/group'
+  require 'notification_handler/engine'
 
-    autoload :Target, 'notification_handler/target'
-    autoload :Object, 'notification_handler/object'
-    autoload :NotificationLibrary, 'notification_handler/notification_library'
-    autoload :NotificationScopes, 'notification_handler/notification_scopes'
+  autoload :Group, 'notification_handler/group'
 
-    require 'notification_handler/railtie'
+  autoload :Target, 'notification_handler/target'
+  autoload :Object, 'notification_handler/object'
+  autoload :NotificationLibrary, 'notification_handler/notification_library'
+  autoload :NotificationScopes, 'notification_handler/notification_scopes'
 
+  require 'notification_handler/railtie'
 end

data/lib/notification_handler/configuration.rb

@@ -1,28 +1,26 @@
-module NotificationHandler
-
-    class << self
-        attr_accessor :configuration
-    end
-
-    def self.configure
-        self.configuration ||= Configuration.new
-        yield configuration
-    end
-
-    class Configuration
-
-        attr_accessor :groups
-        attr_accessor :cache
-
-        def initialize
-            @groups = []
-            @cache = false
-        end
-
-        def define_group name, target_scope
-            self.groups << ::NotificationHandler::Group.new(name.to_sym, target_scope)
-        end
-
-    end
-
-end
+# frozen_string_literal: true
+
+module NotificationHandler
+  class << self
+    attr_accessor :configuration
+  end
+
+  def self.configure
+    self.configuration ||= Configuration.new
+    yield configuration
+  end
+
+  class Configuration
+    attr_accessor :groups
+    attr_accessor :cache
+
+    def initialize
+      @groups = []
+      @cache = false
+    end
+
+    def define_group(name, target_scope)
+      groups << ::NotificationHandler::Group.new(name.to_sym, target_scope)
+    end
+  end
+end

data/lib/notification_handler/engine.rb

@@ -1,7 +1,9 @@
-require 'rails/railtie'
-require 'active_record'
-
-module NotificationHandler
-    class Engine < ::Rails::Engine
-    end
-end
+# frozen_string_literal: true
+
+require 'rails/railtie'
+require 'active_record'
+
+module NotificationHandler
+  class Engine < ::Rails::Engine
+  end
+end

data/lib/notification_handler/group.rb

@@ -1,17 +1,19 @@
-module NotificationHandler
-    class Group
-
-        attr_accessor :name
-        attr_accessor :target_scope
-
-        def initialize name, target_scope
-            @name = name
-            @target_scope = target_scope
-        end
-
-        def self.find_by_name name
-            NotificationHandler.configuration.groups.select { |group| group.name == name.to_sym }
-        end
-
-    end
-end
+# frozen_string_literal: true
+
+module NotificationHandler
+  class Group
+    attr_accessor :name
+    attr_accessor :target_scope
+
+    def initialize(name, target_scope)
+      @name = name
+      @target_scope = target_scope
+    end
+
+    def self.find_by_name(name)
+      NotificationHandler.configuration.groups.select do |group|
+        group.name == name.to_sym
+      end
+    end
+  end
+end

data/lib/notification_handler/notification_library.rb

@@ -1,62 +1,68 @@
-require 'active_support'
-
-module NotificationHandler
-    module NotificationLibrary
-
-        extend ActiveSupport::Concern
-
-        included do
-            self.inheritance_column = :_type_disabled
-            
-            before_validation :create_for_group
-            after_commit :cache
-
-            serialize :metadata, Hash
-            attr_accessor :group
-
-            belongs_to :target, polymorphic: true
-            belongs_to :object, polymorphic: true, optional: true
-
-            include NotificationHandler::NotificationLibrary::InstanceMethods
-
-            include NotificationRenderer::NotificationLibrary if defined?(NotificationRenderer)
-            include NotificationPusher::NotificationLibrary if defined?(NotificationPusher)
-            include NotificationSettings::NotificationLibrary if defined?(NotificationSettings)
-        end
-
-        module InstanceMethods
-
-            def read?
-                self.read
-            end
-            def unread?
-                !self.read
-            end
-
-            private
-
-            def create_for_group
-                unless self.group.nil?
-                    target_scope = NotificationHandler::Group.find_by_name(self.group).last.target_scope
-                    target_scope&.each do |target|
-                        notification = self.dup
-                        notification.target = target
-                        notification.group = nil
-                        notification.save
-                    end
-                    return false
-                end
-            end
-
-            def cache
-                if self.read_changed?
-                    self.target.read_notification_count = self.target.notifications.read.count
-                    self.target.unread_notification_count = self.target.notifications.unread.count
-                    self.target.save!
-                end
-            end
-
-        end
-
-    end
-end
+# frozen_string_literal: true
+
+require 'active_support'
+
+module NotificationHandler
+  module NotificationLibrary
+    extend ActiveSupport::Concern
+
+    included do
+      self.inheritance_column = :_type_disabled
+
+      before_validation :create_for_group
+      after_commit :cache
+
+      serialize :metadata, Hash
+      attr_accessor :group
+
+      belongs_to :target, polymorphic: true
+      belongs_to :object, polymorphic: true, optional: true
+
+      include NotificationHandler::NotificationLibrary::InstanceMethods
+
+      if defined?(NotificationRenderer)
+        include NotificationRenderer::NotificationLibrary
+      end
+      if defined?(NotificationPusher)
+        include NotificationPusher::NotificationLibrary
+      end
+      if defined?(NotificationSettings)
+        include NotificationSettings::NotificationLibrary
+      end
+    end
+
+    module InstanceMethods
+      def read?
+        read
+      end
+
+      def unread?
+        !read
+      end
+
+      private
+
+      def create_for_group
+        return if group.nil?
+
+        target_scope = NotificationHandler::Group.find_by_name(group)
+                                                 .last.target_scope
+        target_scope&.each do |target|
+          notification = dup
+          notification.target = target
+          notification.group = nil
+          notification.save
+        end
+        false
+      end
+
+      def cache
+        return unless read_changed?
+
+        target.read_notification_count = target.notifications.read.count
+        target.unread_notification_count = target.notifications.unread.count
+        target.save!
+      end
+    end
+  end
+end

data/lib/notification_handler/notification_scopes.rb

@@ -1,17 +1,21 @@
-require 'active_support'
-
-module NotificationHandler
-    module NotificationScopes
-
-        extend ActiveSupport::Concern
-
-        included do
-            scope :read, -> { where(read: true) }
-            scope :unread, -> { where(read: false) }
-
-            include NotificationRenderer::NotificationScopes if defined?(NotificationRenderer)
-            include NotificationSettings::NotificationScopes if defined?(NotificationSettings)
-        end
-
-    end
-end
+# frozen_string_literal: true
+
+require 'active_support'
+
+module NotificationHandler
+  module NotificationScopes
+    extend ActiveSupport::Concern
+
+    included do
+      scope :read, -> { where(read: true) }
+      scope :unread, -> { where(read: false) }
+
+      if defined?(NotificationRenderer)
+        include NotificationRenderer::NotificationScopes
+      end
+      if defined?(NotificationSettings)
+        include NotificationSettings::NotificationScopes
+      end
+    end
+  end
+end