Opened 16 years ago
Closed 14 years ago
#16498 closed enhancement (fixed)
simpler printing for "lostpassword" link
| Reported by: | ktdreyer | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | low | Milestone: | 3.6 |
| Component: | General | Version: | |
| Severity: | minor | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
There's no need to have two lines that print the same "lostpassword" link in in wp-login.php. Skip the "else" and just print it.
Attachments (4)
Change History (9)
#2
@
14 years ago
- Milestone Future Release → 3.6
- Priority normal → low
- Type defect (bug) → enhancement
16498.diff is actually incorrect since there are actually 3 conditions here, not two. See 16498.2.patch, where I've made this code a little more clear.
#3
@
14 years ago
The "Intentionally empty" section (as labeled in 16498.2.patch) was introduced in [6612]. We can rewrite the condition so that there's no need for it.
16498.3.patch is the refreshed lostpassword.patch, which I guess simplifies the logic in the most obvious way.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
even better, with the negative mark in the right place