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.

Regexp selection of priority zones in time_zone_select

#479

Per Geoff Buesing's request in #195 Convenience methods for time zones outside the U.S., this patch implements =~ on ActiveSupport::TimeZone for selection of zones based on a match of name or TZInfo identifier in the MAPPING hash for easier localization. I had to flesh out the TimeZone mock a little more for testing, there may well be a better way to do this because this is my first Rails patch submission.

I'd really appreciate any input.

Reported by Ernie Miller · June 25th, 2008 @ 04:00 AM

State: invalid
Milestone: none
Assigned to: Geoff Buesing Geoff Buesing
Importance: none

Activity

  1. Jeremy Kemper
    Jeremy Kemper
    • State changed from new to invalid

    Hey Ernie, the patch looks good to me, but could you attach it to the original ticket? Let's keep work on this issue in one place. Thanks :)

    June 25th, 2008 @ 04:20 AM

  2. Ernie Miller
    Ernie Miller

    Sure. Sorry. Was going to, but instructions for submitting patches called for creating a ticket with the tag patch. Just following directions. ;D

    June 25th, 2008 @ 04:24 AM

  3. Ernie Miller
    Ernie Miller

    Actually, I'm just stupid. It said "Attach the patch to a lighthouse ticket and add the patch tag." Never said I needed to create the ticket. :) Anyway, updating now.

    June 25th, 2008 @ 04:28 AM