ama_layout 5.11.0 → 5.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b6cab6556e7a4d5677a6d0eb5bf73962f0955910
-  data.tar.gz: 430b36246c6a7591a06429118f359e792e1b05d8
+  metadata.gz: 68fd224038d65d4b4733ef89e62342eb54616de9
+  data.tar.gz: 1748ef1c697447bbc45b6e0de04feda2f16f6f76
 SHA512:
-  metadata.gz: b805a26a984a01280896dc2e4e4f63eaf54c0f3c630f6b0606130a756b0e1d585382765ed59da6c38259b99c339b42b759d6cb47370a1d9b12cb131f6dab59d2
-  data.tar.gz: b98870ea9937be4ff27db4d8df34f3d9103aed56a3fa96f1baf476cbee4c1ecba9e05f396be535e4e34fb80348c675624474c77cf731f59f012274f35e52b959
+  metadata.gz: 03fdbd8ba50676c84aab9634f1476edd64396d483a6d21150f6f002b1f47181066ec1c7467891ae1e872014b56333d24ede29db654a758440bd3f50d85646734
+  data.tar.gz: e96716e1f6c132e99a08ba9b0fd7bb452c2dc3cb3e5abe2d68c1105e13f2465311e664fe75a729c5c1d107e670610916614bb68a028bd35dccd011b4ade57548

data/ama_layout.gemspec

@@ -19,8 +19,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'foundation-rails', '>= 6.4.1.2'
-  spec.add_dependency 'rails', '~> 4.2'
-  spec.add_dependency 'draper', '~> 2.1'
+  spec.add_dependency 'rails', '>= 4.2'
   spec.add_dependency 'browser', '~> 2.0'
   spec.add_dependency 'breadcrumbs_on_rails', '~> 3.0.1'
   spec.add_dependency 'redis-rails'

data/app/views/ama_layout/_sign_out_link.html.erb

@@ -0,0 +1,3 @@
+<li class="side-nav__item">
+  <a class="side-nav__link" href="/logout">Sign Out</a>
+</li>

data/lib/ama_layout.rb

@@ -1,10 +1,10 @@
 require 'ama_layout/version'
 require 'rails/all'
 require 'foundation-rails'
-require 'draper'
 require 'browser'
 require 'breadcrumbs_on_rails'
 require 'redis-rails'
+require 'ama_layout/draper_replacement'
 require 'ama_layout/breadcrumb_builder'
 require 'ama_layout/moneris'
 require 'ama_layout/navigation'
@@ -25,6 +25,7 @@ require 'ama_layout/notifications'
 module AmaLayout
   class Engine < Rails::Engine
     initializer('ama_layout') do
+      Rails.configuration.view_paths = File.join(self.root, 'app', 'views')
       I18n.load_path << File.join(self.root, 'app', 'config', 'locales', 'en.yml')
       ::ActionController::Base.send :include, AmaLayout::ActionController
     end

data/lib/ama_layout/decorators/moneris_decorator.rb

@@ -1,6 +1,7 @@
 module AmaLayout
-  class MonerisDecorator < Draper::Decorator
-    delegate_all
+  class MonerisDecorator
+    include AmaLayout::DraperReplacement
+    extend AmaLayout::DraperReplacement
 
     def textbox
       h.raw File.read textbox_style_file

data/lib/ama_layout/decorators/navigation_decorator.rb

@@ -1,20 +1,19 @@
 module AmaLayout
-  class NavigationDecorator < Draper::Decorator
-    delegate_all
+  class NavigationDecorator
+    include AmaLayout::DraperReplacement
+    extend AmaLayout::DraperReplacement
 
     def items
       object.items.map { |i| i.decorate }
     end
 
     def display_name_text
-      name_or_email.truncate(30)
+      name_or_email.try(:truncate,30)
     end
 
     def sign_out_link
       return "" unless user
-      h.content_tag :li, class: "side-nav__item" do
-        h.concat h.link_to "Sign Out", "/logout", class: "side-nav__link"
-      end
+      h.render partial: "ama_layout/sign_out_link"
     end
 
     def top_nav

data/lib/ama_layout/decorators/navigation_item_decorator.rb

@@ -1,6 +1,7 @@
 module AmaLayout
-  class NavigationItemDecorator < Draper::Decorator
-    delegate_all
+  class NavigationItemDecorator
+    include AmaLayout::DraperReplacement
+    extend AmaLayout::DraperReplacement
 
     def sub_nav
       object.sub_nav.map { |sn| sn.decorate }

data/lib/ama_layout/decorators/notification_decorator.rb

@@ -1,6 +1,7 @@
 module AmaLayout
-  class NotificationDecorator < Draper::Decorator
-    delegate_all
+  class NotificationDecorator
+    include AmaLayout::DraperReplacement
+    extend AmaLayout::DraperReplacement
 
     ICONS = {
       notice: {

data/lib/ama_layout/draper_replacement.rb

@@ -0,0 +1,27 @@
+module AmaLayout
+  module DraperReplacement
+    attr_accessor :object
+
+    def h
+      ActionView::Base.new(::ActionController::Base.view_paths, {}, ::ApplicationController.new)
+    end
+
+    def initialize(args = {})
+      self.object = args
+    end
+
+    def self.decorate_collection(objects = {})
+      objects.map { |o| self.new(o) }
+    end
+
+    def method_missing(method, *args, &block)
+      return super unless delegatable?(method)
+
+      (object || DraperReplacement).send(method, *args, &block)
+    end
+
+    def delegatable?(method)
+      object.respond_to?(method) || DraperReplacement.respond_to?(method)
+    end
+  end
+end

data/lib/ama_layout/moneris.rb

@@ -1,7 +1,10 @@
 module AmaLayout
   class Moneris
     include ActiveModel::Model
-    include Draper::Decoratable
+
+    def decorate
+      AmaLayout::MonerisDecorator.new(self)
+    end
 
     attr_accessor :textbox_style_file
 

data/lib/ama_layout/navigation.rb

@@ -1,7 +1,10 @@
 module AmaLayout
   class Navigation
     include ActiveModel::Model
-    include Draper::Decoratable
+
+    def decorate
+      AmaLayout::NavigationDecorator.new(self)
+    end
 
     attr_accessor :user, :current_url, :nav_file_path, :display_name
 

data/lib/ama_layout/navigation_item.rb

@@ -1,7 +1,10 @@
 module AmaLayout
   class NavigationItem
     include ActiveModel::Model
-    include Draper::Decoratable
+
+    def decorate
+      AmaLayout::NavigationItemDecorator.new(self)
+    end
 
     attr_accessor :text, :icon, :link, :target, :alt, :sub_nav, :current_url
 

data/lib/ama_layout/notification.rb

@@ -1,7 +1,5 @@
 module AmaLayout
   class Notification
-    include Draper::Decoratable
-
     TYPES = %i[notice warning alert].freeze
     DEFAULT_LIFESPAN = 1.year.freeze
     FORMAT_VERSION = '1.0.0'.freeze

data/lib/ama_layout/version.rb

@@ -1,3 +1,3 @@
 module AmaLayout
-  VERSION = '5.11.0'
+  VERSION = '5.12.0'
 end

data/spec/ama_layout/decorators/navigation_decorator_spec.rb

@@ -94,8 +94,9 @@ describe AmaLayout::NavigationDecorator do
     context "with items" do
       it "renders the partial" do
         allow_any_instance_of(AmaLayout::Navigation).to receive(:user).and_return(OpenStruct.new(navigation: "member"))
-        allow_any_instance_of(Draper::HelperProxy).to receive(:render).and_return "render"
-        expect(navigation_presenter.top_nav).to eq "render"
+        allow(Rails.configuration).to receive(:amaabca_site).and_return("amaabca.example.com")
+        allow(navigation_presenter).to receive(:sign_out_link).and_return("sign_out_link")
+        expect(navigation_presenter.top_nav).to include("AMA Road Reports", "http://auth.waffles.ca")
       end
     end
 
@@ -110,8 +111,7 @@ describe AmaLayout::NavigationDecorator do
     context "with items" do
       it "renders the partial" do
         allow_any_instance_of(AmaLayout::Navigation).to receive(:user).and_return(OpenStruct.new(navigation: "member"))
-        allow_any_instance_of(Draper::HelperProxy).to receive(:render).and_return "render"
-        expect(navigation_presenter.sidebar).to eq "render"
+        expect(navigation_presenter.sidebar).to include("Online Account", "Membership Overview", "Rewards Overview")
       end
     end
 
@@ -125,8 +125,7 @@ describe AmaLayout::NavigationDecorator do
   context "account toggle" do
     it "in ama_layout it renders a blank partial" do
       allow_any_instance_of(AmaLayout::Navigation).to receive(:user).and_return(OpenStruct.new(navigation: "member"))
-      allow_any_instance_of(Draper::HelperProxy).to receive(:render).and_return "render"
-      expect(navigation_presenter.account_toggle).to eq "render"
+      expect(navigation_presenter.account_toggle).to eq ""
     end
   end
 
@@ -153,8 +152,7 @@ describe AmaLayout::NavigationDecorator do
 
     describe '#notifications' do
       it 'renders the content to the page' do
-        expect(subject.h).to receive(:render).once.and_return true
-        expect(subject.notifications).to be true
+        expect(subject.notifications).to include("data-notifications-toggle")
       end
     end
 
@@ -217,8 +215,7 @@ describe AmaLayout::NavigationDecorator do
 
     describe '#notification_sidebar' do
       it 'renders content to the page' do
-        expect(subject.h).to receive(:render).once.and_return true
-        expect(subject.notification_sidebar).to be true
+        expect(subject.notification_sidebar).to include("Sign up now","off-canvas position-right")
       end
     end
 

data/spec/ama_layout/decorators/navigation_item_decorator_spec.rb

@@ -39,8 +39,7 @@ describe AmaLayout::NavigationItemDecorator do
     context "with items" do
       it "renders the partial" do
         navigation_item.sub_nav = items
-        allow_any_instance_of(Draper::HelperProxy).to receive(:render).and_return "render"
-        expect(navigation_item_presenter.top_sub_nav).to eq "render"
+        expect(navigation_item_presenter.top_sub_nav).to include("Othersite Overview", "http://othersite.com")
       end
     end
 
@@ -55,8 +54,7 @@ describe AmaLayout::NavigationItemDecorator do
     context "with items" do
       it "renders the partial" do
         navigation_item.sub_nav = items
-        allow_any_instance_of(Draper::HelperProxy).to receive(:render).and_return "render"
-        expect(navigation_item_presenter.sidebar_sub_nav).to eq "render"
+        expect(navigation_item_presenter.sidebar_sub_nav).to include(">Othersite Overview", "http://othersite.com")
       end
     end
 

data/spec/spec_helper.rb

@@ -18,8 +18,6 @@ ActionView::TestCase::TestController.instance_eval do
   helper Rails.application.routes.url_helpers
 end
 
-Draper::ViewContext.test_strategy :fast
-
 RSpec.configure do |config|
   config.infer_spec_type_from_file_location!
   config.include Rails.application.routes.url_helpers

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ama_layout
 version: !ruby/object:Gem::Version
-  version: 5.11.0
+  version: 5.12.0
 platform: ruby
 authors:
 - Michael van den Beuken
@@ -18,7 +18,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-02 00:00:00.000000000 Z
+date: 2017-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foundation-rails
@@ -38,30 +38,16 @@ dependencies:
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '4.2'
-- !ruby/object:Gem::Dependency
-  name: draper
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.1'
 - !ruby/object:Gem::Dependency
   name: browser
   requirement: !ruby/object:Gem::Requirement
@@ -293,6 +279,7 @@ files:
 - app/views/ama_layout/_notification_sidebar.html.erb
 - app/views/ama_layout/_notifications.html.erb
 - app/views/ama_layout/_sidebar.html.erb
+- app/views/ama_layout/_sign_out_link.html.erb
 - app/views/ama_layout/_siteheader.html.erb
 - app/views/ama_layout/_sub_nav.html.erb
 - app/views/ama_layout/_sub_nav_item.html.erb
@@ -309,6 +296,7 @@ files:
 - lib/ama_layout/decorators/navigation_decorator.rb
 - lib/ama_layout/decorators/navigation_item_decorator.rb
 - lib/ama_layout/decorators/notification_decorator.rb
+- lib/ama_layout/draper_replacement.rb
 - lib/ama_layout/moneris.rb
 - lib/ama_layout/moneris/textbox.txt
 - lib/ama_layout/navigation.rb
@@ -377,7 +365,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: ".ama.ab.ca site layouts"