3 comments
on commit 15b1b2b

Shouldn’t the check for preexisting alias_method_chain’d methods be performed by alias_method_chain itself? That would prevent infinite recursions in case the same alias_method_chain call is evaluated more than once (double require’s or load’s).

We had this in an earlier commit but it hosed migrations. Odds are it was simply mis-detecting whether the method existed and falling back on the super classes. If you want to take a look it’s in the logs here somewhere.