Make WordPress Core

Opened 15 years ago

Closed 13 years ago

Last modified 13 years ago

#10308 closed defect (bug) (worksforme)

Adding two categories in a row creates an erroneous slug

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.8
Component: Taxonomy Keywords:
Focuses: Cc:

Description (last modified by Denis-de-Bernardy)

Browse Posts / Categories.

  1. Create a test1, a root category
  2. Without reloading the page, create test2, a child category of the newly created test1
  3. QuickEdit test2 - its slug is test1-test2 instead of test2

Change History (10)

#1 @Denis-de-Bernardy
15 years ago

  • Description modified (diff)

#2 @hakre
15 years ago

Can not reproduce in current trunk. for me the slug on quickedit is "test2" which seems valid to me.

BUT: To re-test, I deleted my first test1 and test2 category and then I recreated two new ones. For the second test2 I wanted to set the parent and what happened? The deleted categories still were in the dropdown list.

So this are my actual categories:

  • test1 — test2
  • test1
  • Uncategorized

And this is the dropdown:

  • None
  • Uncategorized
  • Test1
  • Test1 — test2
  • test1
  • test1 — test2

the italic ones are acutally non-existant.

#3 @Denis-de-Bernardy
15 years ago

Can not reproduce in current trunk.

yeah, had the same thought when the user reported it too. but there it was, a few tests later... :-)

#4 @Denis-de-Bernardy
15 years ago

btw, I *think* it's due to the fact that, when added using ajax, the slug would be calculated automatically based on the name. but since the name shows, for newly created sub cats, as test1 - test2, a slug of test1-test2 would seem right in this case.

#5 @hakre
15 years ago

Then a comparison of the javscript code with the php code is necessary. If the javascript not absolutely correctly mimics the behaviours of the php code, then I tend to say it is buggy and we have a bug here.

But that must be proven first.

#6 @janeforshort
15 years ago

  • Milestone changed from 2.9 to Future Release

Punting due to timing.

#7 @filosofo
14 years ago

  • Owner filosofo deleted
  • Status changed from new to assigned

#8 @kevinB
14 years ago

  • Cc kevinB added

#9 @solarissmoke
13 years ago

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

Cannot reproduce in trunk - even with the procedure in comment:2. Please reopen if it still is an issue.

#10 @scribu
13 years ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.