Question I have always had is whether to pass in current_user.  Should
the helper module know about the current user?
It is definitely easier to test if you pass in user so maybe that is a hint.

On 9/5/13, john irving <notifications@github.com> wrote:
>> @@ -0,0 +1,16 @@
>> += render :partial => "application/donate-tab"
>> +.container#contact_message
>> +  %section.first
>> +    .inner-container
>> +      .row-fluid
>> +        .span5#contact_form
>> +          %h1 Contact us
>> +          = simple_form_for @contact_message, :defaults => {:required =>
>> true}, :validate => true do |f|
>> +            = f.input :name, :input_html => { :value =>
>> current_user.try(:name) }, label: "Your name"
>> +            = f.input :email, :input_html => { :value =>
>> current_user.try(:email) }, label: "Your email address"
>
> like this line. this bit `:value => current_user.try(:email)` is tasty.
> @jonlemmon suggested pulling this out into a helper method.
> here's my guess at it but it might be wrong?
>
> ```ruby
> = f.input :name, :input_html => suggest_user_name, label: "Your name"
>
> def suggest_user_name
>    { :value => current_user.try(:name) }
> end
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/loomio/loomio/pull/1007/files#r6171123
>