Make WordPress Core

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)

lostpassword.patch (1.0 KB ) - added by ktdreyer 16 years ago.
even better, with the negative mark in the right place
16498.diff (994 bytes ) - added by MikeHansenMe 14 years ago.
Removes duplicate from code.
16498.2.patch (1.4 KB ) - added by bpetty 14 years ago.
16498.3.patch (1.4 KB ) - added by SergeyBiryukov 14 years ago.

Download all attachments as: .zip

Change History (9)

@ktdreyer
16 years ago

even better, with the negative mark in the right place

#1 @markjaquith
16 years ago

  • Milestone Awaiting ReviewFuture Release

@MikeHansenMe
14 years ago

Removes duplicate from code.

@bpetty
14 years ago

#2 @bpetty
14 years ago

  • Milestone Future Release3.6
  • Priority normallow
  • 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 @SergeyBiryukov
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.

#4 @SergeyBiryukov
14 years ago

  • Keywords commit added

#5 @SergeyBiryukov
14 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status newclosed

In 23336:

Simplify the logic for printing "Lost your password?" link. props ktdreyer. fixes #16498.

Note: See TracTickets for help on using tickets.