I’m having problems with the code below. The @photo.save seems to work (an entry is made in the database), but ‘Success!’ doesn’t appear on the "
tagging.rhtml" page unless I refresh the browser. It looks like some sort of caching issue, but, strangely, redirect_to doesn’t work no matter where I point it to. I’m not sure what to do…can someone tell me what’s going on? (and how to fix it?)
For the flash part you’ll need flash.now[:notice] = “Success!” since you haven’t redirected.
For the redirect_to, I suspect you should be having a double render/redirect issue since you’re not telling it to redirect_to :whatevs and return. Am I right?
> if @photo.save
> flash[:notice] = 'Success!'
> # redirect_to :action => 'list'
> # render :action => 'new'
> end
> end
If I just write flash.now[:notice] = "Success!" (no redirection),
nothing happens at all. If I write flash[:notice] (no .now), then
"Success!" will show on another page if refresh my browser or even
click to another page.
Redirecting continues not to work, even written without the render or
flash statements, so it doesn't seem to be a double render/redirect.
Just to be clear, the three lines after if @photo.save are the three
statements I've tried, both together and by themselves - none work!
It's very strange, because besides being rather elementary, they're
copied from another def in the same Controller.
I'm pretty much ready to scrap my project. I rewrote everything that
applies to tagging, including the Controller, but no luck. In
'addtags', I even placed *just* flash[:notice] and *just* redirect_to,
but troubles persisted.
If anyone is interested, before I take down the server, I'd be glad to
show off my application. Just email me and I'll send you the link.
I've had trouble with everything from screen refresh, to documented
bugs/workarounds, to image uploading, to gem installation...it's too
much. Ruby on Rails doesn't seem ready for prime time.
DON'T GIVE UP!!!! I can tell that you are a talented programmer because
you bit off a lot for a first project. Maybe a smaller first project
getting simple to things to work and then adding to it might have saved
a bit of frustration. But I've been there as well
I don't know if you saw Alan's response to your "flash" problem on the
other thread. You were missing the "=" in the <%= flash[:notice] %>
statement. Using the equal sign actually inserts the text in the output
html stream. So the way you had it coded it was being executed but
never displayed. Use <% .. some code .. %> when you simply need to wrap
Ruby code in your rhtml template but not actually output it.
I'm not sure what the redirect problem is but it's not related to the
flash message. Good luck going forward and don't quit.
Thank you for the encouragement. Last night, I took down the server
and then thought about how else I could create my photo tagging
application. The prospect of SQL statements everywhere terrified me,
so I reinstalled everything (this time not InstantRails) and rewrote
the entire application this afternoon changing the organization a
little. [I'm a graduate student on winter break (not comp sci), so I
have the time. :]
However, I STILL have the same redirect_to and flash[:notice] problem!
The commented out statements below all work (when they're uncommented,
obviously), but neither redirect_to() or flash[:notice] works together
or alone.
This is a major problem for this page, because once a tag is dropped on
a photo and the relationship is recorded in the database there needs to
be some sort of feedback (ie. 'Success!').
In response to previous suggestions, I have <%= flash[:notice] %> at
the top of my standard layout, so it's always there in every view.
Just in case, I added another <%= flash[:notice] %> in the tagging
view.
And, just as before, if I drop a tag and then click to another page or
refresh the browser, the flash[:notice] will be displayed! So strange!
However, I STILL have the same redirect_to and flash[:notice] problem!
The commented out statements below all work (when they're uncommented,
obviously), but neither redirect_to() or flash[:notice] works together
or alone.
I think the order of redirect_to follow by flash is wrong. It should be the reverse.
flash says set this message 'Great Success!' for display on the next page (redirect_to).
I think the order of redirect_to follow by flash is wrong. It should be the reverse.
flash says set this message 'Great Success!' for display on the next page (redirect_to).
Yes, I wanted to show the various statements I've tried (in combination
together and alone). The reality is even these most basic def's won't
work:
def addtag
redirect_to(:controller => 'photos', :action => 'list')
end
-- or --
def addtag
flash[:notice] = 'Great Success!'
end
Yet 'addtag' will execute other statements, such as those required to
make a database entry.
I don't really understand how to write breakpoints and test on Rails,
but maybe someone can help me there? [Very explicitly though, because
it's new to me.]
The problem is that the drag/drop support is provided via the prototype
and scriptaculous libraries. Therefore, the Ajax callback into your
photo/addtag method is expecting Javascript back. Your current
redirect, while it will work, is sending html back to the browser which
it is not expecting. That is why the flash message appears not to work
and also why the next page picks it up. Try the following:
render :update do |page|
page.redirect_to :action => 'tagging'
end
Yes! You're exactly right! I plugged your code into my def. Now the
page is automatically refreshed and 'Great Success!' is displayed. I'm
so glad I know what's going on!
From here, I'm going to have the function add the name of the dropped
person/tag to a list under each picture.
Thank you Paul - and everyone - for your help!
# The working code!
def addtag
@person_id = params[:id].split("_")[1]
@photo_id = params[:where]
@photo = Photo.find(@photo_id)
@photo.people << Person.find(@person_id)
if @photo.save
flash[:notice] = "Great success!"
render :update do |page|
page.redirect_to :action => 'list'
end
end
end
