ensure reference to Rails is always prefixed with :: #880

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@kristianmandrup

Fixed problem with formtastic-bootstrap gem, in trying to make it work with formtastic 2.2.

@travisbot

This pull request fails (merged 3d1cb89 into 6378c45).

@justinfrench
Owner

@kristianmandrup hi, this diff looks okay, but I'm not sure why it's needed. can you go into a little more detail? why is the fix in formtastic instead of formtastic-bootstrap?

@kristianmandrup

At one point it complained that a constant FormtasticBootstrap::X::Y::Z::Rails didn't exist. I later discovered it was because Rails was not defined when running the specs. the FormtasticBootstrap spec_helper.rb simply needed to be changed in order to ensure that a Rails app is defined, much like you do in the formtastic equivalent :)

@justinfrench
Owner

I still don't understand why this fix/change is in formtastic instead of formtastic-bootstrap, since you keep talking about formtastic-bootstrap ;)

@justinfrench
Owner

Closing due to lack of activity, still not clear to me if it's a formtastic-bootstrap issue, or ours.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment