From 22cfa784d46ae178318dd14bb175300bd05b7e21 Mon Sep 17 00:00:00 2001 From: Nathan Van der Auwera Date: Thu, 25 Oct 2012 10:11:33 +0300 Subject: [PATCH] Clarified the :wrap_object documentation. It was not made clear enough that the callable needs to return the object. Fixes #103. --- README.markdown | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/README.markdown b/README.markdown index 94eb97a..3a798e6 100644 --- a/README.markdown +++ b/README.markdown @@ -279,13 +279,16 @@ link_to_add_association('add something', @form_obj, :comments, :wrap_object => P ``` Note that the `:wrap_object` expects an object that is _callable_, so any `Proc` will do. So you could as well use it to do some fancy extra initialisation (if needed). +But note you will have to return the (nested) object you want used. E.g. ``` -link_to_add_association('add something', @form_obj, :comments, :wrap_object => Proc.new {|comment| comment.name = current_user.name }) +link_to_add_association('add something', @form_obj, :comments, + :wrap_object => Proc.new { |comment| comment.name = current_user.name; comment }) ``` +> A cleaner option would be to call a function that performs this initialisation and returns `self` at the end. ### link_to_remove_association