jekyll-ical-tag 1.2.1 → 1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b8fa3886e8839e49ce0ca467b8aeb70e0df5610e43d04e9a668776b1779c6d6c
-  data.tar.gz: 59cdb87d2305ce67429ecc434664decc14f6ae76053ef1cb7840faa679ae405f
+  metadata.gz: 21b56f384e8ebf764e98c667f0c5d1def0bf4a15242d69d9faf7c7aa4c0aa91d
+  data.tar.gz: c7f6c97e3befba9801001ecf3f323323e140f20b4d7035eca117a518970b3e37
 SHA512:
-  metadata.gz: 93f3276d4043e1c5a0b5bfde84fa3c01ba4279d2bf60237bab34e5217f2938f91eda04ffd220dab89b7deb1168e5d3d916835dffc49f3d5e0acdc178a7d01cbd
-  data.tar.gz: 10de4f0d4667666a405171225cdd7230d496c6b3aa1722e8c37cbc1bee91521e3d307aaafe6562b2ad399f909dff2b40816f1f5f24aa890bb10adac2d5b6b1af
+  metadata.gz: 4b526819666063d9643194de143026532777dd3e40883aee04a0e49a17413fa8e14bef518b7bd01e74241ce498d12a09d75c1f45401ebd7c0a3da9ea9456b2a9
+  data.tar.gz: 9ae1b75c37b044f2352fa2c20bf26de4a085675abe0f9acd83674d57fb47e783bd2230ab8769b8403fd6067db79866bdca6b409972620e3c043543b5773bc88d

data/.gitignore

@@ -1,3 +1,5 @@
 .DS_Store
 .bundle
-*.gem
+*.gem
+
+spec/examples.txt

data/.rspec

@@ -0,0 +1 @@
+--require spec_helper

data/Gemfile

@@ -7,7 +7,6 @@ gem 'icalendar'
 gem 'activesupport'
 
 group :development, :test do
-  gem 'pry'
   gem 'rspec'
   gem 'nokogiri'
 end

data/Gemfile.lock

@@ -1,7 +1,7 @@
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (5.2.1)
+    activesupport (5.2.4.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
       minitest (~> 5.1)
@@ -9,9 +9,8 @@ GEM
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
     api_cache (0.3.0)
-    coderay (1.1.2)
     colorator (1.1.0)
-    concurrent-ruby (1.0.5)
+    concurrent-ruby (1.1.6)
     diff-lcs (1.3)
     em-websocket (0.5.1)
       eventmachine (>= 0.12.9)
@@ -20,7 +19,7 @@ GEM
     ffi (1.12.2)
     forwardable-extended (2.6.0)
     http_parser.rb (0.6.0)
-    i18n (1.1.1)
+    i18n (1.8.2)
       concurrent-ruby (~> 1.0)
     icalendar (2.5.0)
       ice_cube (~> 0.16)
@@ -44,7 +43,8 @@ GEM
       sassc (> 2.0.1, < 3.0)
     jekyll-watch (2.2.1)
       listen (~> 3.0)
-    kramdown (2.1.0)
+    kramdown (2.3.0)
+      rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
     liquid (4.0.3)
@@ -52,20 +52,17 @@ GEM
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
     mercenary (0.3.6)
-    method_source (0.9.0)
     mini_portile2 (2.4.0)
-    minitest (5.11.3)
+    minitest (5.14.1)
     nokogiri (1.10.8)
       mini_portile2 (~> 2.4.0)
     pathutil (0.16.2)
       forwardable-extended (~> 2.6)
-    pry (0.11.3)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
     public_suffix (4.0.3)
     rb-fsevent (0.10.3)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
+    rexml (3.2.4)
     rouge (3.17.0)
     rspec (3.7.0)
       rspec-core (~> 3.7.0)
@@ -86,7 +83,7 @@ GEM
     terminal-table (1.8.0)
       unicode-display_width (~> 1.1, >= 1.1.1)
     thread_safe (0.3.6)
-    tzinfo (1.2.5)
+    tzinfo (1.2.7)
       thread_safe (~> 0.1)
     unicode-display_width (1.7.0)
 
@@ -99,7 +96,6 @@ DEPENDENCIES
   icalendar
   jekyll
   nokogiri
-  pry
   rspec
 
 BUNDLED WITH

data/bin/rspec

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'rspec' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rspec-core", "rspec")

data/jekyll-ical-tag.gemspec

@@ -30,11 +30,10 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r!^exe/!) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "jekyll", "~> 3"
+  spec.add_dependency "jekyll", "> 3"
   spec.add_dependency "api_cache"
   spec.add_dependency "activesupport"
   spec.add_dependency "icalendar"
