[Bug 12325] New: winebrowser / mapi32. dll does not support accented characters in mailto:

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

Summary: winebrowser / mapi32.dll does not support accented
                    characters in mailto:
           Product: Wine
           Version: 0.9.58.
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: programs
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

When I try to execute:

wine C:\windows\system32\winebrowser.exe 'mailto:Tefnet
Developers?subject=TEST&body=ąóś'

my default mail client (opened by xdg-email) creates mail but without mail
body...

There is the same problem with e-mail generated by MAPISendMail (from
mapi32.dll)

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #1 from Tefnet developers   2008-04-02 06:39:33 ---
Created an attachment (id=11801)
 --> (http://bugs.winehq.org/attachment.cgi?id=11801)
Argument passed to xdg-email

Argument has a accented characters wrongly encoded

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #2 from Dmitry Timoshkov   2008-04-02 07:26:51 ---
What's your locale?

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #3 from Tefnet developers   2008-04-02 08:34:37 ---
(In reply to comment #2)
> What's your locale?

pl_PL.utf8

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #4 from Tefnet developers   2008-04-02 09:44:24 ---
(In reply to comment #1)
> Created an attachment (id=11801)

--> (http://bugs.winehq.org/attachment.cgi?id=11801) [details]
> Argument passed to xdg-email
>
> Argument has a accented characters wrongly encoded

it's cp1250, should be utf8

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #5 from Dmitry Timoshkov   2008-04-02 10:40:37 ---
Yes, that's a bug in winebrowser: winebrowser should be converted to unicode
and use CP_UNIXCP to convert strings before passing them to native apps.

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #6 from Tefnet developers   2008-04-03 06:29:22 ---
Created an attachment (id=11821)
 --> (http://bugs.winehq.org/attachment.cgi?id=11821)
winebrowser CP_UNIXCP support for "mailto:" URL

Here is a quick patch for that bug.
There is probably better way to FIX it.

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #7 from Dmitry Timoshkov   2008-04-03 07:31:03 ---
(In reply to comment #6)
> Created an attachment (id=11821)

--> (http://bugs.winehq.org/attachment.cgi?id=11821) [details]
> winebrowser CP_UNIXCP support for "mailto:" URL

Looks good, except there is no need check MultiByteToWideChar and
WideCharToMultiByte return values since url guaranteed to contain at least
"mailto:" and the APIs should never fail in that case.

Please send a corrected version of your patch to wine-patches, since it's
an improvement.
> Here is a quick patch for that bug.
> There is probably better way to FIX it.

As I said a better way is convert winebrowser to unicode, i.e. replace
main() by wmain().

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

Austin English  changed:

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

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

Hans Leidekker  changed:

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

--- Comment #8 from Hans Leidekker   2008-04-06 07:45:34 ---
I submitted a patch that converts winebrowser to Unicode.

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #9 from Hans Leidekker   2008-04-07 07:59:51 ---
Patch committed:
http://source.winehq.org/git/wine.git/?a=commit;h=e547cf7043e36fecfe5e23d1d6...

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

--- Comment #10 from Hans Leidekker   2008-04-18 09:07:30 ---
Reporter, can you confirm that this bug is fixed in git?

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

Tefnet developers  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #11 from Tefnet developers   2008-04-18 09:32:09 ---
Yes, it's fixed.

Thanks

[Bug 12325] winebrowser / mapi32. dll does not support accented characters in mailto:

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

Alexandre Julliard  changed:

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

--- Comment #12 from Alexandre Julliard   2008-04-18 11:04:28 ---
Closing bugs reported fixed in 0.9.60.

[Bug 12325] winebrowser / mapi32.dll does not support accented characters in mailto:

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Platform|All                         |Other
         OS/Version|All                         |other

--- Comment #13 from Austin English  2012-02-23 15:14:50 CST ---
Removing deprecated 'All' Platform/OS.