Manage -> Categories Ajax problems

[davidhouse]

I tried fixing this but gave up quickly. I think whoever wrote list-manipulation-js.php had a broken spacebar. :)

Anyway, there were two problems I could see:

• Adding a category which has a parent doesn't add that category to the list beneath its parent, it just adds it at the bottom
• Adding a category doesn't add it to the 'Category parent:' dropdown. In other words, you can't add a category, then add a second category which is a child of the first without refreshing the page.

Also, perhaps move the ajax-response dialog to above the form itself. I didn't realise it was there when using it! And it might be nice to say 'Successfully added, Jump ...' instead of just posting an anonymous link.

[mdawaffe]

The dropdown should be straightforward. That we can do for 2.1. See wp-admin/custom-fields.js for an example of listMan::addComplete (to abuse notation).

Currently, listMan can only add to the top or the bottom. Arbitrary insertion is doable: add a new XML tag in the response that specifies the insert position. That said, I'd leave this for 2.2. Perhaps in the interim, we can have the new category row show

parent_name - new_cat_name

instead of just

- new_cat_name

Thoughts?

[mdawaffe]

Huh. I do not remember hitting the milestone. Clearly I have a bad memory.

[mdawaffe]

First pass: 2803a.diff

1. Splits cat_rows() into cat_rows() and _cat_row() to eliminate code duplication.
2. In wp_insert_category(), make sure parent category exists.
3. When AJAX adding category, display name as "parent - cat" instead of "- cat" (the quick fix mentioned above to this larger problem).
4. When AJAX adding, add to dropdown.
5. When AJAX deleting, remove from dropdown.

[mdawaffe]

Works in Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.4) Gecko/20060508 Firefox/1.5.0.4

Please test in other browsers.

[mdawaffe]

Simpler. Fixes typo.

[mdawaffe]

2308b.diff

1. Simplifies.
2. Removes typo.

[ryan]

(In [4041]) Category Ajax improvements from mdawaffe. fixes #2803