-  spec.add_dependency "pry"
 
   spec.add_development_dependency "bundler", "~> 1.15"
   spec.add_development_dependency "rspec", "~> 3.5"

data/lib/jekyll-ical-tag.rb

@@ -6,15 +6,18 @@ require "jekyll-ical-tag/version"
 
 module Jekyll
   class IcalTag < Liquid::Block
-    require_relative "jekyll-ical-tag/event"
-    require_relative "jekyll-ical-tag/calendar_parser"
+    require_relative "jekyll-ical-tag/calendar_feed_coordinator"
+    require_relative "jekyll-ical-tag/calendar_fetcher"
     require_relative "jekyll-ical-tag/calendar_limiter"
+    require_relative "jekyll-ical-tag/calendar_parser"
+    require_relative "jekyll-ical-tag/event"
 
     include Convertible
 
     def initialize(tag_name, markup, parse_context)
       super
       @markup = markup
+      @attributes = {}
 
       scan_attributes!
       set_limit!
@@ -31,22 +34,15 @@ module Jekyll
       result = []
 
       context.stack do
-        url = get_dereferenced_url(context) ||
-              @url
-
-        raise "No URL provided or in innapropriate form '#{url}'" unless is_valid_url?(url)
-
-        puts "Fetching #{url}"
+        url = get_dereferenced_url(context) || @url
 
-        parser = CalendarParser.new(url)
-        parser = CalendarLimiter.new(parser, only: @only)
-        parser = CalendarLimiter.new(parser, reverse: @reverse)
-        parser = CalendarLimiter.new(parser, before_date: @before_date)
-        parser = CalendarLimiter.new(parser, after_date: @after_date)
-        parser = CalendarLimiter.new(parser, limit: @limit)
-
-        events = parser.events
-        length = events.length
+        calendar_feed_coordinator = CalendarFeedCoordinator.new(
+          url: url, only: @only, reverse: @reverse,
+          before_date: @before_date, after_date: @after_date,
+          limit: @limit
+        )
+        events = calendar_feed_coordinator.events
+        event_count = events.length
 
         events.each_with_index do |event, index|
           # Init
@@ -69,32 +65,15 @@ module Jekyll
           context["event"]["end_time"] = context["event"]["dtend"]
           context["event"]["start_time"] = context["event"]["dtstart"]
 
-          # Ensure all event values are utf8 encoded strings
-          # Ensure times (from dates)
-          # Ensure present
-          context["event"].transform_values! do |value|
-            v = case value
-              when String, Icalendar::Values::Text
-                value.force_encoding("UTF-8")
-              when Date, Icalendar::Values::DateTime
-                value.to_time
-              when Icalendar::Values::Uri
-                value.to_s
-              else
-                value
-              end
-            v.presence
-          end
-
           context["forloop"] = {
             "name" => "ical",
-            "length" => length,
+            "length" => event_count,
             "index" => index + 1,
             "index0" => index,
-            "rindex" => length - index,
-            "rindex0" => length - index - 1,
+            "rindex" => event_count - index,
+            "rindex0" => event_count - index - 1,
             "first" => (index == 0),
-            "last" => (index == length - 1),
+            "last" => (index == event_count - 1),
           }
 
           result << nodelist.map do |n|
@@ -112,10 +91,6 @@ module Jekyll
 
     private
 
-    def is_valid_url?(url)
-      !!(url =~ URI::regexp)
-    end
-
     def get_dereferenced_url(context)
       return unless context.key?(@url)
 
@@ -123,7 +98,6 @@ module Jekyll
     end
 
     def scan_attributes!
-      @attributes = {}
       @markup.scan(Liquid::TagAttributes) do |key, value|
         @attributes[key] = value
       end
@@ -147,10 +121,11 @@ module Jekyll
       only_past = @attributes["only_past"] == "true"
 
       raise "Set only_future OR only_past, not both" if only_future && only_past
