WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#10915 closed defect (bug) (fixed)

DRY: WP should use current_time() to calculate current time

Reported by: filosofo Owned by: ryan
Priority: normal Milestone: 3.0
Component: Date/Time Version:
Severity: normal Keywords: has-patch current_time
Cc: Focuses:

Description

Multiple places in the code calculate the current time, repeating what current_time('timestamp') already does.

Patch fixes.

Attachments (2)

use.current_time.10915.diff​ (3.2 KB ) - added by filosofo 17 years ago.
10915.patch​ (3.1 KB ) - added by hakre 17 years ago.

Download all attachments as: .zip

Change History (8)

@filosofo
17 years ago

#1 @ryan
17 years ago

  • Component General → Date/Time

#2 @ryan
17 years ago

  • Milestone 2.9 → 3.0

Related: #9588 #8662

#3 @Denis-de-Bernardy
17 years ago

  • Cc Denis-de-Bernardy added

@hakre
17 years ago

#4 @hakre
17 years ago

Patch still applied clean. Found a duplicate line in the review and therefore updated the patch.

#5 @ryan
17 years ago

See also #11672 and #11665.

#6 @automattor
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [12590]) Use current_time() to calculate current time. Props filosofo. fixes #10915

Note: See TracTickets for help on using tickets.