do not set flash.now with redirect action #63

Merged
merged 1 commit into from Apr 2, 2013

Conversation

Projects
None yet
2 participants
@nashby
Collaborator

nashby commented Apr 2, 2013

closes #61

lib/responders/flash_responder.rb
+ return true if @flash_now == true || format == :js
+
+ if has_errors?
+ @flash_now == :on_failure && default_action

This comment has been minimized.

Show comment Hide comment
@rafaelfranca

rafaelfranca Apr 2, 2013

Collaborator
default_action && (has_errors? ? @flash_now == :on_failure : @flash_now == :on_success)
@rafaelfranca

rafaelfranca Apr 2, 2013

Collaborator
default_action && (has_errors? ? @flash_now == :on_failure : @flash_now == :on_success)

This comment has been minimized.

Show comment Hide comment
@nashby

nashby Apr 2, 2013

Collaborator

👍

@nashby

nashby Apr 2, 2013

Collaborator

👍

rafaelfranca added a commit that referenced this pull request Apr 2, 2013

Merge pull request #63 from nashby/issue-61
do not set flash.now with redirect action

@rafaelfranca rafaelfranca merged commit 3368cb7 into plataformatec:master Apr 2, 2013

@nashby nashby deleted the nashby:issue-61 branch Apr 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment