adequate_exposure 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9c0b98b45275a2969fdf0e5e3816d30681355c7f
-  data.tar.gz: 50b5e87966249713a1bf28dc21770996d1b0a4ec
+  metadata.gz: 5915dfaad2605bed7254feedffb99c2127dacfbc
+  data.tar.gz: 3f18159d981648e7bfb9f0856dcf08c43ebc419f
 SHA512:
-  metadata.gz: 93c096e3def6f624250401c00096b16fefeb8ca9fc11d13b1aa0c457db4b81178cb890f152b4ce4f9f922a535f6b72cd960b43b81fd4c36186f6f2dc9e6dd15b
-  data.tar.gz: 4a57bb49d5911ed55607e884d8561bb1b82ef009b60ccf54c5e427704e6ed3c60c9c85aabc2adfdf4438146987d5ffceea298ed397df4f602eec322bf32cd7d5
+  metadata.gz: 423058f319f76c5d7b6c05b93401b32f3febf8d435d265156206f8fe904dd95a59c1fdd35eb598d9f3ad7fc85bb715bb3b5e8913ec5c694fc6e86341ad16bc12
+  data.tar.gz: 8cda1d9cb3b2c6607d3947bb46150e4c8d50182bac4f2a67ce2842387ec5a61df5b255213f03314218aa56e30afb47cde6e4a036974d7c441efd29f687679ebe

data/README.md

@@ -34,6 +34,46 @@ ID and try to perform `Thing.find(id)`. If the ID isn't found, it'll call
 `Thing.new(things_params)`. The result will be memoized in an `@exposed_thing`
 instance variable.
 
+## Example Controller
+
+Here's what a standard Rails 4 CRUD controller using Adequate Exposure might look like:
+
+```ruby
+class ThingsController < ApplicationController
+  expose :things, ->{ Thing.all }
+  expose :thing
+
+  def create
+    if thing.save
+      redirect_to thing_path(thing)
+    else
+      render :new
+    end
+  end
+
+  def update
+    if thing.update(thing_params)
+      redirect_to thing_path(thing)
+    else
+      render :edit
+    end
+  end
+
+  def destroy
+    thing.destroy
+    redirect_to things_path
+  end
+
+  private
+
+  def thing_params
+    params.require(:thing).permit(:foo, :bar)
+  end
+end
+```
+
+## Under the Hood
+
 The default resolving workflow is pretty powerful and customizable. It could be
 expressed with the following pseudocode:
 
@@ -273,7 +313,6 @@ expose :thing, with: [:cool_find, :cool_build]
 expose :another_thing, with: :cool_build
 ```
 
-
 ## Contributing
 
 1. Fork it (https://github.com/rwz/adequate_exposure/fork)

data/lib/adequate_exposure/behavior.rb

@@ -1,7 +1,6 @@
 module AdequateExposure
   module Behavior
     def fetch
-      computed_scope = scope(model)
       instance = id ? find(id, computed_scope) : build(build_params, computed_scope)
       decorate(instance)
     end
@@ -52,5 +51,9 @@ module AdequateExposure
     def model_param_key
       model.name.underscore
     end
+
+    def computed_scope
+      scope(model)
+    end
   end
 end

data/lib/adequate_exposure/exposure.rb

@@ -74,7 +74,9 @@ module AdequateExposure
       exposure_name = options.fetch(:name)
 
       if from = options.delete(:from)
-        merge_lambda_option :fetch, ->{ send(from).send(exposure_name) }
+        merge_lambda_option :build, ->{ send(from).send(exposure_name) }
+        merge_lambda_option :model, ->{ nil }
+        merge_lambda_option :id, ->{ nil }
       end
     end
 
@@ -159,7 +161,7 @@ module AdequateExposure
     end
 
     def assert_singleton_option(name)
-      if options.except(name, :name).any? && options.key?(name)
+      if options.except(name, :name, :decorate).any? && options.key?(name)
         fail ArgumentError, "Using #{name.inspect} option with other options doesn't make sense"
       end
     end

data/lib/adequate_exposure/version.rb

@@ -1,3 +1,3 @@
 module AdequateExposure
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/spec/controller_spec.rb

@@ -348,10 +348,9 @@ describe AdequateExposure::Controller do
 
   context "from option" do
     it "allows scope to be called from method" do
-      post = double("Post")
       comments = double("Comments")
+      post = double("Post", comments: comments)
       allow(controller).to receive(:post).and_return(post)
-      expect(post).to receive(:comments).and_return(comments)
       expose :comments, from: :post
 
       expect(controller.comments).to eq(comments)
@@ -361,5 +360,15 @@ describe AdequateExposure::Controller do
       action = ->{ expose :thing, from: :foo, parent: :bar }
       expect(&action).to raise_error(ArgumentError, "Using :from option with other options doesn't make sense")
     end
+
+    it "should still work with decorate option" do
+      decorated_thing = double("DecoratedThing")
+      thing = double("Thing")
+      foo = double("Foo", thing: thing)
+      expect(controller).to receive(:foo).and_return(foo)
+      expect(controller).to receive(:decorate).with(thing).and_return(decorated_thing)
+      expose :thing, from: :foo, decorate: ->(thing){ decorate(thing) }
+      expect(controller.thing).to eq(decorated_thing)
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: adequate_exposure
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Pavel Pravosud
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-29 00:00:00.000000000 Z
+date: 2014-09-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties