Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#824 closed defect (bug) (fixed)

Apostrophe in submitter name borks comment submission

Reported by: dnorman's profile dnorman Owned by: ryan's profile ryan
Milestone: Priority: normal
Severity: major Version: 1.5
Component: General Keywords:
Focuses: Cc:

Description

I've got an apostrophe in my name (D'Arcy), and on my weblog I've set up my user with that as the display name etc... I just went to add a comment to my weblog to respond to someone else's comment, and it looked like it was barfing pretty violently on the apostrophe. I took it out of my WP user (Changed firstname and display name to "DArcy"), and all was well... That may be coincidence, since I also removed the apostrophe from the comment submitter's name at one point.

I was able to comment successfully with the previous beta, but I just installed the latest gamma (1.5-gamma 2005-02-05) and now get an error.

Here's the error I got:

WordPress database error: [You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'Arcy' AND comment_author_email = 'dlnorman@…' and com]<br />
SELECT comment_approved FROM wp_comments WHERE comment_author = 'D'Arcy' AND comment_author_email = 'dlnorman@…' and comment_approved = '1'

WordPress database error: [You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'Arcy', 'dlnorman@…', 'http://www.darcynorman.net]<br />
INSERT INTO wp_comments (comment_post_ID, comment_author, comment_author_email, comment_author_url, comment_author_IP, comment_date, comment_date_gmt, comment_content, comment_approved, comment_agent, comment_type, user_id) VALUES ('547', 'D'Arcy', 'dlnorman@…', 'http://www.darcynorman.net', 'IP_ADDRESS_REMOVED', '2005-02-05 18:57:32', '2005-02-06 01:57:32', 'testing comment submission', '0', 'Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-us) AppleWebKit/125.5.6 (KHTML, like Gecko) Safari/125.12', , '2')

Change History (2)

#1 @dnorman
19 years ago

  • Patch set to No

#2 @ryan
19 years ago

  • fixed_in_version set to 1.5
  • Owner changed from anonymous to rboren
  • Resolution changed from 10 to 20
  • Status changed from new to closed

http://wordpress.org/pipermail/cvs/2005-February/001054.html

I think that will fix it for you. If not, reopen.

Note: See TracTickets for help on using tickets.