Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

[PATCH] Prefer tap over returning in ActiveSupport

#3281

ActiveSupport's returning function has been replaced by object#tap whenever was possible.

I've attached a patch to start using tap in those areas of ActiveSupport that were still using returning.

Notice that there are two different patches, one for master and the other for 2-3-stable.

Reported by Matías Flores · September 27th, 2009 @ 10:14 PM

State: resolved
Milestone: none
Assigned to: nobody
Importance: none

Activity

  1. Gaius Centus Novus
    Gaius Centus Novus

    +1 and verified in 2.3 stable

    September 27th, 2009 @ 11:37 PM

  2. Gaius Centus Novus
    Gaius Centus Novus
    • Tag changed from activesupport, bugmash, patch to 2-3-stable, activesupport, bugmash, patch
    • Title changed from Prefer tap over returning in ActiveSupport to [PATCH] Prefer tap over returning in ActiveSupport

    September 28th, 2009 @ 12:00 AM

  3. Rizwan Reza
    Rizwan Reza
    • Tag changed from 2-3-stable, activesupport, bugmash, patch to 2-3-stable, activesupport, patch

    February 12th, 2010 @ 12:46 PM

  4. Rohit Arondekar
    Rohit Arondekar
    • State changed from new to stale
    • Importance changed from to

    The file this patch was provided for has changed considerably.

    Master: http://github.com/smtlaissezfaire/rails/blob/2d233ebdcf0de9e769da52...

    2-3-stable: http://github.com/smtlaissezfaire/rails/blob/2-3-stable/activesuppo...

    Closing ticket as stale. Matias, if this is still an issue please make a comment to reopen and I'll oblige.

    July 1st, 2010 @ 10:27 AM

  5. Ryan Bigg
    Ryan Bigg
    • Tag cleared.

    Automatic cleanup of spam.

    October 9th, 2010 @ 09:53 PM

  6. Ryan Bigg
    Ryan Bigg
    • State changed from stale to resolved

    October 14th, 2010 @ 01:55 AM