[wp-trac] [WordPress Trac] #15855: Dropdown isn't shown when doing a user 'removal'
WordPress Trac
wp-trac at lists.automattic.com
Sat Dec 25 08:02:53 UTC 2010
#15855: Dropdown isn't shown when doing a user 'removal'
--------------------------+------------------
Reporter: scribu | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.1
Component: Multisite | Version:
Severity: normal | Resolution:
Keywords: has-patch |
--------------------------+------------------
Comment (by westi):
Replying to [comment:18 SergeyBiryukov]:
> Replying to [comment:16 westi]:
> > A shared function for the shared code maybe but this doesn't sit well
in it's current state for me.
> How can it be improved?
Doing this {{{switch ( $action ) {}}} three times in the same function
makes it really hard to ensure that any changes made in the future are
done correctly.
I would make a shared function for the dropdown generation code and move
the rest of the code back into wp-admin/users.php in the two different
cases where your new function is called with the different actions.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/15855#comment:19>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list