#210 ✓resolved
leiyou

Migration generator broken with stack overflow errors.

Reported by leiyou | August 5th, 2008 @ 02:10 PM

Hi guys,

After upgrading hobo and changing some of my models, I found that the migration_generator is no longer working with an stack overflow errors.

Had some dig into the problem, and the stack overflow occured around where we load the models from the app's model files.

This is most likely to do with the way alias_method_chain works.
If the model class were previously defined, the alias_method_chain would have ran once already, and therefore, you would be extending your method against something that has already been extended (and the extended methods may end up calling itself)

A temp patch is included without test.

Comments and changes to this ticket

  • Tom Locke

    Tom Locke August 6th, 2008 @ 11:06 AM

    • State changed from “new” to “resolved”

    Thanks for that fix. I tweaked it a little to cope with models in modules and added in

    e009e899afa64f236234cf2874228450e1521d90

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

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

People watching this ticket

Attachments

Pages