Make WordPress Core

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#30845 closed enhancement (fixed)

the number of weeks in a given year

Reported by: luciole135 Owned by: boonebgorges
Priority: normal Milestone: 4.4
Component: Date/Time Version: 4.1
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Firstly my apologies if I'm wrong place.
I propose a correction of the function validate_date_values ($date_query = array()) of the /wp-includes/date.php file
the following 341 to 343 lines:
$date = new DateTime();
$date-> setISODate ($date_query year, 53);
$week_count = $date-> format ('W') = '53'? 53: 52;

may be replaced by a single line (without using the class dateTime):
$week_count = date ('W', mktime (0,0,0,12,28,$date_query year));

Indeed, the number of weeks in a given year is equal to the corresponding week number of 28 December.
​http://en.Wikipedia.org/wiki/ISO_week_date

Attachments (2)

30845.patch​ (798 bytes ) - added by luciole135 12 years ago.
number of weeks in a given year
30845.2.patch​ (552 bytes ) - added by luciole135 11 years ago.
patch for WordPress 4.3

Download all attachments as: .zip

Change History (7)

#1 @luciole135
12 years ago

  • Type feature request → enhancement

@luciole135
12 years ago

number of weeks in a given year

#2 @luciole135
11 years ago

  • Keywords has-patch added

#3 @morganestes
11 years ago

#32835 was marked as a duplicate.

@luciole135
11 years ago

patch for WordPress 4.3

#4 @boonebgorges
11 years ago

  • Owner set to boonebgorges
  • Resolution → fixed
  • Status new → closed

In 33803:

Simplify the weeks-per-year calculation WP_Date_Query::validate_date_values().

Props luciole135.
Fixes #30845.

#5 @SergeyBiryukov
11 years ago

  • Milestone Awaiting Review → 4.4
Note: See TracTickets for help on using tickets.