Moment: moment.utc("2016-07-01").year(2013).toString()을 호출하면 잘못된 날짜가 제공됩니다.

에 만든 2017년 10월 13일 · 4코멘트 · 출처: moment/moment

문제에 대한 설명 및 재현 단계:

날짜를 과거 날짜로 설정하고 연도를 아무거나 설정하면 출력 날짜가 월말이 되는 것처럼 보입니다.

당신이 호출하면 moment.utc("2016-07-01").year(2013).toString() 당신은 얻을 것이다 Wed Jul 31 2013 00:00:00 GMT+0000 . 7월 1일이 아니라 7월 31일임을 주목하십시오.

환경:

나는 이것을 Chrome 61과 Node 5.10.1에서 재현했습니다.

도움이 될 수 있는 기타 정보:

이것은 2016년의 날짜를 사용하는 경우에만 문제인 것 같습니다. moment.utc("2015-07-01").year(2013).toString() 를 시도하면 잘 작동합니다.

나는 이것이 순간 2.19.0+ 이전의 문제가 아님을 확인할 수 있습니다. 2.19.0 이상에서만 발생하는 문제입니다.

문제를 보고하는 경우 사용 중인 환경에서 다음 코드를 실행하고 출력을 포함하십시오.

console.log( (new Date()).toString())
console.log((new Date()).toLocaleString())
console.log( (new Date()).getTimezoneOffset())
console.log( navigator.userAgent)
console.log(moment.version)

Thu Oct 12 2017 23:10:36 GMT-0300 (ADT)
10/12/2017, 11:10:36 PM
180
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36
2.19.1

출처

stringbeans

👍2

가장 유용한 댓글

@HolgerFrank 이 문제를 해결하기 위해 변경 사항과 함께 PR을 제출하시겠습니까? 빨리 고쳐야 할 아주 치명적인 버그입니다..

편집: PR을 제출했는데 기다릴 수 없었습니다.

Kerumen 에 2017년 10월 19일

👍3

모든 4 댓글

버전 2.19.0에서 set$1() 함수가 변경되었습니다.

이 변경으로 인해 윤년 순간에 새해를 설정하면 항상 월의 마지막 날을 날짜로 설정합니다.

moment('2010-02-25').year(2017) // result is: 2017-02-28
moment('2010-10-15').year(2017) // result is: 2017-10-31

현재(잘못된) 구현

function set$1 (mom, unit, value) {
    if (mom.isValid() && !isNaN(value)) {
        if (unit === 'FullYear' && isLeapYear(mom.year())) {
            mom._d['set' + (mom._isUTC ? 'UTC' : '') + unit](value, mom.month(), daysInMonth(value, mom.month()));
        }
        else {
            mom._d['set' + (mom._isUTC ? 'UTC' : '') + unit](value);
        }
    }
}

특별한 경우는 윤년의 2월 29일에만 필요하므로 올바른 구현은 다음과 같아야 합니다.

function set$1 (mom, unit, value) {
    if (mom.isValid() && !isNaN(value)) {
        if (unit === 'FullYear' && isLeapYear(mom.year()) && mom.month() === 1 && mom.date() === 29) {
            mom._d['set' + (mom._isUTC ? 'UTC' : '') + unit](value, 1, daysInMonth(value, 1));
        }
        else {
            mom._d['set' + (mom._isUTC ? 'UTC' : '') + unit](value);
        }
    }
}

이것은 치명적인 오류이므로 가능한 한 빨리 수정해야 합니다.

HolgerFrank 에 2017년 10월 13일

👍1

죄송합니다. "이것이 1.19.0+ 이전의 문제가 아님을 확인할 수 있습니다. 1.19.0+의 문제일 뿐입니다."라고 썼지만 실제로는 2.19를 의미했습니다... 원래 게시물에서 이것을 업데이트했습니다.

stringbeans 에 2017년 10월 13일

@HolgerFrank 이 문제를 해결하기 위해 변경 사항과 함께 PR을 제출하시겠습니까? 빨리 고쳐야 할 아주 치명적인 버그입니다..

편집: PR을 제출했는데 기다릴 수 없었습니다.

Kerumen 에 2017년 10월 19일

👍3

PR을 위해 마감합니다.

maggiepint 에 2017년 10월 19일

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Moment: moment.utc("2016-07-01").year(2013).toString()을 호출하면 잘못된 날짜가 제공됩니다.

가장 유용한 댓글

모든 4 댓글

관련 문제