Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#14032 closed defect (bug) (fixed)

When site is "Deactivated" it says "Deleted"

Reported by: bigdawggi's profile bigdawggi Owned by: tmoorewp's profile tmoorewp
Milestone: 3.1 Priority: normal
Severity: minor Version: 3.0
Component: Multisite Keywords: has-patch
Focuses: Cc:


When you "deactivate" a site, the listing of sites on wp-admin/ms-sites.php displays the site as "deleted" as opposed to "inactive". This is slightly misleading, because there is a "Delete" link for the site which actually deletes the site from the database.

Attachments (3)

14032.diff (875 bytes) - added by wojtek.szkutnik 14 years ago.
14032-2.diff (2.5 KB) - added by tmoorewp 14 years ago.
14032-3.diff (629 bytes) - added by tmoorewp 14 years ago.

Download all attachments as: .zip

Change History (14)

#1 @bigdawggi
14 years ago

  • Cc matt@… added

#2 @nacin
14 years ago

  • Keywords needs-patch added; delete deactivate multi-site removed
  • Milestone changed from Unassigned to 3.1

#3 @wojtek.szkutnik
14 years ago

  • Cc wojtek.szkutnik@… added
  • Keywords has-patch needs-testing gsoc added; needs-patch removed

#4 @nacin
14 years ago

  • Keywords needs-refresh added; has-patch needs-testing gsoc removed
  • Milestone changed from Awaiting Triage to 3.1

Patch needs refresh.

14 years ago

#5 @tmoorewp
14 years ago

  • Cc tim@… added
  • Keywords has-patch needs-testing added; needs-refresh removed
  • Owner set to tmoorewp
  • Status changed from new to assigned

14032-2.diff should do it. The db increment needs to be upped so that the $wpdb->blogs table gets modified.

#6 @nacin
14 years ago

Deleted == deactivated, I thought. We're not adding more fields.

#7 @tmoorewp
14 years ago

Yeah, you're right. I didn't look at it the right way initially. I'll update the file tomorrow.

14 years ago

#8 @tmoorewp
14 years ago

Here's a more accurate patch I tested this morning. Simple wording change in one of the class files.

#9 @tmoorewp
14 years ago

  • Keywords tested added; needs-testing removed

#10 @tmoorewp
14 years ago

  • Keywords tested removed

#11 @markmcwilliams
14 years ago

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

Looks like this has since been fixed during another commit, probably (most likely anyway) relating to #14579 and all the work carried out on it! :)

Note: See TracTickets for help on using tickets.