Skip to content
This repository has been archived by the owner on Jan 8, 2018. It is now read-only.

$Date constructor arguments set to 1 by default, when arguments are not given #3

Closed
netil opened this issue Jan 2, 2015 · 1 comment
Closed

$Date constructor arguments set to 1 by default, when arguments are not given #3

netil opened this issue Jan 2, 2015 · 1 comment
Assignees
@netil
Labels
bug desktop dev-complete jindo.$Date mobile tc-complete
Milestone
2015/02 release

Comments

@netil
Copy link
Member

netil commented Jan 2, 2015

When instantiating Date object parameters are optional.
new Date(yr_num,mo_num,day_num[,hr_num,min_num,sec_num]);

When omit some parameters the date values are set by default, but in $Date object values are set 1 by default which not coinciding with native Date object value.

new Date(2010,6);  // Thu Jul 01 2010 00:00:00 GMT+0900
jindo.$Date(2010,6)._date;  // Thu Jul 01 2010 01:01:01 GMT+0900, the result should be as above
@netil netil self-assigned this Jan 2, 2015
@netil netil added this to the 2015/02 release milestone Jan 2, 2015
netil pushed a commit that referenced this issue Jan 2, 2015
<log>
b276873 
    <bts>https://github.com/naver/jindojs-jindo/issues/3</bts>
    <method>jindo.$Date</method>
    <coverage>desktop,mobile</coverage>
    <type>1</type>
    <level>2</level>
    <desc><![CDATA[
        // description
        jindo.$Date(2010,6)._date;

        // 2.12.0 and prior version
        // return 'Thu Jul 01 2010 01:01:01 GMT+0900'

        // later version
        // return 'Thu Jul 01 2010 00:00:00 GMT+0900'

    ]]></desc>
</log>
@netil
Copy link
Member Author

netil commented Mar 5, 2015

Released v2.12.1 on 5th March 2015.

@netil netil closed this as completed Mar 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@netil netil

Labels
bug desktop dev-complete jindo.$Date mobile tc-complete
Projects
None yet
Milestone
2015/02 release
Development

No branches or pull requests
1 participant
@netil