hobo_create_for passes a bogus default_action to create_response
Reported by Matt Jones | August 19th, 2009 @ 10:35 PM | in Hobo 1.0 - Final
The default action passed to create_response in hobo_create_for is incorrect - the code presently uses:
create_response(:"new_for_#{owner}", &b)
where owner is set to the parent object. This causes some bizarre error messages in functional tests, as owner.to_s typically is equivalent to owner.name, which is wrong.
The actions work on a running site, as page_path is getting updated, but it isn't (by default) set in functional tests.
Resolving this will inevitably tie into #477 and friends.
Comments and changes to this ticket
-
Bryan Larsen August 28th, 2009 @ 02:56 PM
- Tag changed from “auto_actions_for, bug” to “auto_actions_for, defect”
- State changed from “new” to “open”
-
Tom Locke November 26th, 2009 @ 03:57 PM
- State changed from “open” to “resolved”
(from [2ce25a7ecc7f4c6eee4a1a56ab593a3aa2ee3025]) [#491 state:resolved] pass a correctly inflected association name to create_response in hobo_create_for http://github.com/tablatom/hobo/commit/2ce25a7ecc7f4c6eee4a1a56ab59...
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
People watching this ticket
Bryan Larsen
Matt Jones
Tags
Referenced by
-
491
hobo_create_for passes a bogus default_action to create_response
(from [2ce25a7ecc7f4c6eee4a1a56ab593a3aa2ee3025])
[#491
s...