This project is archived and is in readonly mode.

#3816 ✓invalid
Max Hawkins

Code that depends on old MissingSourceFile breaks

Reported by Max Hawkins | January 30th, 2010 @ 03:31 AM

In Rails 2.x MissingSourceFile had the method signature

def initialize(message, path)

Under Rails 3 it extends LoadError which only takes one argument so calling

MissingSourceFile.new(message, path) throws an ArgumentError

This breaks the current version of Polyglot and probably other gems.

Comments and changes to this ticket

  • David Trasbo

    David Trasbo April 15th, 2010 @ 10:15 PM

    • Assigned user set to “Ryan Bigg”

    A major release means no backwards compatibility is promised. If you think this should be fixed anyway, feel free to provide a patch. Meanwhile this can be marked as invalid.

  • Ryan Bigg

    Ryan Bigg April 15th, 2010 @ 10:32 PM

    • State changed from “new” to “incomplete”

    Max, do you have a patch for this?

  • Rohit Arondekar

    Rohit Arondekar June 16th, 2010 @ 03:16 AM

    • State changed from “incomplete” to “invalid”

    I'm marking this as invalid for the time being, because as David said, major releases don't guarantee backward compatibility. Also Polyglot already works, and I'm sure other gems that are being actively maintained will start migrating.

    If you have a patch, or still feel the need to re-open this ticket, just make a comment to do so, and I'll oblige.

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

Pages