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

Uncaught error when adding an answer #5

Closed
ilieandriuta opened this issue Mar 9, 2018 · 1 comment
Closed

Uncaught error when adding an answer #5

ilieandriuta opened this issue Mar 9, 2018 · 1 comment
Assignees
Milestone

Comments

@ilieandriuta
Copy link
Collaborator

@ilieandriuta ilieandriuta commented Mar 9, 2018

STEPS TO REPRODUCE

  1. Go to a forum
  2. Click on a topic
  3. Click 'Add answer' button and write an answer
  4. Click 'Add answer'
  5. Observe the console (from Dev tools)

EXPECTED RESULTS

The answer is added without any errors.

ACTUAL RESULTS

When the answer is added to the topic, the following error occurs in console:

Uncaught Error: Syntax error, unrecognized expression: #This is a good way to promote the company's values... .comment at Function.B.error (prototype.js:19) at n (prototype.js:19) at B.select (prototype.js:19) at B (prototype.js:19) at Object.a [as select] (prototype.js:19) at $$ (prototype.js:8) at e.loadIDs (JavascriptExtension?language=en&docVersion=1.1&defer=false&minify=false:54) at e.startup (JavascriptExtension?language=en&docVersion=1.1&defer=false&minify=false:34) at e.initialize (JavascriptExtension?language=en&docVersion=1.1&defer=false&minify=false:21) at new e (prototype.js:8)

Environment: Windows 10 Pro 64 bit, Chrome 65

@acotiuga
Copy link
Collaborator

@acotiuga acotiuga commented Jun 12, 2018

The fix for #8 also solve this issue. JavaScript errors caused by the bad answer ids are gone.

@acotiuga acotiuga closed this Jun 12, 2018
@acotiuga acotiuga self-assigned this Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants