1

I wanted to set one date to the first week of the year that i would pass to a function as argument. I mean if my date is 03/18/2015 and i wish to set this date to the first week of the year the result should be: 12/31/2014 (this is the first week of the first date [03/18/2015] ). This is the code im trying but when change the year of the date gives me one previous or the next week of the first week of year:

$actualDate = new DateTime("03/18/2015");
$actualDate = setFirstWeekOfYear($actualDate);

function setFirstWeekOfYear( $currentDate )
{
    // this variable will contain the current week number
    $currentWeek = $currentDate->format("W");

    // Get the current year (2015 in this moment)
    $currentYear = $currentDate->format("Y");

    // Rest the weeks number to the current date
    $currentDate = $currentDate->modify("-{$currentWeek} week");

    return $currentDate;
}

// 03/18/2017 => the output is 12/31/2016
// 03/18/2015 => the output should be 12/31/2014 but what i'm getting is 12/24/2014

Note: with the date 03/18/2017 works well but with 03/18/2015 is giving me one week previous to the first week of the year. I'm taking java Calendar.WEEK_OF_YEAR, 1 function as reference

Thanks in advance :)

martinezjc
  • 3,415
  • 3
  • 21
  • 29
  • That is the same day of the last week of the previous year which is what your code should produce. Now, do you want the last week of the year? Or last calender day of the year? – John Conde Mar 18 '15 at 19:58
  • i wanted the first week of the year, im trying to reproduce the same java code to php: in java you make: `Calendar workingCal = Calendar.getInstance(); System.out.println("Previous WEEK_OF_YEAR: " + workingCal.getTime()); workingCal.set(Calendar.WEEK_OF_YEAR, 1); System.out.println("WEEK_OF_YEAR: " + workingCal.getTime());`. The first output is the original date and the second is with the first week of year – martinezjc Mar 18 '15 at 20:01

2 Answers2

2

The DateTime class understands ISO week numbering, so you can do something like this:-

function getFirstWeekOfYear(\DateTime $date = null)
{
    if(!$date){
        $date = new \DateTime();
    }
    return (new \DateTime())->setISODate((int)$date->format('o'), 1, $date->format('w'));
}

You should note that the ISO 8601 definition for week 01 is the week with the year's first Thursday in it1.

A working example.

1: ISO week date
2: PHP date format strings

I am not a Java programmer, but a quick bit of googling suggests to me that Java does not use ISO week numbers, so may be giving you the wrong result. This question and answers may help you further.

Community
  • 1
  • 1
vascowhite
  • 18,120
  • 9
  • 61
  • 77
  • i know that the logic of the first week of the year is something wrong, but java does it with the calendar class :|, the example show the first week of the next year not the current one – martinezjc Mar 18 '15 at 21:31
  • Sorry, my mistake using copy pasted code from a project. See my edits and the corrected example. – vascowhite Mar 18 '15 at 21:47
  • When i replace the date for `2017-03-18` it returns `2017-01-07`, it should return `12/31/2016` instead – martinezjc Mar 18 '15 at 22:19
  • Why do you expect it to return that? That is Saturday of the last week of 2016, week 1 of 2017 starts on Sunday 1st Jan. – vascowhite Mar 18 '15 at 23:28
  • Once again, Java give me that result, i know logically this is wrong but the goal is make the output as same as Java does. This is the Java Code: `Calendar workingCal = Calendar.getInstance(); System.out.println("Previous WEEK_OF_YEAR: " + workingCal.getTime()); workingCal.set(Calendar.WEEK_OF_YEAR, 1); System.out.println("WEEK_OF_YEAR: " + workingCal.getTime());` – martinezjc Mar 19 '15 at 14:43
  • 1
    See my edit, sorry I can't help further, but my Java knowledge is minimal. – vascowhite Mar 19 '15 at 16:59
  • Thanks a lot, i have the php code that makes the magic i will post the answer to help someone else having this problem :) – martinezjc Mar 19 '15 at 17:00
0

By doing some tests on http://writecodeonline.com/php/ ... I found that if you did:

$currentWeek = $currentDate->format("W") - 1; // subtract 1

that would produce the 31st as the date ... I think the reasoning has to do with the fact that you don't want to include the current week of the $actualDate. EG: If March 18, 2015 = the 12th week ... subtract 11 weeks before the 12th week.

Try it out, maybe that'll work for you?

UPDATE (Revised)

Look at the comments for an explanation:

$actualDate = new DateTime("03/18/2016");
$actualDate = setFirstWeekOfYear($actualDate);

function setFirstWeekOfYear($currentDate)
{
  // Grab year of current date
  $currentYear = $currentDate->format("Y");

  // Make a new DateTime variable and set the date
  $date = new DateTime();
  $date->setISODate($currentYear, 1, -1);

  // This should give you the Monday of the
  // first week that the year starts on
  return $date;
}

// 03/19/2018 = Saturday, December 30, 2017
// 03/19/2017 = Saturday, December 31, 2016
// 03/19/2016 = Saturday, January 02, 2016
// 03/19/2015 = Saturday, December 27, 2014
// 03/19/2014 = Saturday, December 28, 2013
echo $actualDate->format("l, F d, Y");
Mr. Meeseeks
  • 1,841
  • 2
  • 21
  • 37