Make WordPress Core

Opened 7 years ago

Closed 6 years ago

#25875 closed defect (bug) (worksforme)

Issue with category named "$"

Reported by: MMDeveloper Owned by:
Milestone: Priority: normal
Severity: minor Version:
Component: Taxonomy Keywords: reporter-feedback
Focuses: Cc:

Description (last modified by johnbillion)

You are able to submit a category name with a $. When you assign that category, it crashes the non-admin portion of the site. It is possible to unselect the category, but I didn't know this was the reason and I lost about 3 hours trying the find out what the issue was. I think Wordpress should either remove the $ or save it as \$. But since it will create a category slug, I think it should be removed.

Issue on wp-includes/class-wp.php. Line:

if ( preg_match("#^$match#", $request_match, $matches) ||
	preg_match("#^$match#", urldecode($request_match), $matches) ) {

Change History (6)

#1 @nacin
7 years ago

  • Component changed from General to Taxonomy

#2 @johnbillion
7 years ago

  • Description modified (diff)
  • Keywords reporter-feedback added
  • Type changed from feature request to defect (bug)

I'm unable to reproduce a problem with a category named "$" on the front end. Can you explain what you mean by "When you assign that category, it crashes the non-admin portion of the site"? What exactly crashes? What errors do you see?

On a related note, it's not possible to create a category called "$" on the post editing screen if one doesn't already exist. It is possible to create it on the category management screen. The same applies to any category with a name that only contains characters that get stripped from the term slug. There may be an existing ticket for this.

#3 @ocean90
7 years ago

  • Summary changed from Category Names to Issue with category named "$"

#4 @wonderboymusic
7 years ago

  • Keywords needs-patch needs-unit-tests added
  • Milestone changed from Awaiting Review to Future Release

Needs a patch to move forward

#5 @collinsinternet
7 years ago

Like @johnbillion I'm not able to reproduce this problem. If you create a category with "$" the slug defaults to the category ID.

#6 @SergeyBiryukov
6 years ago

  • Keywords needs-patch needs-unit-tests removed
  • Milestone Future Release deleted
  • Resolution set to worksforme
  • Status changed from new to closed

No feedback in 10 months.

Feel free to reopen with more information if there's still a problem.

Note: See TracTickets for help on using tickets.