crewd_policies 0.2.2 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6527ffcacef669ab324ad83777f7cd17fc1f9247
-  data.tar.gz: 54e20371399a543275ce840d166b7c023f6d57da
+  metadata.gz: da58fbf9df8d461a914137bab938033f4efee6a0
+  data.tar.gz: b553dbabc6208daa598cabb5b727e9bfec5080b1
 SHA512:
-  metadata.gz: c9615f3137be60929726f364fc737de22aed85b507e0917439f7f468e1987d16b251e9345e0e40aff8d471e3d3148536674e14e69be85cf694a83938bee1f8dd
-  data.tar.gz: 96234ccddb979b38d50bbe238eff59e0757f38d9a89883584ce0ffec12883ba155d5d0d5af081a5c04a590af4a57fbae7db34e5975e2ae1fafaa611581adc7b7
+  metadata.gz: 638f046880f1001e59df48d61dba5300d4fcdaf2a5d8e2c0d3a5d14c832519afd69ec939d2e66e2aafd0fefe808fe1cdfce4c2a742fbb3ca2ca3473133326c0b
+  data.tar.gz: 02aa2ac5d8dc3f9aaf3df456a0ed96a75ec37121cecb2713ac67ae2c0b4a6886b3c521a4b6e6caa7bf197c830ff1ea7526453e40b6b51c64285813927ffd4662

data/lib/crewd_policies/model.rb

@@ -21,9 +21,9 @@ module CrewdPolicies::Model
 				aRole.each {|r| allow(r,aAbilities.dup) }
 				return
 			end
-			raise "aRole must be a string or a symbol" unless aRole.is_a?(String) or aRole.is_a?(Symbol)
+			raise ::StandardExceptions::Http::InternalServerError.new "aRole must be a string or a symbol" unless aRole.is_a?(String) or aRole.is_a?(Symbol)
 			aRole = aRole.to_s
-			raise "aAbilities must be a Hash" unless aAbilities.is_a? Hash # eg. :write => [:name,:address]
+			raise ::StandardExceptions::Http::InternalServerError.new "aAbilities must be a Hash" unless aAbilities.is_a? Hash # eg. :write => [:name,:address]
 
 			role_rules = (self.roles_rules[aRole] ||= [])
 			conditions = {}
@@ -42,7 +42,7 @@ module CrewdPolicies::Model
 					if fields==true  # special "field" value to mean the record or class
 						rule[:allowed] = true
 					else
-						raise "create, destroy and index must have true as a value, not an array of fields" if a=='create' or a=='destroy' or a=='index'
+						raise ::StandardExceptions::Http::InternalServerError.new "create, destroy and index must have true as a value, not an array of fields" if a=='create' or a=='destroy' or a=='index'
 						rule[:fields] = fields
 					end
 				end

data/lib/crewd_policies/policy.rb

@@ -1,5 +1,6 @@
 require 'active_support/core_ext/class/attribute'
 require 'active_support/core_ext/string/inflections'
+require 'pundit'
 require 'standard_exceptions'
 
 module CrewdPolicies
@@ -141,7 +142,7 @@ module CrewdPolicies
 	  def inner_query_fields(aAbility)
 		  internal_server_error! "roles_rules not found on #{record_class.name}, make sure it has \"include CrewdPolicies::Model\"" unless ra = record_class.roles_rules rescue nil
 		  unauthorized! "identity not given" if !identity
-		  internal_server_error! "identity must implement has_role?" if !identity.responds_to? :has_role?
+		  internal_server_error! "identity must implement has_role?" if !identity.respond_to? :has_role?
 
 		  ability = coalesce_field_ability(aAbility)
 
@@ -209,7 +210,7 @@ module CrewdPolicies
 
 		# does the identity have this ability on the record/resource at all?
 		def inner_query_ability(aAbility)
-			raise "aAbility must be a string or a symbol" unless aAbility.is_a?(String) or aAbility.is_a?(Symbol)
+			internal_server_error! "aAbility must be a string or a symbol" unless aAbility.is_a?(String) or aAbility.is_a?(Symbol)
 			aAbility = aAbility.to_s
 
 			case aAbility
@@ -218,7 +219,7 @@ module CrewdPolicies
 				when 'create','destroy','index'
 					inner_query_resource(aAbility)
 				else
-					raise 'this ability is unknown'
+					internal_server_error! 'this ability is unknown'
 			end
 		end
 	end

data/lib/crewd_policies/version.rb

@@ -1,3 +1,3 @@
 module CrewdPolicies
-  VERSION = "0.2.2"
+  VERSION = "0.3.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: crewd_policies
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Gary McGhee