Opened 19 years ago
Closed 19 years ago
#5736 closed defect (bug) (fixed)
Merge profile.php and user-edit.php
| Reported by: | nbachiyski | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.5 |
| Component: | Administration | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
Inspired by #5508, this patch tries to merge wp-admin/profile.php and wp-admin/user-edit.php, since these files contain much common code.
Some details:
- Both files are retained in order all old links to work.
profile.phpis almost empty and just sets a variable and loadsuser-edit.php - Password strength meter and pluggable personal options are still available only on the profile page.
- The capability system was modified (with great help from Ryan) so that
current_user_can('edit_user', $uid)returns true if the current user has id$uid
Attachments (1)
Change History (11)
#2
@
19 years ago
This one function is presentation-oriented and isn't used anywhere else, so it would better stay there.
#6
@
19 years ago
- Resolution fixed
- Status closed → reopened
profile-update.php is no longer needed -- all the work is done in user-edit.php
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
That is a pretty sweet patch. I'll see about testing it over the weekend. If at all possible, can you move the function outside of the file into another file in the wp-admin/includes folder?
I think the goal is to have as much separate of presentation and business logic as possible. Just a suggestion, if you want to spend time doing it.