#3464 closed defect (bug) (fixed)
Category delete AYS prompt incorrectly states outcome of category deletion
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.2 | Priority: | low |
Severity: | minor | Version: | 2.1 |
Component: | Administration | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
When deleting a category using AJAX, the AYS prompt tells the user that all the posts will go into the default post category and all links will go into the default link category. This is only the case if the deleted category was the only category for that post.
Not sure if it's worth making that distinction in the AYS or if there's even a good way to word such a thing.
"All posts that were only assigned to this category will be assigned to the "Uncategorized" category. All links that were only assigned to this category will be assigned to the "Blogroll" category" ?
Attachments (5)
Change History (18)
#1
@
19 years ago
Actually, look at the cat-delete-note.png attachment... there is also an HTML note that incorrectly states the outcome.
#2
@
19 years ago
- Owner changed from anonymous to postalservice14
- Status changed from new to assigned
#3
@
19 years ago
- Resolution set to fixed
- Status changed from assigned to closed
Let me know if you think the wording should be any different.
John
#4
@
19 years ago
- Keywords has-patch commit added
- Resolution fixed deleted
- Status changed from closed to reopened
Hi John,
Thanks for the patch, don't market tickets fixed though. Someone with commit access will when a fix is checked in.
Wording looks good.
#5
@
19 years ago
Instead, posts only in the deleted category are set to the category <strong>%s</strong> and links only set to this category are set to <strong>%s</strong>
I'm not 100% on this. How about:
Instead, posts that were only assigned to the deleted category are set to the category <strong>%s</strong> and links that were only assigned to the deleted category are set to <strong>%s</strong>
That makes the wording more consistent with the wording in the JS alert() portion of your patch, which I like.
#8
@
18 years ago
- Status changed from new to assigned
I'll work on a new patch incorporating MarkJaquith's fixes, as the reporter seems to be MIA.
screenshot of AYS