Darragh Curran

Nice one for getting around to this. I checked out 2-3-stable. I get one failing test 'test_absolute_pathed_layout_without_layouts_in_path' before and after app...

The error message incorrectly (and confusingly) says:

... content_url has the following required parameters....

when it should be

... my_particular_named_route_...

No problem.

For me, the value for helper :all was on a reasonably small app, with a few html emails. Some of my mailers were adding nearly all the helpers. I ha...

Sometimes I do things in my layout based on whether content has been yielded to a particular content_for label.

Easy to do at the moment - but I reckoned it mig...

Yep Mike's suggestion works - as does what I do currently

class ActionMailer::Base def perform_deliveries

@mail.header['to'].to_s =~ /@example.com$/

end end

...

Here's a patch so that ActionMailer::Base.perform_deliveries will accept a proc -

currently it expects a boolean -

I wanted to easily perform deliveries conditi...

in controllers you can do helper :all and it will include all helpers in /app/helpers

but currently mailers don't allow the :all option (actually it will just l...

If this is desired behavior - then it should at least be made clear to the user.

It would be better to just abort startup than startup in broken state with no...

What you think about change_table?

change_table :users do |t|

t.references :addressable, :polymorphic => true

end

calling add column with expectation of ha...

I'm rewriting some old code that generated atom feeds. I want to preserve the id's of all existing entries - aswell as the current id of the feed. I want to av...