bullet_train-roles 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6c87ff4b45f3eed3dc5e3b2d7e6239160d4f83c3fee07483647b960f10e22ff1
-  data.tar.gz: 9302b17d61bbf57210ba56edacb75a5b4e4b00e4568ac67fe157fe9872cfcff5
+  metadata.gz: b04c87b6d26e9f326fc15cd10833fe874d9a2351d5a733d8c5fcff2210a10b7d
+  data.tar.gz: 7993de795a580ecac546e2697c79af02e7803f1530d3d9da51801b849ed53d37
 SHA512:
-  metadata.gz: f766e94d3a385fa76603f50d462e1606d9fc7038b17855fa5300f227a81160c79fc489ad201e457c8e35ee1a364d27617e8d8b839fd94071bb581dd0b7d91dce
-  data.tar.gz: cd8e636d008364f3e6169dd67e0839fbb362167ce4cf19fce135d35020a81143573072848f5062683c94fb696a4c69020c83e72f4339733d5e4d212c6cc1e184
+  metadata.gz: d8cd90ecb2da1e985dfe304fbbf287a7c4878415844888f359659deeca337271b18a793f79888897f10ee629d6f3986ffbf047dc65c0cf2d6776598771da345b
+  data.tar.gz: 21ae06c5cd83a62edca13832ad4a59330273b97022aff8b0ffdc7236b8f0dad238d459fb9610882f1511ee5e3559b790a2fc9156c8f024f97b00767971ef3ad1

data/Gemfile.lock

@@ -9,7 +9,7 @@ GIT
 PATH
   remote: .
   specs:
-    bullet_train-roles (0.1.0)
+    bullet_train-roles (0.1.1)
       active_hash
       activesupport
       cancancan
@@ -173,6 +173,7 @@ GEM
 
 PLATFORMS
   arm64-darwin-20
+  arm64-darwin-21
 
 DEPENDENCIES
   active_hash!
@@ -187,4 +188,4 @@ DEPENDENCIES
   standard (~> 1.5.0)
 
 BUNDLED WITH
-   2.2.15
+   2.3.4

data/README.md

@@ -1,6 +1,10 @@
 # Bullet Train Roles
 
-This Ruby Gem provides a Yaml-based configuration layer on top of CanCanCan's ability file. You can use this configuration file to simplify the definition of the most common types of permissions, while still implementing more complicated permissions in CanCanCan's traditional `app/model/ability.rb`.
+Bullet Train Roles provides a Yaml-based configuration layer on top of [CanCanCan](https://github.com/CanCanCommunity/cancancan). You can use this configuration file to simplify the definition of many common permissions, while still implementing more complicated permissions in CanCanCan's traditional `app/model/ability.rb`. 
+
+Additionally, Bullet Train Roles makes it trivial to assign the same roles and associated permissions at different levels in your application. For example, you can assign someone administrative privileges at a team level, or only at a project level.
+
+Bullet Train Roles was created by [Andrew Culver](http://twitter.com/andrewculver) and [Adam Pallozzi](https://twitter.com/adampallozzi).
 
 ## Example Domain Model 
 
@@ -17,12 +21,15 @@ You don't have to name your models the same thing in order to use this Ruby Gem,
 
 ## Installation
 
-Add this line to your application's Gemfile:
+Add these lines to your application's Gemfile:
 
 ```ruby
+gem "active_hash", github: "bullet-train-co/active_hash"
 gem "bullet_train-roles"
 ```
 
+> We have to link to a specific downstream version of ActiveHash temporarily while working to merge certain fixes and updates upstream.
+
 And then execute the following in your shell:
 
 ```
@@ -32,23 +39,26 @@ bundle install
 Finally, run the installation generator:
 
 ```
-rails generate bullet_train:roles:install User Membership Team
+rails generate bullet_train:roles:install
 ```
 
-This will:
+The installer defaults to installing a configuration for `Membership` and `Team`, but it will prompt you so you can specify different models if they differ in your application.
+
+The installer will:
 
  - stub out a configuration file in `config/models/roles.yml`.
  - create a database migration to add `role_ids:jsonb` to `Membership`.
  - add `include Role::Support` to `app/models/membership.rb`.
  - add a basic `permit` call in `app/models/ability.rb`.
 
+
 ### Limitations
 
 The generators currently assume you're using PostgreSQL and `jsonb` will be available when generating a `role_ids` column. If you're using MySQL, you can edit these migrations and use `json` instead, although you won't be able to set a default value and you'll need to take care of this in the model.
 
 ## Usage
 
-The provided `Role` model is backed (via [ActiveHash](https://github.com/active-hash/active_hash)) by a Yaml configuration in `config/models/roles.yml`.
+The provided `Role` model is backed by a Yaml configuration in `config/models/roles.yml`.
 
 To help explain this configuration and it's options, we'll provide the following hypothetical example:
 

data/lib/bullet_train/roles/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Roles
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/lib/bullet_train/roles.rb

@@ -5,6 +5,7 @@ require_relative "roles/version"
 require_relative "../models/role"
 require_relative "../roles/permit"
 require_relative "../roles/support"
+require_relative "../roles/user"
 
 module Roles
   class Error < StandardError; end

data/lib/generators/bullet_train/roles/install/install_generator.rb

@@ -31,6 +31,7 @@ module BulletTrain
         # TODO: follow back Andrew on question about this in Slack
 
         add_permit_to_ability_model(top_level_model, associated_model)
+        add_concern_to_user
       end
 
       private
@@ -160,22 +161,44 @@ module BulletTrain
       end
 
       def add_permit_to_ability_model(top_level_model, associated_model)
-        # TODO: need to make this more smart e.g.
-        # 1. should know what parameter is used for user e.g. def initialize(user)
-        # 2. what if code doesn't include "if user.present?", maybe we need to handle that, consult with Andrew
         file_location = "app/models/ability.rb"
-        line_to_match = "if user.present?"
-        content_to_add = "\n      permit user, through: :#{top_level_model.downcase.pluralize}, parent: :#{associated_model.downcase}\n"
-
-        puts("Adding 'permit user, through: :#{top_level_model.downcase}, parent: :#{associated_model.downcase}' to #{associated_model}\n\n")
+        line_to_match = "include CanCan::Ability"
+        content_to_add = "\n  include Roles::Permit\n"
+        puts("Adding 'include Roles::Permit' to #{associated_model}\n\n")
 
         if line_exists_in_file?(file_location, content_to_add)
           message = "#{remove_new_lines_and_spaces(content_to_add)} already exists in #{associated_model}!!\n\n"
+          line_already_exists(message)
+        else
+          add_in_file(file_location, line_to_match, content_to_add)
+        end
 
-          return line_already_exists(message)
+        line_to_match = "def initialize(user)"
+        content_to_add = "\n    permit user, through: :#{top_level_model.downcase.pluralize}, parent: :#{associated_model.downcase} if user.present?\n"
+        puts("Adding 'permit' to #{associated_model}\n\n")
+
+        if line_exists_in_file?(file_location, content_to_add)
+          message = "#{remove_new_lines_and_spaces(content_to_add)} already exists in #{associated_model}!!\n\n"
+          line_already_exists(message)
+        else
+          add_in_file(file_location, line_to_match, content_to_add)
         end
 
-        add_in_file(file_location, line_to_match, content_to_add)
+        puts("Success 🎉🎉\n\n")
+      end
+
+      def add_concern_to_user
+        file_location = "app/models/user.rb"
+        line_to_match = "class User < ApplicationRecord"
+        content_to_add = "\n  include Roles::User\n"
+        puts("Adding 'include Roles::User' to User\n\n")
+
+        if line_exists_in_file?(file_location, content_to_add)
+          message = "#{remove_new_lines_and_spaces(content_to_add)} already exists in User!!\n\n"
+          line_already_exists(message)
+        else
+          add_in_file(file_location, line_to_match, content_to_add)
+        end
 
         puts("Success 🎉🎉\n\n")
       end

data/lib/generators/bullet_train/roles/install/templates/roles.yml

@@ -1,10 +1,8 @@
 default:
   models:
-    -
 
 editor:
   models:
-    -
   manageable_roles:
     - editor
 

data/lib/models/role.rb

@@ -2,8 +2,6 @@
 
 require "active_hash"
 
-class RemovingLastTeamAdminException < RuntimeError; end
-
 class Role < ActiveYaml::Base
   include ActiveYaml::Aliases
   set_root_path "config/models"

data/lib/roles/user.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require "active_support"
+
+module Roles
+  module User
+    extend ActiveSupport::Concern
+
+    included do
+      def parent_ids_for(role, through, parent)
+        parent_id_column = "#{parent}_id"
+        key = "#{role.key}_#{through}_#{parent_id_column}s"
+        # TODO Maybe we should make ability caching a default feature of the gem?
+        # If we do that, we would just make it check whether `ability_cache` exists.
+        # return ability_cache[key] if ability_cache && ability_cache[key]
+        role = nil if role.default?
+        value = send(through).with_role(role).distinct.pluck(parent_id_column)
+        # TODO Maybe we should make ability caching a default feature of the gem?
+        # current_cache = ability_cache || {}
+        # current_cache[key] = value
+        # update_column :ability_cache, current_cache
+        value
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-roles
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Prabin Poudel
@@ -192,6 +192,7 @@ files:
 - lib/models/role.rb
 - lib/roles/permit.rb
 - lib/roles/support.rb
+- lib/roles/user.rb
 homepage: https://github.com/bullet-train-co/bullet_train-roles
 licenses:
 - MIT