-      @only = case
-        when only_future
+
+      @only =
+        if only_future
           :future
-        when only_past
+        elsif only_past
           :past
         else
           :all
@@ -158,21 +133,19 @@ module Jekyll
     end
 
     def set_before_date!
-      @before_date = begin
-          if @attributes["before_date"]
-            Time.parse(@attributes["before_date"])
-          end
-        rescue => e
+      @before_date =
+        begin
+          Time.parse(@attributes["before_date"])
+        rescue
           nil
         end
     end
 
     def set_after_date!
-      @after_date = begin
-          if @attributes["after_date"]
-            Time.parse(@attributes["after_date"])
-          end
-        rescue => e
+      @after_date =
+        begin
+          Time.parse(@attributes["after_date"])
+        rescue
           nil
         end
     end

data/lib/jekyll-ical-tag/calendar_feed_coordinator.rb

@@ -0,0 +1,40 @@
+# encoding: utf-8
+# frozen_string_literal: true
+
+module Jekyll
+  class IcalTag
+    class CalendarFeedCoordinator
+      def initialize(url:, only: nil, reverse: nil, before_date: nil, after_date: nil, limit: nil)
+        @url = url
+        @only = only
+        @reverse = reverse
+        @before_date = before_date
+        @after_date = after_date
+        @limit = limit
+      end
+
+      def events
+        parser.events
+      end
+
+      private
+
+      attr_reader :url, :only, :reverse, :before_date, :after_date, :limit
+
+      def raw_ical
+        @raw_ical ||= CalendarFetcher.new(url).fetch
+      end
+
+      def parser
+        @parser ||= begin
+          parser = CalendarParser.new(raw_ical)
+          parser = CalendarLimiter.new(parser, only: only)
+          parser = CalendarLimiter.new(parser, reverse: reverse)
+          parser = CalendarLimiter.new(parser, before_date: before_date)
+          parser = CalendarLimiter.new(parser, after_date: after_date)
+          CalendarLimiter.new(parser, limit: limit)
+        end
+      end
+    end
+  end
+end

data/lib/jekyll-ical-tag/calendar_fetcher.rb

@@ -0,0 +1,30 @@
+# encoding: utf-8
+# frozen_string_literal: true
+
+require "cgi"
+require "api_cache"
+
+module Jekyll
+  class IcalTag
+    class CalendarFetcher
+      def initialize(url)
+        @url = CGI.unescape(url)
+
+        raise "No URL provided or in innapropriate form '#{url}'" unless is_valid_url?
+      end
+
+      def fetch
+        puts "Fetching #{url}"
+        @fetch ||= APICache.get(url)
+      end
+
+      private
+
+      attr_reader :url
+
+      def is_valid_url?
+        !!(url =~ URI::regexp)
+      end
+    end
+  end
+end

data/lib/jekyll-ical-tag/calendar_parser.rb

@@ -1,29 +1,25 @@
 # encoding: utf-8
 # frozen_string_literal: true
 
-require "api_cache"
 require "active_support"
 require "icalendar"
 
 module Jekyll
   class IcalTag
     class CalendarParser
-      def initialize(url)
-        @url = URI.unescape(url)
+      def initialize(raw_feed)
+        @raw_feed = raw_feed
       end
 
       def events
-        @events ||= begin
-          Icalendar::Event.parse(ics_feed)
-                          .sort { |e1, e2| e1.dtstart <=> e2.dtstart }
-                          .map { |e| Jekyll::IcalTag::Event.new(e) }
-        end
+        @events ||= parsed_feed.sort { |event1, event2| event1.dtstart <=> event2.dtstart }
+                               .map { |event| Jekyll::IcalTag::Event.new(event) }
       end
 
       private
 
-      def ics_feed
-        @ics_feed ||= APICache.get(@url)
+      def parsed_feed
+        Icalendar::Event.parse(@raw_feed)
       end
     end
   end

data/lib/jekyll-ical-tag/event.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require "uri"
+require "active_support"
 
 module Jekyll
   class IcalTag
