WordPress.org
Get WordPress

Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 15 months ago

#3028 closed defect (bug) (fixed)

Comment Problems

Reported by: turk's profile turk Owned by: westi's profile westi
Milestone: 2.1 Priority: normal
Severity: normal Version: 2.1
Component: General Keywords: bg|has-patch bg|commit comment
Focuses: Cc:

Description

With the latest svn I'm getting these errors while commenting.

These errors return when one in not logged in:

Warning: preg_replace(): Compilation failed: unmatched parentheses at offset 48 in /home/virtual/site6/fst/var/www/html/wp-includes/functions.php on line 1015

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site6/fst/var/www/html/wp-includes/functions.php:1015) in /home/virtual/site6/fst/var/www/html/wp-comments-post.php on line 55

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site6/fst/var/www/html/wp-includes/functions.php:1015) in /home/virtual/site6/fst/var/www/html/wp-comments-post.php on line 56

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site6/fst/var/www/html/wp-includes/functions.php:1015) in /home/virtual/site6/fst/var/www/html/wp-comments-post.php on line 57

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site6/fst/var/www/html/wp-includes/functions.php:1015) in /home/virtual/site6/fst/var/www/html/wp-includes/pluggable.php on line 272

Then this error as a logged in user (Subscriber):

Warning: preg_replace(): Compilation failed: unmatched parentheses at offset 48 in /home/virtual/site6/fst/var/www/html/wp-includes/functions.php on line 1015

Warning: Cannot modify header information - headers already sent by (output started at /home/virtual/site6/fst/var/www/html/wp-includes/functions.php:1015) in /home/virtual/site6/fst/var/www/html/wp-includes/pluggable.php on line 272

No errors if comments made by an admin.

Attachments (1)

proxy_check.diff​ (636 bytes) - added by westi 19 years ago.
Fix the proxy check preg_replace to capture the whole IP

Download all attachments as: .zip

Change History (6)

@westi
19 years ago

Fix the proxy check preg_replace to capture the whole IP

#1 @westi
19 years ago

  • Keywords bg|has-patch added
  • Milestone set to 2.1
  • Owner changed from anonymous to westi
  • Status changed from new to assigned

Attached patch should fix this I believe.

We want the whole IP address captured

#2 @turk
19 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

#3 @westi
19 years ago

  • Keywords bg|commit added
  • Resolution fixed deleted
  • Status changed from closed to reopened

This can't be closed till ryan/matt check's it in!

#4 @ryan
19 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [4091]) Fix proxy check regex. Props westi. fixes #3028

#5 @smithrdz33ib
15 months ago

What could be causing the warning messages related to unmatched parentheses in the preg_replace function?

Note: See TracTickets for help on using tickets.