[#1007] target.new and target.build adds new empty element in collection...

[adoyen]

Hi,
As said in ticket 1007, this issue is not solved by using build. An empty element is still added to the collection when using a tag.

[bryanlarsen]

Arnaud, can you send me a sample app or test case that demonstrate's
the behaviour? Your patch causes one of the existing tests to fail,
so we need to find a different solution.

Bryan

On Wed, Jun 20, 2012 at 5:07 PM, Arnaud Doyen
[email protected]
wrote:

Hi,
As said in ticket 1007, this issue is not solved by using build. An empty element is still added to the collection when using a tag.

You can merge this Pull Request by running:

 git pull https://github.com/A-Kyma/hobo collection_empty_elem

Or you can view, comment on it, or merge it online at:

 #44

-- Commit Summary --

• [#1007] target.new and target.build adds new empty element in collection.

-- File Changes --

M hobo_rapid/taglibs/html/a.dryml (2)

-- Patch Links --

 https://github.com/tablatom/hobo/pull/44.patch
 https://github.com/tablatom/hobo/pull/44.diff

---

Reply to this email directly or view it on GitHub:
#44

[adoyen]

Hi Bryan,
I'll try to have an example application for that issue (if still exists).
Sorry for the delay, I'm not full time at developping hobo application (it's more like a hobby than a real job and I don't have always time for that in the evening)

I keep you informed when I reproduce the issue on a small app.

[al2o3cr]

There was a change in Rails 3 such that calling (for instance) @blog.posts.new was exactly identical to calling @blog.posts.build - that would explain why this didn't show up on Hobo 1.0.

Haven't tried it, but isn't a case like this exactly why we add the new_candidate method to association collections? - https://github.com/tablatom/hobo/blob/master/hobo/lib/hobo/extensions/active_record/associations/collection.rb#L13