date_book 0.0.6 → 0.1.0

data/lib/date_book/concerns/ability.rb

@@ -7,21 +7,18 @@ module DateBook
 
     def initialize_date_book(user)
       alias_action :index, :show, :popover, to: :read
-      if user.has_role? :admin
-        can :manage, :all
-      elsif user.new_record?
-        can :read, Calendar
-        can :read, Event
-      else
-        can :read, Calendar
-        can :create, Calendar
-        can :manage, Calendar, id: Calendar.with_role(:owner, user).ids
 
-        can :read, Event
-        cannot :create, Event
-        can :manage, Event, calendar_id: Calendar.with_role(:owner, user).ids
+      can :read, [Calendar, Event]
+
+      unless user.new_record?
+        my_calendar_ids = Calendar.with_role(:owner, user).ids
+        can :create, Calendar
+        can :manage, Calendar, id: my_calendar_ids
+        can :manage, Event, calendar_id: my_calendar_ids
         can :manage, Event, id: Event.with_role(:owner, user).ids
       end
+
+      can :manage, :all if user.has_role? :admin
     end
   end
 end

data/lib/date_book/concerns/acts_as_calendar.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_calendar behavior in Calendar model
   module ActsAsCalendar
-    def acts_as_calendar(options = {})
+    def acts_as_calendar(_options = {})
       acts_as_ownable
 
       validates_presence_of :name, :slug
@@ -13,17 +16,13 @@ module DateBook
       has_many :events, dependent: :destroy
 
       include InstanceMethods
-      extend ClassMethods
     end
 
+    # Instance Methods
     module InstanceMethods
       def event_occurrences
         events.as_occurrences.ascending
       end
     end
-
-    module ClassMethods
-    end
-
   end
 end

data/lib/date_book/concerns/acts_as_event.rb

@@ -1,6 +1,11 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_event behavior in Event model
   module ActsAsEvent
-    def acts_as_event(options = {})
+    # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/MethodLength
+    def acts_as_event(_options = {})
       acts_as_ownable
 
       delegate :all_day, :duration, to: :schedule
@@ -21,20 +26,26 @@ module DateBook
       accepts_nested_attributes_for :schedule
 
       # Scopes
-      scope :ending_after, -> (start_date) {
+      scope :ending_after, (lambda do |start_date|
         where id: ::EventOccurrence.ending_after(start_date).event_ids
-      }
-      scope :starting_before, -> (end_date) {
+      end)
+      scope :starting_before, (lambda do |end_date|
         where id: ::EventOccurrence.starting_before(end_date).event_ids
-      }
+      end)
 
       include InstanceMethods
       extend ClassMethods
     end
+    # rubocop:enable Metrics/AbcSize
+    # rubocop:enable Metrics/MethodLength
 
+    # Instance Methods
     module InstanceMethods
       def schedule
-        super || build_schedule(date: Time.now.to_date, time: Time.now.to_s(:time))
+        super || build_schedule(
+          date: Time.now.to_date,
+          time: Time.now.to_s(:time)
+        )
       end
 
       def previous_occurrence(occurrence = nil)
@@ -48,19 +59,29 @@ module DateBook
       end
 
       def url(occurrence = nil)
-        DateBook::Engine.routes.url_helpers.calendar_event_path(calendar, slug, occurrence_id: occurrence.id)
+        DateBook::Engine
+          .routes
+          .url_helpers
+          .calendar_event_path(calendar, slug, occurrence_id: occurrence&.id)
       end
 
       def popover_url(occurrence = nil)
-        DateBook::Engine.routes.url_helpers.popover_calendar_event_path(calendar, slug, occurrence_id: occurrence.id)
+        DateBook::Engine
+          .routes
+          .url_helpers
+          .popover_calendar_event_path(
+            calendar,
+            slug,
+            occurrence_id: occurrence&.id
+          )
       end
     end
 
+    # Class Methods
     module ClassMethods
       def as_occurrences
         ::EventOccurrence.for_schedulables('Event', ids)
       end
     end
-
   end
 end

data/lib/date_book/concerns/acts_as_event_occurrence.rb

@@ -1,6 +1,11 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_event_occurrence behavior in EventOccurrence model
   module ActsAsEventOccurrence
-    def acts_as_event_occurrence(options = {})
+    # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/MethodLength
+    def acts_as_event_occurrence(_options = {})
       before_save :set_end_date
 
       alias_attribute :start_date, :date
@@ -10,19 +15,26 @@ module DateBook
       delegate :schedule, to: :schedulable
 
       # Scopes
-      scope :remaining, -> { where('date >= ?',Time.now) }
-      scope :previous, -> { where('date < ?',Time.now) }
-      scope :ending_after, -> (start_date) { (where 'end_date >= ?', start_date) }
-      scope :starting_before, -> (end_date) { where 'date < ?', end_date }
+      scope :remaining, -> { where('date >= ?', Time.now) }
+      scope :previous, -> { where('date < ?', Time.now) }
+      scope :ending_after, (lambda do |start_date|
+        (where 'end_date >= ?', start_date)
+      end)
+      scope :starting_before, ->(end_date) { where 'date < ?', end_date }
       scope :for_events, -> { where(schedulable_type: 'Event') }
-      scope :for_schedulables, -> (model_name, ids) { where(schedulable_type: model_name).where('schedulable_id IN (?)',  ids) }
+      scope :for_schedulables, (lambda do |model_name, ids|
+        where(schedulable_type: model_name).where('schedulable_id IN (?)', ids)
+      end)
       scope :ascending, -> { order date: :asc }
       scope :descending, -> { order date: :desc }
 
       include InstanceMethods
       extend ClassMethods
     end
+    # rubocop:enable Metrics/AbcSize
+    # rubocop:enable Metrics/MethodLength
 
+    # Instance Methods
     module InstanceMethods
       def url
         event.url(self)
@@ -39,7 +51,7 @@ module DateBook
           I18n.localize date, format: :moment_datetime
         end
       end
-      alias_method :start, :start_moment
+      alias start start_moment
 
       def end_moment
         if schedule.all_day
@@ -48,7 +60,7 @@ module DateBook
           I18n.localize end_date, format: :moment_datetime
         end
       end
-      alias_method :end, :end_moment
+      alias end end_moment
 
       private
 
@@ -57,11 +69,11 @@ module DateBook
       end
     end
 
+    # Class Methods
     module ClassMethods
       def event_ids
         for_events.pluck(:schedulable_id).uniq
       end
     end
-
   end
 end

data/lib/date_book/concerns/acts_as_ownable.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_ownable behavior in Calendar and Event models
   module ActsAsOwnable
-    def acts_as_ownable(options = {})
+    def acts_as_ownable(_options = {})
       # Rolify Gem
       resourcify
 
@@ -9,6 +12,7 @@ module DateBook
       include InstanceMethods
     end
 
+    # Instance Methods
     module InstanceMethods
       def owners
         User.with_role(:owner, self)
@@ -17,11 +21,13 @@ module DateBook
       def owners=(revised_owners)
         # Remove owners not listed in new value
         owners.each do |owner|
-          owner.remove_role(:owner, self) unless revised_owners.include? owner
+          next if revised_owners.include? owner
+          owner.remove_role(:owner, self)
         end
         # Add owners not previously held
         revised_owners.each do |revised_owner|
-          revised_owner.add_role(:owner, self) unless owners.include? revised_owner
+          next if owners.include? revised_owner
+          revised_owner.add_role :owner, self
         end
       end
     end

data/lib/date_book/concerns/acts_as_owner.rb

@@ -1,18 +1,17 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_owner behavior in User model
   module ActsAsOwner
-    def acts_as_owner(options = {})
+    def acts_as_owner(_options = {})
       include InstanceMethods
-      extend ClassMethods
     end
 
+    # Instance Methods
     module InstanceMethods
       def calendars
         Calendar.with_role(:owner, self)
       end
     end
-
-    module ClassMethods
-    end
-
   end
 end

data/lib/date_book/concerns/acts_as_schedule.rb

@@ -1,9 +1,13 @@
+# frozen_string_literal: true
+
 module DateBook
+  # Mixin to allow acts_as_schedule behavior in Schedule model
   module ActsAsSchedule
-    def acts_as_schedule(options = {})
+    def acts_as_schedule(_options = {})
       include InstanceMethods
     end
 
+    # Instance Methods
     module InstanceMethods
       def duration_attributes
         OpenStruct.new count: duration_count, unit: duration_unit
@@ -21,8 +25,12 @@ module DateBook
       end
 
       def duration_unit
-        return 'seconds' if duration == 0
-        DateBook.configuration.duration_units.select { |x| is_unit? x }.first || 'seconds'
+        return 'seconds' if duration.zero?
+        DateBook
+          .configuration
+          .duration_units
+          .select { |x| unit_matches? x }
+          .first || 'seconds'
       end
 
       def human_date
@@ -35,10 +43,9 @@ module DateBook
 
       private
 
-      def is_unit?(unit)
-        duration % 1.send(unit.singularize.to_sym).to_i == 0
+      def unit_matches?(unit)
+        (duration % 1.send(unit.singularize.to_sym).to_i).zero?
       end
     end
-
   end
-end
+end

data/lib/date_book/configuration.rb

@@ -12,18 +12,21 @@ module DateBook
   end
 
   def self.weekdays
-    @weekdays ||= Date::DAYNAMES
-                     .map
-                     .with_index do |x, i|
-                       OpenStruct.new(id: i, name: x, slug: x.downcase)
-                     end
-                     .sort_by do |value|
-                       (value.id - week_start_index) % 7
-                     end
+    return @weekdays if @weekdays
+
+    initial_days = Date::DAYNAMES.map.with_index do |x, i|
+      OpenStruct.new(id: i, name: x, slug: x.downcase)
+    end
+
+    @weekdays = initial_days.sort_by do |value|
+      (value.id - week_start_index) % 7
+    end
   end
 
   def self.week_start_index
-    @week_start_index ||= Date::DAYNAMES.find_index(configuration.week_starts_on)
+    @week_start_index ||= Date::DAYNAMES.find_index(
+      configuration.week_starts_on
+    )
   end
 
   # DateBook Configuration
@@ -36,8 +39,8 @@ module DateBook
 
     def initialize
       self.week_starts_on = 'Sunday'
-      self.rules = %w(singular daily weekly monthly)
-      self.duration_units = %w(years months weeks days hours minutes seconds)
+      self.rules = %w[singular daily weekly monthly]
+      self.duration_units = %w[years months weeks days hours minutes seconds]
     end
   end
 end

data/lib/date_book/engine.rb

@@ -19,19 +19,21 @@ module DateBook
     end
 
     initializer 'date_book.assets.precompile' do |app|
-      app.config.assets.precompile += %w(
+      app.config.assets.precompile += %w[
         date_book.js date_book.css bootstrap-wysiwyg/bootstrap-wysiwyg.css
         bootstrap-wysiwyg.js jquery_nested_form.js jquery_nested_form.js
         bootstrap-datetimepicker.js bootstrap-datetimepicker.css all-day-bg.png
-        part-day-bg.png
-      )
+        part-day-bg.png moment.js glyphicons-halflings-regular.eot
+        glyphicons-halflings-regular.svg glyphicons-halflings-regular.ttf
+        glyphicons-halflings-regular.woff glyphicons-halflings-regular.woff2
+      ]
     end
 
-    initializer "date_book.add_middleware" do |app|
+    initializer 'date_book.add_middleware' do |app|
       app.config.middleware.insert_before 0, Rack::Cors do
         allow do
           origins %r{\Ahttp://localhost:3000\z}
-          resource '*', :headers => :any, :methods => [:get, :post, :options]
+          resource '*', headers: :any, methods: %i[get post options]
         end
       end
 

data/lib/date_book/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Message Train module
 module DateBook
   VERSION = File.read(File.expand_path('../../../VERSION', __FILE__))

data/lib/generators/date_book/install/install_generator.rb

@@ -4,8 +4,8 @@ module DateBook
   # DateBook Install Generator
   class InstallGenerator < Rails::Generators::Base
     argument :user_model_name, type: :string, default: 'User'
-    source_root File.expand_path('../templates', __FILE__)
-    require File.expand_path('../../utils', __FILE__)
+    source_root DateBook::Engine.root.join('lib/generators/date_book/install/templates')
+    require DateBook::Engine.root.join('lib/generators/date_book/utils')
     include Generators::Utils
     include Rails::Generators::Migration
 
@@ -21,7 +21,7 @@ module DateBook
           "install it separately, I'll do that now.", :magenta
       )
       generate('devise:install')
-      generate('devise User')
+      generate("devise #{user_model_name}")
     end
 
     def install_cancan
@@ -39,7 +39,23 @@ module DateBook
           'up now.',
         :magenta
       )
-      generate('rolify', 'Role User')
+      generate('rolify', "Role #{user_model_name}")
+    end
+
+    def add_to_user
+      output(
+        "Adding DateBook to your #{user_model_name.downcase} model",
+        :magenta
+      )
+      gsub_file "app/models/#{user_model_name.downcase}.rb", /acts_as_owner/, ''
+      inject_into_file(
+        "app/models/#{user_model_name.downcase}.rb",
+        after: "rolify\n"
+      ) do
+        <<-'RUBY'
+  acts_as_owner
+        RUBY
+      end
     end
 
     def install_bootstrap_leather
@@ -55,7 +71,10 @@ module DateBook
         "Next, you'll need an initializer for Date Book.",
         :magenta
       )
-      template 'config/initializers/date_book.rb', 'config/initializers/date_book.rb'
+      template(
+        'config/initializers/date_book.rb',
+        'config/initializers/date_book.rb'
+      )
     end
 
     def add_models
@@ -65,22 +84,10 @@ module DateBook
       )
       template 'app/models/calendar.rb', 'app/models/calendar.rb'
       template 'app/models/event.rb', 'app/models/event.rb'
-      template 'app/models/event_occurrence.rb', 'app/models/event_occurrence.rb'
-      template 'app/models/schedule.rb', 'app/models/schedule.rb'
-    end
-
-    def add_to_user
-      output 'Adding DateBook to your User model', :magenta
-      gsub_file(
-        'app/models/user.rb',
-        %r{acts_as_owner},
-        ''
+      template(
+        'app/models/event_occurrence.rb', 'app/models/event_occurrence.rb'
       )
-      inject_into_file 'app/models/user.rb', after: "rolify\n" do
-<<-'RUBY'
-  acts_as_owner
-RUBY
-      end
+      template 'app/models/schedule.rb', 'app/models/schedule.rb'
     end
 
     def add_migrations
@@ -88,15 +95,6 @@ RUBY
       rake 'date_book:install:migrations'
     end
 
-    # def add_to_model
-    #   output 'Adding DateBook to your User model', :magenta
-    #   gsub_file 'app/models/user.rb', /^\n  subscriber$/, ''
-    #   inject_into_file(
-    #     'app/models/user.rb', "\n  subscriber",
-    #     after: 'class User < ActiveRecord::Base'
-    #   )
-    # end
-
     def add_route
       output 'Adding DateBook to your routes.rb file', :magenta
       gsub_file(