Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

to_text and Loofah::Helpers #75

Closed
morus opened this issue Jun 23, 2014 · 3 comments
Closed

to_text and Loofah::Helpers #75

morus opened this issue Jun 23, 2014 · 3 comments

Comments

@morus
Copy link

morus commented Jun 23, 2014

Hi,
is is possible that the removal to automatically load Loofah::Helpers
breaks to_text?
irb
require 'loofah'
Loofah.fragment("abc").to_text
NameError: uninitialized constant Loofah::Helpers

OTOH Loofah.fragment("abc").to_s works.

Or is to_text considered part of the ActionView helpers?
Feels a bit strange. And: the to_text method itself is there.

best
Morus

@agius
Copy link

agius commented Jun 26, 2014

+1: was using to_text, not working in my env

@morus
Copy link
Author

morus commented Jun 27, 2014

Hi,
in case that's unclear: you just have to explicitly
require 'loofah/helpers'
to make it work again.
But I do not consider that a fix, it's just a workaround.

best
Morus

@flavorjones
Copy link
Owner

Confirmed the issue, apologies for not responding more quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants