plug 0.1.23 → 0.1.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb469dfe189640aa012dfba0b1784b6b58af48ab0a72f43f86e18aa9557cf342
-  data.tar.gz: b0ecc6ea23fdbfc103e4b66fbd7ac98c6acbad94e580e2b9372cbc5aa4445607
+  metadata.gz: 887214bb19f4ba2255374fb542b912c43932f1d0aefdc6a4e2c3235957fbf6fd
+  data.tar.gz: ae3217d9d07a8ad3bbf57f3f84c86bd50fa78eab23583850a014850e3049862b
 SHA512:
-  metadata.gz: 88ed333f9efd4189debd2e8b892d3da8a33775a1d47919bab8819314b5b0860bb76f0892cb97dee6b93771a5dc543c6c528f3216d42872f5054f0313a5e9840a
-  data.tar.gz: ca7f17e98fc0902191b0734b61c6cb69a4adff7ca3ba7ba1eee7342b0d049793bc06c8022ebf421f1fab2b813ea847432d62f429f2518f44718a8aa41ae2847f
+  metadata.gz: 389e71cfb8b619d007e164b0ed967ad1c79a2824a85c77c7b8e983d08f4c812a8cecd2ebcf09a8f350887da147c8b9409ac888397410a694edcf11ef72303b9e
+  data.tar.gz: be0c9b7483d6d9d6a019daaa1aaf60a97b92d70dbcf1c7df26e22cb5ec79f642e0fb2663f8d5ce94c5650e20766484737214181302b66d648442eb2b9f42177c

data/README.md

@@ -1,4 +1,4 @@
-# Plug [![Maintainability](https://api.codeclimate.com/v1/badges/6246b1cd8e42603c42f6/maintainability)](https://codeclimate.com/github/DigitalNZ/plug/maintainability) [![Build Status](https://travis-ci.org/DigitalNZ/plug.svg?branch=master)](https://travis-ci.org/DigitalNZ/plug)
+# Plug [![Maintainability](https://api.codeclimate.com/v1/badges/6246b1cd8e42603c42f6/maintainability)](https://codeclimate.com/github/DigitalNZ/plug/maintainability)
 
 A Rails engine to turn on/off features (Feature flipper).
 

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 begin
   require 'bundler/setup'
 rescue LoadError
@@ -14,7 +16,7 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-APP_RAKEFILE = File.expand_path('../spec/dummy/Rakefile', __FILE__)
+APP_RAKEFILE = File.expand_path('spec/dummy/Rakefile', __dir__)
 load 'rails/tasks/engine.rake'
 
 load 'rails/tasks/statistics.rake'

data/app/assets/javascripts/plug/application.js

@@ -10,8 +10,6 @@
 // Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
 // about supported directives.
 //
-//= require jquery
-//= require jquery_ujs
 //= require_tree .
 
 // Prevent drag and drop from the editor

data/app/controllers/plug/api/api_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   module Api
     class ApiController < ActionController::Base

data/app/controllers/plug/api/site_notices_controller.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_dependency 'plug/application_controller'
 
 module Plug

data/app/controllers/plug/application_controller.rb

@@ -1,11 +1,11 @@
+# frozen_string_literal: true
+
 module Plug
   class ApplicationController < ActionController::Base
     protect_from_forgery with: :exception
 
     content_security_policy false
 
-    if Plug.auth_user.present? && Plug.auth_password.present?
-      http_basic_authenticate_with name: Plug.auth_user, password: Plug.auth_password
-    end
+    http_basic_authenticate_with name: Plug.auth_user, password: Plug.auth_password if Plug.auth_user.present? && Plug.auth_password.present?
   end
 end

data/app/controllers/plug/features_controller.rb

@@ -1,12 +1,10 @@
+# frozen_string_literal: true
+
 require_dependency 'plug/application_controller'
 
 module Plug
   class FeaturesController < ApplicationController
-    if Rails.version.to_i < 5
-      before_filter :set_feature, only: [:edit, :update, :destroy]
-    else
-      before_action :set_feature, only: [:edit, :update, :destroy]
-    end
+    before_action :set_feature, only: %i[edit update destroy]
 
     # GET /features
     def index
@@ -37,7 +35,7 @@ module Plug
 
     # PATCH/PUT /features/1
     def update
-      if @feature.update_attributes(feature_params)
+      if @feature.update(feature_params)
         redirect_to features_path, notice: 'Feature was successfully updated.'
       else
         render :edit
@@ -65,19 +63,15 @@ module Plug
     end
 
     private
-      # Use callbacks to share common setup or constraints between actions.
-      def set_feature
-        @feature = Feature.find(params[:id])
-      end
 
-      # Only allow a trusted parameter "white list" through.
-      # TODO: Strong params not available for older Rails
-      def feature_params
-        if Rails.version.to_i < 5
-          ActiveSupport::HashWithIndifferentAccess.new(params[:feature])
-        else
-          params.require(:feature).permit(:name, :description, :state, :notice)
-        end
-      end
+    # Use callbacks to share common setup or constraints between actions.
+    def set_feature
+      @feature = Feature.find(params[:id])
+    end
+
+    # Only allow a trusted parameter "white list" through.
+    def feature_params
+      params.require(:feature).permit(:name, :description, :state, :notice)
+    end
   end
 end

data/app/controllers/plug/site_notices_controller.rb

@@ -1,12 +1,10 @@
+# frozen_string_literal: true
+
 require_dependency 'plug/application_controller'
 
 module Plug
   class SiteNoticesController < ApplicationController
-    if Rails.version.to_i < 5
-      before_filter :set_site_notice, only: [:edit, :update, :destroy]
-    else
-      before_action :set_site_notice, only: [:edit, :update, :destroy]
-    end
+    before_action :set_site_notice, only: %i[edit update destroy]
 
     # GET /site_notices
     def index
@@ -37,7 +35,7 @@ module Plug
 
     # PATCH/PUT /site_notices/1
     def update
-      if @site_notice.update_attributes(site_notice_params)
+      if @site_notice.update(site_notice_params)
         redirect_to site_notices_path, notice: 'Site Notice was successfully updated.'
       else
         render :edit
@@ -51,19 +49,15 @@ module Plug
     end
 
     private
-      # Use callbacks to share common setup or constraints between actions.
-      def set_site_notice
-        @site_notice = SiteNotice.find(params[:id])
-      end
 
-      # Only allow a trusted parameter "white list" through.
-      # TODO: Strong params not available for older Rails
-      def site_notice_params
-        if Rails.version.to_i < 5
-          ActiveSupport::HashWithIndifferentAccess.new(params[:site_notice])
-        else
-          params.require(:site_notice).permit(:name, :notice, :state, :theme)
-        end
-      end
+    # Use callbacks to share common setup or constraints between actions.
+    def set_site_notice
+      @site_notice = SiteNotice.find(params[:id])
+    end
+
+    # Only allow a trusted parameter "white list" through.
+    def site_notice_params
+      params.require(:site_notice).permit(:name, :notice, :state, :theme)
+    end
   end
 end

data/app/helpers/plug/application_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   module ApplicationHelper
   end

data/app/mailers/plug/application_mailer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   class ApplicationMailer < ActionMailer::Base
     default from: 'from@example.com'

data/app/models/plug/application_record.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   class ApplicationRecord < ActiveRecord::Base
     self.abstract_class = true

data/app/models/plug/concerns/model_helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support/concern'
 
 module Plug
@@ -9,9 +11,9 @@ module Plug
         include AASM
 
         # Validations
-        validates :name, presence: { message: "#{self.humanized_class_name} name is required" },
-                         uniqueness: { message: "#{self.humanized_class_name} name must be unique", case_sensitive: false }
-        validates :state, presence: { message: "#{self.humanized_class_name} state is required" }
+        validates :name, presence: { message: "#{humanized_class_name} name is required" },
+                         uniqueness: { message: "#{humanized_class_name} name must be unique", case_sensitive: false }
+        validates :state, presence: { message: "#{humanized_class_name} state is required" }
 
         # Callbacks
         before_save { self.slug = name.parameterize }
@@ -32,12 +34,11 @@ module Plug
             transitions from: :enabled, to: :disabled
           end
         end
-
       end
 
       module ClassMethods
         def humanized_class_name
-          self.name.demodulize.underscore.humanize.capitalize
+          name.demodulize.underscore.humanize.capitalize
         end
       end
     end

data/app/models/plug/feature.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   class Feature < ApplicationRecord
     include AASM

data/app/models/plug/resources/site_notice.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   module Resources
     class SiteNotice < ActiveResource::Base

data/app/models/plug/site_notice.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
   class SiteNotice < ApplicationRecord
     include AASM

data/app/services/plug/task_execution_service.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake'
 
 module Plug

data/config/routes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Plug::Engine.routes.draw do
   root to: 'features#index'
 

data/db/migrate/20180424015828_add_theme_to_plug_site_notice.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class AddThemeToPlugSiteNotice < ActiveRecord::Migration[5.1]
   def change
     add_column :plug_site_notices, :theme, :string

data/lib/generators/plug/install_generator.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module Plug
   class InstallGenerator < Rails::Generators::Base
-    source_root File.expand_path('../templates', __FILE__)
+    source_root File.expand_path('templates', __dir__)
 
     def copy_initializer_file
       copy_file 'plug.rb', 'config/initializers/plug.rb'

data/lib/plug/configuration.rb

@@ -1,26 +1,28 @@
+# frozen_string_literal: true
+
 module Plug
   module Configuration
     AUTH_USER         = ''
     AUTH_PASSWORD     = ''
     ALLOW_DELETE      = true
 
-    VALID_OPTIONS_KEYS = [
-      :auth_user,
-      :auth_password,
-      :allow_delete,
-      :buttons,
-      :api_path,
-      :themes
+    VALID_OPTIONS_KEYS = %i[
+      auth_user
+      auth_password
+      allow_delete
+      buttons
+      api_path
+      themes
     ].freeze
 
-    attr_accessor *VALID_OPTIONS_KEYS
+    attr_accessor(*VALID_OPTIONS_KEYS)
 
     def configure
       yield self
     end
 
     def options
-      Hash[ * VALID_OPTIONS_KEYS.map { |key| [key, send(key)] }.flatten ]
+      Hash[* VALID_OPTIONS_KEYS.map { |key| [key, send(key)] }.flatten]
     end
   end
 end

data/lib/plug/constraint.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 module Plug
   class Constraint
     def initialize(feature)
       @feature = feature
     end
 
-    def matches?(request)
+    def matches?(_request)
       Plug.enabled?(@feature)
-    rescue
+    rescue StandardError
       true
     end
   end

data/lib/plug/engine.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 # Require engine dependencies
 require 'haml-rails'
 require 'aasm'
-require 'jquery-rails'
 
 module Plug
   class Engine < ::Rails::Engine
@@ -9,6 +10,8 @@ module Plug
 
     config.generators do |g|
       g.test_framework :rspec
+      g.fixture_replacement :factory_bot
+      g.factory_bot dir: 'spec/factories'
     end
   end
 end

data/lib/plug/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Plug
-  VERSION = '0.1.23'
+  VERSION = '0.1.25'
 end

data/lib/plug.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'plug/engine'
 require 'plug/configuration'
 require 'plug/constraint'
@@ -12,9 +14,9 @@ module Plug
   #
   # @return [Boolean] true - Feature found and enabled | true - Feature not found (We don't want to block) | false - Feature was set to disabled
   def enabled?(arg)
-    return get_feature(arg).enabled?
-  rescue
-    return true
+    get_feature(arg).enabled?
+  rescue StandardError
+    true
   end
 
   #
@@ -27,21 +29,21 @@ module Plug
     feature = get_feature(arg)
 
     render_notice(feature.notice, &block) unless feature.enabled? || feature.notice.blank?
-  rescue
-    return nil
+  rescue StandardError
+    nil
   end
 
   private
 
-    def get_feature(arg)
-      arg = arg.to_s if arg.is_a? Symbol
+  def get_feature(arg)
+    arg = arg.to_s if arg.is_a? Symbol
 
-      return Plug::Feature.slug_and_name(arg).first
-    end
+    return Plug::Feature.slug_and_name(arg).first
+  end
 
-    def render_notice(notice, &block)
-      return notice unless block_given?
+  def render_notice(notice, &block)
+    return notice unless block_given?
 
-      yield(notice)
-    end
+    yield(notice)
+  end
 end

data/lib/tasks/plug_tasks.rake

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # desc "Explaining what the task does"
 # task :plug do
 #   # Task goes here

metadata

@@ -1,18 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: plug
 version: !ruby/object:Gem::Version
-  version: 0.1.23
+  version: 0.1.25
 platform: ruby
 authors:
-- Ben
 - Boost
-autorequire: 
+- DNZ
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-16 00:00:00.000000000 Z
+date: 2023-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: aasm
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -26,7 +26,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sass-rails
+  name: activeresource
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -40,7 +40,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: jquery-rails
+  name: haml-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -54,21 +54,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: haml-rails
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.2'
 - !ruby/object:Gem::Dependency
-  name: aasm
+  name: sass-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -82,13 +82,13 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: activeresource
+  name: capybara
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -96,7 +96,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: factory_bot_rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,36 +109,22 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec-rails
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.7.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.7.1
 - !ruby/object:Gem::Dependency
   name: rails-controller-testing
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.2
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.2
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: factory_bot_rails
+  name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -152,7 +138,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: capybara
+  name: selenium-webdriver
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -166,7 +152,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: selenium-webdriver
+  name: shoulda-matchers
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -180,7 +166,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: shoulda-matchers
+  name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -193,9 +179,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Rails engine that can plug/unplug features with notice (WIP)
+description: Rails engine that can plug/unplug features with notice
 email:
-- benedict@boost.co.nz
 - info@boost.co.nz
 executables: []
 extensions: []
@@ -207,7 +192,6 @@ files:
 - app/assets/config/plug_manifest.js
 - app/assets/images/favicon.png
 - app/assets/javascripts/plug/application.js
-- app/assets/javascripts/plug/features.js
 - app/assets/javascripts/plug/trix-core.js
 - app/assets/stylesheets/plug/_settings.scss
 - app/assets/stylesheets/plug/application.scss
@@ -267,11 +251,11 @@ files:
 - lib/plug/engine.rb
 - lib/plug/version.rb
 - lib/tasks/plug_tasks.rake
-homepage: https://github.com/hapiben/plug
+homepage: https://github.com/digitalnz/plug
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -286,8 +270,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.32
+signing_key:
 specification_version: 4
 summary: Rails engine that can plug/unplug features
 test_files: []

data/app/assets/javascripts/plug/features.js

@@ -1,2 +0,0 @@
-// Place all the behaviors and hooks related to the matching controller here.
-// All this logic will automatically be available in application.js.