[Bug 31726] New: JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

Bug #: 31726
           Summary: JScript Date.parse doesn't work
           Product: Wine
           Version: 1.5.13
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: minor
          Priority: P2
         Component: jscript
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

wine iexplore http://netikka.net/dev/dateparse.html

fixme:jscript:Date_value unimplemented flags 2
err:ole:ITypeInfo_fnInvoke failed to convert param 0 to VT_BSTR from
VT_DISPATCH

Native oleaut32 error is gone, but it still shows nothing. Works with native
jscript.

You can also test

wine iexplore http://netikka.net/dev/dateparse3.html

That shows incorrect value with builtin jscript

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #1 from Austin English  2012-09-17 17:33:45 CDT ---
Please attach a +jscript,+tid trace.

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #2 from [e-mail] 2012-09-18 02:06:51 CDT ---
Created attachment 41724
  --> http://bugs.winehq.org/attachment.cgi?id=41724
jscript, tid

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |[e-mail]
          Component|jscript                     |mshtml
     Ever Confirmed|0                           |1

--- Comment #3 from Jacek Caban  2012-09-18 03:20:17 CDT ---
It's MSHTML bug (it can't convert alert argument (which is Date object in this
case) to string. MSHTML should use jscript's IVariantChangeType interface for
that.

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #4 from [e-mail] 2012-09-18 06:09:00 CDT ---
After adding toString, it shows year 2065, there also bug in jscript?

wine iexplore http://netikka.net/dev/dateparse2.html

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|mshtml                      |jscript

--- Comment #5 from Jacek Caban  2012-09-19 04:24:36 CDT ---
Yeah, it seems like jscript bug. Let's track this one in this bug report.

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #6 from Austin English  2013-03-05 17:12:34 CST ---
Still in wine-1.5.25-69-g402b100

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #7 from [e-mail] 2013-09-09 10:44:08 CDT ---
Still in wine-1.7.1-158-g88c2a18

[Bug 31726] JScript Date.parse doesn't work

http://bugs.winehq.org/show_bug.cgi?id=31726

--- Comment #8 from [e-mail] ---
Both issues still in wine-1.7.20-82-geb4d39b

[Bug 31726] JScript Date.parse doesn't work

https://bugs.winehq.org/show_bug.cgi?id=31726

Teras  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #9 from Teras  ---
Still an issue 1.7.36

[Bug 31726] JScript Date.parse doesn't work

https://bugs.winehq.org/show_bug.cgi?id=31726

Thomas Faller  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #10 from Thomas Faller  ---
I wrote a patch for the parsing issue:
http://source.winehq.org/patches/data/110939
Maybe it will be committed.

[Bug 31726] JScript Date.parse doesn't work

https://bugs.winehq.org/show_bug.cgi?id=31726

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

--- Comment #11 from [e-mail] ---
Patch works!

(Second mshtml issue was fixed earlier)

[Bug 31726] JScript Date.parse doesn't work

https://bugs.winehq.org/show_bug.cgi?id=31726

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |fdd22f197ace226ceae8531c126
                   |                            |51feb192b7619
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #12 from [e-mail] ---
Thanks

[Bug 31726] JScript Date.parse doesn't work

https://bugs.winehq.org/show_bug.cgi?id=31726

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #13 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.43.