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

sanitizeString is not defined #80

Closed
syphernl opened this issue Apr 26, 2012 · 4 comments
Closed

sanitizeString is not defined #80

syphernl opened this issue Apr 26, 2012 · 4 comments
Labels

Comments

@syphernl
Copy link

sanitizeString is not defined

Line 59 of inline document.

This breaks the login and probably a lot of other functionalities. I'm amazed to see how something like that could pass testing...

@nilsteampassnet
Copy link
Owner

Hum what version do you use? In what page? Can you provide me the complete error message?
Are you sure the cache of your server has been cleared?

sanitizeString function is of course an existing and defining function.
But nowhere in the project I can see it called at line 59 of a file.

I'm very concerned about this ... I do all the tests I can reasonably do ...

@syphernl
Copy link
Author

2.1.7, this error happens when you try to login. Was this function added recently?

@syphernl
Copy link
Author

syphernl commented May 2, 2012

it was indeed the cache. perhaps you can add a query string to all js files that contains the version so issues like these will not happen anymore whenever you make changes to js and CSS

@nilsteampassnet
Copy link
Owner

Ok good news!

Concerning your suggestion, seems feasible ;-)

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

No branches or pull requests

2 participants