Вычисление количества недель в году в соответствии с ISO 8601 #72
Conversation
|
Ну фикс есть фикс |
|
Напишите, пожалуйста, кто против того, чтоб влить, кто не против. |
|
Я бы попросил ещё пару тестов на крайние значения и пару рандомных каких-то, например, 96 и 83 годы. В общем, ещё пару тестов, кск и, думаю, стоит влить. |
|
Надо вливать народ :) |
|
кск и тесты, потом вливать. |
|
Добавьте, пожалуйста в тест 2009 год (в нем было 53 недели), 2010 (он соседний с годом, где 53 недели). |
|
Почистил пробелы, добавил тесты на рандомные года и на 2009 как просили. Тест оформил в виде отдельного метода. Какие-нибудь ещё пожелания будут? |
влить ;) |
|
Друзья, давайте вольем. @wizardjedi у тебя есть права на merge? Если нет, отпишись плз здесь, чтобы влил кто-нибудь другой. |
Вычисление количества недель в году в соответствии с ISO 8601
|
Замержил, сейчас в 1.0 добавлю. |
|
changelog бы еще ) |
|
Ах да. Вот чего я ждал и забыл. Не помню отчество из инициалов, что б в чейнджлог добавить. Выпытаю и чуть позже добавлю. |
|
обновил ChangeLog |
Привет, коллеги.
Заметили, что при попытке создания даты из номера недели в 2012 году вываливается exception от assert'а. Всему виной то, что по стандарту ISO 8601 (вызов date('W', ...)) номер недели рассчитывается из того на какое число выпадает первый четверг года. Например, в 2012 году 1 января выпадает на воскресенье и на 52 неделю 2011 года! А вот 31 декабря 2012 года попадает на понедельник и на 1 неделю 2013 года, отсюда и assert.
Предлагаю следующие изменения.
Буду рад услышать ваши комментарии и предложения. Тесты насколько я вижу не сломал.