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.

find_last_by_* dynamic finder

#621

This patch allows the following:

  Account.find_last_by_limit(50)
  Account.find_first_by_limit(50)

Reported by Lawrence Pit · July 15th, 2008 @ 02:51 AM

State: wontfix
Milestone: 2.x
Assigned to: nobody
Importance: Low

Activity

  1. Tarmo Tänav
    Tarmo Tänav

    I think you forgot to remove the old implementation of determine_finder().

    I'm not sure if find_first_by is a good addition though because one would need to decide between using find_by and find_first_by and in this case a choice creates confusion (ie. do you use 'first_by' because it is more explicit and communicates the meaning of the code better of 'by' because it's shorter, easier to read/write).

    July 15th, 2008 @ 03:41 AM

  2. Lawrence Pit
    Lawrence Pit

    Thanks for the comment. Attached new patch.

    July 15th, 2008 @ 05:00 AM

  3. Pratik
    Pratik
    • State changed from new to wontfix

    I don't really think this is needed and will be useful. But please do continue the discussion in mailing list if you think it's really useful.

    Thanks.

    July 17th, 2008 @ 12:56 AM

  4. Ryan Bigg
    Ryan Bigg
    • Tag cleared.
    • Importance changed from to Low

    Automatic cleanup of spam.

    October 11th, 2010 @ 10:53 AM

  5. Jeff Kreeftmeijer