Make WordPress Core

Opened 13 years ago

Last modified 9 months ago

#15801 assigned defect (bug)

Network Admin: Deactivated / Deleted inconsistency

Reported by: kawauso's profile kawauso Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.1
Component: Networks and Sites Keywords: dev-feedback needs-patch
Focuses: multisite Cc:

Description

Under the Sites screen, there are distinct inline links for Deactivate and Delete. However, when a site is deactivated, it is referred to as deleted in:

  • Sites screen inline status
  • Edit Site screen attributes section
  • Error page for non-admins visiting the site

Attachments (1)

deactivated.diff (629 bytes) - added by wonderboymusic 11 years ago.

Download all attachments as: .zip

Change History (38)

#1 @kawauso
13 years ago

  • Severity changed from minor to normal

Leaving severity alone

#2 @nacin
13 years ago

  • Component changed from UI to Network Admin
  • Milestone changed from Awaiting Review to 3.1

#3 @PeteMall
13 years ago

  • Owner set to PeteMall
  • Status changed from new to assigned

#4 @ryan_b
13 years ago

  • Cc ryan_b added

#5 @ryan_b
13 years ago

From what I can tell, the delete link removes the site from the db, deactivate leaves it in the db and sets a flag called "deleted", so my question is if the flag is called deleted is the status being listed as "Deleted" the desired effect?

#6 @ryan_b
13 years ago

  • Keywords dev-feedback added

#7 @ryan
13 years ago

  • Milestone changed from 3.1 to Future Release

#8 @scribu
13 years ago

Related: #17164

#9 @novasource
13 years ago

  • Cc aren@… added

#10 @wonderboymusic
11 years ago

  • Keywords has-patch added; dev-feedback removed
  • Milestone changed from Future Release to 3.5

Deactivated is the status for the Delete flag, Delete is an action that removes the site. Patched.

#11 @nacin
11 years ago

  • Milestone changed from 3.5 to Future Release

There's a lot of inconsistency here. Moving out of 3.5 for further review.

#12 @knutsp
11 years ago

  • Cc knut@… added

#13 @wonderboymusic
11 years ago

  • Milestone changed from Future Release to 3.6

#14 @DrewAPicture
11 years ago

  • Component changed from Network Admin to Text Changes

deactivated.diff looks good to me. It's a shame the flag is called 'deleted' but I'd assume changing it isn't an option.

#15 @wonderboymusic
11 years ago

  • Milestone changed from 3.6 to Future Release

#16 @wonderboymusic
11 years ago

  • Keywords easy-fix added
  • Milestone changed from Future Release to 3.7

#17 @martythornley
11 years ago

It seems there is still an inconsistency in that from a user's POV, they go through the process of saying to 'delete' their site, which is why it now shows up as 'deleted' but from the super admin POV it is really deactivated.

So... we could either change the "delete" link from the Super Admin level and rename it to "remove" or "permanently delete", then change the "deactivate" link back to "delete". This would make user and super admin "delete" functions act the same. "Delete" would mean temporary, like moving to the trash. While "Remove" or "Permanently Delete" would remove it altogether.

#18 @SergeyBiryukov
11 years ago

  • Keywords easy-fix removed

"easy-fix" is a tag for attracting new contributors to tickets that don't have a patch yet.

See the IRC discussion: part 1, part 2.

#19 @martythornley
11 years ago

I played with some of the wording in those links and some seem to work better than others.

It seems from the user level, they go through the delete site process and to them the site is gone. But really to the Super Admin it is "marked for deletion" which seems an even better label to me than "deactivated".

As a Super Admin, when we deactivate and reactivate a site, it is like trash/untrash or again, marking for possible deletion.

Once sites are "marked for deletion" or in the trash, we can use the "Delete" link to do it permanently.

Changing the wording of the links makes the super admin process make more sense, but it starts messing with the flags and would be inconsistent when looking at the code.