@@ -27,52 +28,84 @@ module Jekyll
         (?::\d{2,5})?           # Optional port number
         (?:\/[^\s"]*)?          # Anything that is not a space or a double quote
         /x
-      extend Forwardable
 
-      def initialize(event)
-        @event = event
+      def initialize(ical_event)
+        @ical_event = ical_event
+        setup_all_properties!
+        setup_property_accessor_methods!
       end
 
-      def_delegators :event, :dtstart, :dtend
-
-      def all_properties
-        @props ||= begin
-            props = {}
-
-            # RFC 5545 Properties
-            event.class.properties.each do |property|
-              props[property] = event.property(property)
-            end
-
-            # custom properties
-            props = props.merge(event.custom_properties)
-
-            props
-          end
-      end
+      attr_reader :all_properties
 
       def simple_html_description
         @simple_html_description ||= begin
-            description&.clone.tap do |d|
-              description_urls.each do |url|
-                d.force_encoding("UTF-8").gsub! url, %(<a href='#{url}'>#{url}</a>)
-              end
+          description.clone.tap do |description|
+            description_urls.each do |url|
+              description.gsub! url, %(<a href='#{url}'>#{url}</a>)
             end
           end
+        end
       end
 
       def attendees
-        attendee.map(&:to_s).map { |a| a.slice!("mailto:"); a }
+        ical_event.attendee.map(&:to_s).map { |a| a.slice!("mailto:"); a }
       end
 
       def description_urls
-        @description_urls ||= description.to_s.force_encoding("UTF-8").scan(URL_REGEX).to_a
+        return [] unless description
+
+        @description_urls ||= description.scan(URL_REGEX).to_a
       end
 
       private
 
-      def_delegators :event, :description, :attendee
-      attr_reader :event
+      attr_reader :ical_event
+
+      def setup_all_properties!
+        @all_properties ||= begin
+          props = {}
+
+          # RFC 5545 Properties
+          ical_event.class.properties.each do |property|
+            props[property] = ical_event.property(property)
+          end
+
+          # custom properties
+          props = props.merge(ical_event.custom_properties)
+
+          # Ensure all arrays get flattened to utf8 encoded strings
+          # Ensure all event values are utf8 encoded strings
+          # Ensure times (from dates)
+          # Ensure present
+          props.transform_values! do |value|
+            new_value =
+              case value
+              when Array, Icalendar::Values::Array
+                value.join("\n").force_encoding("UTF-8")
+              when String, Icalendar::Values::Text
+                value.force_encoding("UTF-8")
+              when Date, Icalendar::Values::DateTime
+                value.to_time
+              when Icalendar::Values::Uri
+                value.to_s
+              else
+                value
+              end
+
+            new_value.presence
+          end
+
+          props
+        end
+      end
+
+      def setup_property_accessor_methods!
+        all_properties.each do |prop, value|
+          define_singleton_method prop do
+            all_properties[prop]
+          end
+        end
+      end
     end
   end
 end

data/lib/jekyll-ical-tag/version.rb

@@ -2,6 +2,6 @@
 
 module Jekyll
   class IcalTag < Liquid::Block
-    VERSION = "1.2.1"
+    VERSION = "1.3"
   end
 end

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-ical-tag
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: '1.3'
 platform: ruby
 authors:
 - Ricky Chilcott
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-04-12 00:00:00.000000000 Z
+date: 2020-11-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
@@ -66,20 +66,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: pry
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -108,7 +94,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.5'
-description: 
+description:
 email:
 - ricky@rakefire.io
 executables: []
@@ -116,13 +102,16 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - Gemfile
 - Gemfile.lock
 - README.md
-- _plugins/ical_tag.rb
 - bin/build-and-publish-gem
+- bin/rspec
 - jekyll-ical-tag.gemspec
 - lib/jekyll-ical-tag.rb
+- lib/jekyll-ical-tag/calendar_feed_coordinator.rb
+- lib/jekyll-ical-tag/calendar_fetcher.rb
 - lib/jekyll-ical-tag/calendar_limiter.rb
 - lib/jekyll-ical-tag/calendar_parser.rb
 - lib/jekyll-ical-tag/event.rb
@@ -132,7 +121,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -148,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: A Jekyll plugin to pull ICS feed and provide a for-like loop of calendar
   events

data/_plugins/ical_tag.rb

@@ -1,187 +0,0 @@
-require "api_cache"
-require "active_support"
-require "icalendar"
-
-class CalendarParser
-  def initialize(url)
-    @url = URI.unescape(url)
-  end
-
-  def events
-    @events ||= Icalendar::Event.parse(ics_feed).sort { |e1, e2| e1.dtstart <=> e2.dtstart }
-  end
-
-  private
-
-  def ics_feed
-    @ics_feed ||= APICache.get(@url)
-  end
-end
-
-class CalendarLimiter
-  def initialize(parser, options = {})
-    @parser = parser
-    @options = options
-  end
-
-  def events
-    case
-    when options[:only] == :future
-      now = Time.now
-      parser.events.select do |event|
-        event.dtstart.to_time >= now
-      end
-    when options[:only] == :past
-      now = Time.now
-      parser.events.select do |event|
-        event.dtstart.to_time < now
-      end
-    when options[:reverse]
-      parser.events.reverse
-    when options[:after_date]
-      parser.events.select do |event|
-        event.dtstart.to_time >= options[:after_date]
-      end
-    when options[:before_date]
-      parser.events.select do |event|
-        event.dtstart.to_time < options[:before_date]
-      end
-    when options[:limit]
-      parser.events.first(options[:limit].to_i)
-    else
-      parser.events
-    end
-  end
-
-  private
-
-  attr_reader :parser, :options
-end
-
-module Jekyll
-  class CalendarTag < Liquid::Block
-    include Convertible
-
-    def initialize(tag_name, markup, parse_context)
-      super
-
-      @markup = markup
-
-      scan_attributes!
-      set_reverse!
-      set_url!
-      set_only!
-      set_before_date!
-      set_after_date!
-    end
-
-    def render(context)
-      context.registers[:ical] ||= Hash.new(0)
-
-      result = []
-
-      parser = CalendarParser.new(@url)
-      parser = CalendarLimiter.new(parser, only: @only)
-      parser = CalendarLimiter.new(parser, reverse: @reverse)
-      parser = CalendarLimiter.new(parser, before_date: @before_date)
-      parser = CalendarLimiter.new(parser, after_date: @after_date)
-
-      events = parser.events
-      length = events.length
-
-      context.stack do
-        events.each_with_index do |event, index|
-          attendees = event.attendee.map(&:to_s).map { |a| a.slice!("mailto:"); a }
-
-          context["event"] = {
-            "index" => index,
-            "uid" => event.uid,
-            "summary" => event.summary,
-            "description" => event.description,
-            "location" => event.location,
-            "url" => event.url&.to_s,
-            "start_time" => event.dtstart&.to_time,
-            "end_time" => event.dtend&.to_time,
-            "attendees" => attendees,
-          }
-
-          context["forloop"] = {
-            "name" => "calendar",
-            "length" => length,
-            "index" => index + 1,
-            "index0" => index,
-            "rindex" => length - index,
-            "rindex0" => length - index - 1,
-            "first" => (index == 0),
-            "last" => (index == length - 1),
-          }
-
-          result << nodelist.map do |n|
-            if n.respond_to? :render
-              n.render(context)
-            else
-              n
-            end
-          end.join
-        end
-      end
-
-      result
-    end
-
-    private
-
-    def scan_attributes!
-      @attributes = {}
-      @markup.scan(Liquid::TagAttributes) do |key, value|
-        @attributes[key] = value
-      end
-    end
-
-    def set_reverse!
-      @reverse = @attributes["order"] == "reverse"
-    end
-
-    def set_url!
-      @url = @attributes["url"]
-      raise "No URL provided" unless @url
-    end
-
-    def set_only!
-      only_future = @attributes["only_future"] == "true"
-      only_past = @attributes["only_past"] == "true"
-
-      raise "Set only_future OR only_past, not both" if only_future && only_past
-      @only = case
-        when only_future
-          :future
-        when only_past
-          :past
-        else
-          :all
-        end
-    end
-
-    def set_before_date!
-      @before_date = begin
-          if @attributes["before_date"]
-            Time.parse(@attributes["before_date"])
-          end
-        rescue => e
-          nil
-        end
-    end
-
-    def set_after_date!
-      @after_date = begin
-          if @attributes["after_date"]
-            Time.parse(@attributes["after_date"])
-          end
-        rescue => e
-          nil
-        end
-    end
-  end
-end
-
-Liquid::Template.register_tag("ical", Jekyll::CalendarTag)