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.

Stray backup file in Rails repo

#2030

Any particular reason why actionmailer/test/fixtures/test_mailer/implicitly_multipart_example.text.html.erb~ is in the Rails repo? Looks like an auto-generated backup file from Vim or some other editor.

Reported by Wincent Colaiuta · February 20th, 2009 @ 10:03 PM

State: committed
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. Wincent Colaiuta
    Wincent Colaiuta

    I see the file was added in this commit:

    http://github.com/rails/rails/co...

    February 20th, 2009 @ 10:19 PM

  2. DHH
    DHH
    • State changed from new to committed

    "Fixed problem causes by leftover backup templates ending in tilde [state:committed #969 Backup copies of email template ending in tilde cause error]" -- that's what the test is there for.

    February 27th, 2009 @ 02:02 PM

  3. Wincent Colaiuta
    Wincent Colaiuta
    • Tag set to patch

    Ok, I see what's happening now. The answer lies in "test_implicitly_multipart_messages" in actionmailer/test/mail_service_test.rb.

    Part of the problem is that there are three files that are in there which should be ignored. The first two:

    • implicitly_multipart_example.ignored.erb
    • implicitly_multipart_example.rhtml.bak

    Both contain the text "Ignored when searching for implicitly multipart parts."

    The third one, the one which confused me:

    • implicitly_multipart_example.text.html.erb~

    Doesn't contain that explanatory text. In fact, it contains the exact same contents as:

    • implicitly_multipart_example.text.html.erb

    So perhaps we could change it to be like the others?

    February 27th, 2009 @ 02:47 PM