However, the get_blog_status and update_blog_status are only called a couple times in core and it would seem to not too difficult to run a quick update that says if the blog status is called 'deleted' update that to 'trash'. Or whatever the decided wording is. Or am I missing other areas that rely on those flags?

Another, farther reaching idea...

Could we somehow change the blog status to an array that can be whatever we want? That way a blog could be multiple things at once - A blog could be 'Mature', "Spam', and 'Trash" all at once? Or any number of custom status that a plugin might build like "Awesome", "Promoted", etc...

#20 @nacin
10 years ago

  • Component changed from Text Changes to Multisite
  • Milestone changed from 3.7 to 3.8
  • Owner changed from PeteMall to nacin

#21 @jeremyfelt
10 years ago

The behavior makes sense in that a user (not a superadmin) is flagging or marking their site for deletion, not deleting it directly. If the language to the user was updated to better convey this, that might be all that is needed.

The post delete screen of Your site will not be deleted until this link is clicked. could become Your site will not be flagged for deletion until this link is clicked.

An explanation about the network administrator having to complete the process could be added to the delete email.

The text for a non logged-in user *could* stay the same - This user has elected to delete their account and the content is no longer available. or could be changed to This site has been marked for deletion and the content is no longer available. It is the site being deleted, not necessarily the user's account.

And finally, the Deleted marker in the network admin's list table could be left alone or something else could be done to convey that further action is needed to satisfy the user's wishes.

@nacin - do you have thoughts on approach with this? Not sure if it really is something we can squeeze into 3.8.

#22 @nacin
10 years ago

  • Milestone changed from 3.8 to Future Release

@nacin - do you have thoughts on approach with this? Not sure if it really is something we can squeeze into 3.8.

Not really. Let's play a long game here. What's the best approach? Let's figure it out and do it, versus trying to squeeze things into a release over a confusing label.

#23 @jeremyfelt
10 years ago

  • Component changed from Multisite to Networks and Sites
  • Focuses multisite added

#24 @jeremyfelt
9 years ago

#29459 was marked as a duplicate.

#25 @jeremyfelt
9 years ago

  • Keywords needs-patch dev-feedback added; has-patch removed

#26 @jeremyfelt
9 years ago

  • Milestone changed from Future Release to 4.4

This ticket was mentioned in Slack in #core by sergey. View the logs.


9 years ago

#28 @SergeyBiryukov
9 years ago

  • Owner changed from nacin to jeremyfelt

#29 @jeremyfelt
8 years ago

  • Milestone changed from 4.4 to Future Release
  • Owner jeremyfelt deleted

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


7 years ago

#31 @SergeyBiryukov
5 years ago

#44974 was marked as a duplicate.

#32 @SergeyBiryukov
5 years ago

#47275 was marked as a duplicate.

#33 @sebastienserre
5 years ago

Hello,

This ticket seems to be abandoned... but I think there's a really problem with this "deleted label" io a "deactivated" one.

I'm quiet new in Core contribution so, I d'on't know exactly what to do to make this ticket closed :-/

This ticket was mentioned in PR #4476 on WordPress/wordpress-develop by @ideag.


9 months ago
#34

  • Keywords has-patch added; needs-patch removed

This PR changes some of the strings, to consistently refer to sites in multisite as 'Deactivated' when they are marked as such. Previously there were cases where they were being called as 'Deleted', while there is a separate action to 'Delete' a site which actually fully removes the site from the system.

EDIT: Trac ticket: https://core.trac.wordpress.org/ticket/15801

#35 @jorbin
9 months ago

#56076 was marked as a duplicate.

#36 @ideag
9 months ago

#44515 was marked as a duplicate.

#37 @jorbin
9 months ago

  • Keywords needs-patch added; has-patch removed

Looking at @jeremyfelt's comment, I think we should also fix up the other strings as a part of this so that everything is clear and we may need to add some additional code so that a site is considered flagged for deletion along with it being deactivated. That will help super admins know that a user requested the site be deleted vs that they deactivated it.

Note: See TracTickets for help on using tickets.