first see if ti works without updated_by
On Sun, Sep 12, 2010 at 1:42 PM, nobosh <[email protected]> wrote:
> Quick update... I think what I've been doing wrong is not including
> the updated_by current_user in the right place... I now have:
>
> def update
> @book = Book.find(params[:id])
> respond_to do |format|
> if @book.update_attributes(params[:book].merge(:updated_by =>
> current_user))
> format.html { redirect_to(@book, :notice => 'Book was
> successfully updated.') }
> format.xml { head :ok }
> else
> format.html { render :action => "edit" }
> format.xml { render :xml => @book.errors, :status
> => :unprocessable_entity }
> end
> end
> end
>
> Which doesn't error HOWEVER it still isn't populating the right fields
> in the VERSIONS table. Any ideas anyone?
>
>
>
>
>
>
>
> On Sep 11, 9:06 pm, nobosh <[email protected]> wrote:
> > That's good. Seems to confirm that you can record the updated_by on
> > updates. The user's issue in the link was he wants a version for the
> > 1st record created which isn't something I'm worried about since
> > that's in the books table. So some how that user by the updated_by to
> > work and populate the user_id column in the version table.
> >
> > On Sep 11, 9:00 pm, radhames brito <[email protected]> wrote:
> >
> >
> >
> > >http://github.com/laserlemon/vestal_versions/issues#issue/25
> >
> > > On Sat, Sep 11, 2010 at 11:54 PM, radhames brito <[email protected]>
> wrote:
> > > > oh , ok, i think you can also edit the versions table by altering the
> > > > migration that way you can create an association, but im not done
> watching
> > > > the railscast
> >
> > > > On Sat, Sep 11, 2010 at 11:52 PM, nobosh <[email protected]>
> wrote:
> >
> > > >> Thanks radhames but I'm not trying to implement auditing.
> >
> > > >> I'm trying to create a wiki like system for books so I have
> > > >> versioning, which is what vestal_version does. The 'acts_as_audited'
> > > >> is more about creating an audit trail of all activity and isn't
> about
> > > >> seeing past versions, reverting etc...
> >
> > > >> I did find this one post that talks about what I'm trying to get
> > > >> working:
> > > >>http://github.com/laserlemon/vestal_versions/issues/unreads#issue/7
> >
> > > >> The thinking is now:
> >
> > > >> def update
> > > >> @book.update_attributes(params[:book].merge(:updated_by =>
> > > >> current_user))
> > > >> end
> >
> > > >> It isn't working yet but I think we're close :)!
> >
> > > >> On Sep 11, 8:48 pm, radhames brito <[email protected]> wrote:
> > > >> > there is another gem that can help you since it save the user that
> made
> > > >> the
> > > >> > change automaticly is called
> >
> > > >> > acts_as_audited
> >
> > > >> > i think it does exactly what you want.
> >
> > > >> >http://github.com/collectiveidea/acts_as_audited
> >
> > > >> > On Sat, Sep 11, 2010 at 11:34 PM, radhames brito <
> [email protected]>
> > > >> wrote:
> > > >> > > let me see the railscast to remember how to use it
> >
> > > >> > > On Sat, Sep 11, 2010 at 11:34 PM, radhames brito <
> [email protected]
> > > >> >wrote:
> >
> > > >> > >> @note.version is a number, is the version number, you are
> pulling the
> > > >> > >> wrong property
> >
> > > >> > >> On Sat, Sep 11, 2010 at 11:31 PM, nobosh <[email protected]>
> > > >> wrote:
> >
> > > >> > >>> If I update the code to:
> >
> > > >> > >>> @current_userid = current_user.id
> > > >> > >>> @note.version.user_id = @current_userid
> >
> > > >> > >>> Rails3 seems like that better, the error is now:
> > > >> > >>> undefined method `user_id=' for 1:Fixnum
> >
> > > >> > >>> On Sep 11, 8:25 pm, nobosh <[email protected]> wrote:
> > > >> > >>> > Rails error'd on that....
> >
> > > >> > >>> > Error "Called id for nil, which would mistakenly be 4 -- if
> you
> > > >> really
> > > >> > >>> > wanted the id of nil, use object_id"
> >
> > > >> > >>> > On Sep 11, 8:20 pm, radhames brito <[email protected]>
> wrote:
> >
> > > >> > >>> > > this does not work?
> > > >> > >>> > > @book.version.user_id = @current_user.id ?
> >
> > > >> > >>> > > On Sat, Sep 11, 2010 at 8:23 PM, nobosh <
> [email protected]>
> > > >> wrote:
> > > >> > >>> > > > Anyone using vestal_versions:
> > > >> > >>> > > >http://github.com/adamcooper/vestal_versions/
> >
> > > >> > >>> > > > How do you set the user_id in the versions table?
> >
> > > >> > >>> > > > tried:
> >
> > > >> > >>> > > > in:
> >
> > > >> > >>> > > > def update
> > > >> > >>> > > > @book.version.user_id(@current_user.id)
> > > >> > >>> > > > and
> > > >> > >>> > > > @book.user_id(@current_user.id)
> >
> > > >> > >>> > > > both error
> >
> > > >> > >>> > > > thanks
> >
> > > >> > >>> > > > --
> > > >> > >>> > > > You received this message because you are subscribed to
> the
> > > >> Google
> > > >> > >>> Groups
> > > >> > >>> > > > "Ruby on Rails: Talk" group.
> > > >> > >>> > > > To post to this group, send email to
> > > >> > >>> [email protected].
> > > >> > >>> > > > To unsubscribe from this group, send email to
> > > >> > >>> > > > [email protected]<rubyonrails-talk%[email protected]><rubyonrails-talk%2Bunsubscrib
> [email protected]><rubyonrails-talk%2Bunsubscrib
> > > >> [email protected]><rubyonrails-talk%2Bunsubscrib
> > > >> > >>> [email protected]>
> > > >> > >>> > > > .
> > > >> > >>> > > > For more options, visit this group at
> > > >> > >>> > > >http://groups.google.com/group/rubyonrails-talk?hl=en.
> >
> > > >> > >>> --
> > > >> > >>> You received this message because you are subscribed to the
> Google
> > > >> Groups
> > > >> > >>> "Ruby on Rails: Talk" group.
> > > >> > >>> To post to this group, send email to
> > > >> [email protected].
> > > >> > >>> To unsubscribe from this group, send email to
> > > >> > >>> [email protected]<rubyonrails-talk%[email protected]><rubyonrails-talk%2Bunsubscrib
> [email protected]><rubyonrails-talk%2Bunsubscrib
> > > >> [email protected]>
> > > >> > >>> .
> > > >> > >>> For more options, visit this group at
> > > >> > >>>http://groups.google.com/group/rubyonrails-talk?hl=en.
> >
> > > >> --
> > > >> You received this message because you are subscribed to the Google
> Groups
> > > >> "Ruby on Rails: Talk" group.
> > > >> To post to this group, send email to
> [email protected].
> > > >> To unsubscribe from this group, send email to
> > > >> [email protected]<rubyonrails-talk%[email protected]><rubyonrails-talk%2Bunsubscrib
> [email protected]>
> > > >> .
> > > >> For more options, visit this group at
> > > >>http://groups.google.com/group/rubyonrails-talk?hl=en.
>
> --
> You received this message because you are subscribed to the Google Groups
> "Ruby on Rails: Talk" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected]<rubyonrails-talk%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/rubyonrails-talk?hl=en.
>
>
--
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Talk" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/rubyonrails-talk?hl=en.
