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.

Rendering "~" caches (2027) still broken

#2513

2.3.2 is still preferring temp files over exact matches. I saw your added tests so I ran them to see what I was missing and they fail.

I attached the output of

vendor/rails/actionpack $ rake test

Reported by ransom-briggs (at uiowa) · April 16th, 2009 @ 05:28 PM

State: stale
Milestone: 3.x
Assigned to: Yehuda Katz (wycats) Yehuda Katz (wycats)
Importance: none

Activity

  1. Sean Bryant
    Sean Bryant
    • Tag changed from action_view, bug to 2.3.2.1, action_view, bug, render, rendering

    I can confirm this as being a problem. It will want to use emacs backup files.

    April 20th, 2009 @ 09:16 PM

  2. ransom-briggs (at uiowa)
    ransom-briggs (at uiowa)

    This is related to how bzr handles revert backups and cleaning up temporary files. The test suite actually worked until I did a clean-tree --detritus which removed the test files. Although it still does not work with bzr backups

    bzr backups are in the form original_file.html.erb.~1~ which breaks the extension detection code. The extension is detected as erb so valid_extension? does not eliminate this file. I've attached a fix - basically it verifies that when the extensions split is done that it doesn't fall back to the second part if the third part is invalid.

    A breaking test is achieved by doing the following...

    cp ./test/fixtures/test/malformed/malformed.en.html.erb~ ./test/fixtures/test/malformed/malformed.en.html.erb.~1~

    April 22nd, 2009 @ 08:55 PM

  3. ransom-briggs (at uiowa)
  4. thedarkone
    thedarkone

    Guys, have you tried using the 2-3-stable branch instead of v2.3.2.1 tag?

    April 23rd, 2009 @ 08:14 AM

  5. ransom-briggs (at uiowa)
    ransom-briggs (at uiowa)
    • Tag changed from 2.3.2.1, action_view, bug, render, rendering to 2.3.2, 2.3.2.1, action_view, bug, render, rendering

    This is using the release of 2.3.2 as well - adding as a tag

    April 23rd, 2009 @ 02:19 PM

  6. josh
    josh
    • Assigned user changed from josh to Yehuda Katz (wycats)

    Lets make sure this problem doesn't exist on master.

    May 5th, 2009 @ 02:15 AM

  7. thedarkone
    thedarkone

    I think this is fixed in 2-3-stable and I'm pretty sure the current master's Path implementation is immune to this bug.

    May 5th, 2009 @ 08:21 AM

  8. ransom-briggs (at uiowa)
    ransom-briggs (at uiowa)

    This is broken in the gem version of 2.3.2 but is fixed in the current master on github (test passed and I glanced at the current implementation and it looks good - ie: it grabs the last extension explicitly using the regex rather than the second to last in the case when that is in the gem)

    May 5th, 2009 @ 05:21 PM

  9. ransom-briggs (at uiowa)
    ransom-briggs (at uiowa)

    it also appears to be fixed in origin/2-3-stable

    May 5th, 2009 @ 05:25 PM

  10. Jeremy Kemper
    Jeremy Kemper
    • Milestone changed from 2.x to 3.x

    May 4th, 2010 @ 06:48 PM

  11. Santiago Pastorino
    Santiago Pastorino
    • State changed from new to open
    • Tag changed from 2.3.2, 2.3.2.1, action_view, bug, render, rendering to 232, 2321, action_view, bug, render, rendering
    • Importance changed from to

    This issue has been automatically marked as stale because it has not been commented on for at least three months.

    The resources of the Rails core team are limited, and so we are asking for your help. If you can still reproduce this error on the 3-0-stable branch or on master, please reply with all of the information you have about it and add "[state:open]" to your comment. This will reopen the ticket for review. Likewise, if you feel that this is a very important feature for Rails to include, please reply with your explanation so we can consider it.

    Thank you for all your contributions, and we hope you will understand this step to focus our efforts where they are most helpful.

    February 2nd, 2011 @ 05:03 PM

  12. Santiago Pastorino
    Santiago Pastorino
    • State changed from open to stale

    February 2nd, 2011 @ 05:03 PM

  13. gfh