jekyll-ical-tag 1.2.2 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 152da00b22cf9fd63920e1e1c0ce3f9c839e45d809915acbfd675e773d0e93ff
-  data.tar.gz: 8933a1a27439acdc172ffb05f3dc8a4413146d5fd7a458c5e52c6e3b8c81cb38
+  metadata.gz: 9cb9f04ecba697721adc7307872f0370b29b1a351ecb488d69efed62e0c4700f
+  data.tar.gz: 966e0ba6ad28fd3316e278ac1b574e7acd2ccd48198d750afb31a42bbfab1877
 SHA512:
-  metadata.gz: c25659df457036da9d281f22930c379fe42cd13d0a2dea1f4f80118e87d17817888402e1ca40e4fa818e166197b036522791fa91bbd5be28cc93e55e6cbeba48
-  data.tar.gz: ef5afc2ef87e8f85d9284ac25f5bba8efccbde7b1646806612d42441fb90c82e55c260319edcd2e66772da8e85246ca4b928a7231816dc20d8bf9b0f0f3d376c
+  metadata.gz: 145c46aa1d9324812e5342d67b07a2ede7c830632884fdb56ae3ba3e61dbf75cf20b2008a60421ab72f1c4ce2125bc584bf79fbfd4fa2b22334f6240b30245c4
+  data.tar.gz: 2b34febfe473d4c96601e7fdb7ac3b6a73aa835541f72bf0e5432dbcd2b7cfac0b0fc6370627d82c98785c32be239d1822640b61e6f03c00f54e5e6763b55d18

data/.gitignore

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

data/.rspec

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

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)
@@ -10,7 +10,7 @@ GEM
       public_suffix (>= 2.0.2, < 5.0)
     api_cache (0.3.0)
     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)
@@ -19,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)
@@ -43,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.1)
+      rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
     liquid (4.0.3)
@@ -52,7 +53,7 @@ GEM
       rb-inotify (~> 0.9, >= 0.9.10)
     mercenary (0.3.6)
     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)
@@ -61,6 +62,7 @@ GEM
     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)
@@ -81,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)
 

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,7 +30,7 @@ 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"

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.2"
+    VERSION = "1.3.1"
   end
 end

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-ical-tag
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.3.1
 platform: ruby
 authors:
 - Ricky Chilcott
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-04-19 00:00:00.000000000 Z
+date: 2021-03-29 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
@@ -94,7 +94,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.5'
-description: 
+description:
 email:
 - ricky@rakefire.io
 executables: []
@@ -102,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
@@ -118,7 +121,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -134,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)