[Bug 11690] New: fixme:htmlhelp: HtmlHelpW Not all HH cases handled correctly

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

Summary: fixme:htmlhelp:HtmlHelpW Not all HH cases handled
                    correctly
           Product: Wine
           Version: 0.9.55.
          Platform: PC
               URL: http://teach2000.memtrain.com
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

The application Teach2000 has a help file in Windows, but not in Suse Linux.

Press help button on toolbar:
fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

Press F1:
Nothing happens

Invoke Help from menu [Help][Help]:
err:mmio:MMIO_ParseExtA No . in szFileName: "MenuPopup"
fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
err:mmio:MMIO_ParseExtA No . in szFileName: "MenuCommand"

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #1 from Austin English   2008-02-22 16:49:17 ---
(In reply to comment #0)
> The application Teach2000 has a help file in Windows, but not in Suse Linux.
>
> Press help button on toolbar:
> fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
>
> Press F1:
> Nothing happens
>
> Invoke Help from menu [Help][Help]:
> err:mmio:MMIO_ParseExtA No . in szFileName: "MenuPopup"
> fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
> err:mmio:MMIO_ParseExtA No . in szFileName: "MenuCommand"

Can you see if running:
$ wine iexplore.exe http://www.winehq.org
And installing wine gecko helps?

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

James Hawkins  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |hhctrl.ocx

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

bas  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|0.9.55.                     |0.9.56.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #2 from bas   2008-02-23 01:41:27 ---
> > $ wine iexplore.exe http://www.winehq.org

With this command wine Internet Explorer starts, and asks me to Gecko (Mozilla
HTML Engine).
I answer OK and I install Gecko.

But this doesn't help for the problem with Teach.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

bas  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, FIXME

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

Austin English  changed:

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

--- Comment #3 from Austin English   2008-02-23 16:56:21 ---
Confirming in wine 0.9.56. I'll attach a +mmio log.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #4 from Austin English   2008-02-23 16:56:42 ---
Created an attachment (id=10926)
 --> (http://bugs.winehq.org/attachment.cgi?id=10926)
+mmio log

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #5 from Louis Lenders   2008-03-09 16:28:37 ---
Created an attachment (id=11280)
 --> (http://bugs.winehq.org/attachment.cgi?id=11280)
screenshot

Hi, keep an I on bug #8246. The patch
http://bugs.winehq.org/attachment.cgi?id=11275 resolves this bug completely,
see screenshot. Help seems to work perfectly fine with that patch. Regards

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #6 from Timo-Heikki Mäkelä   2008-03-09 18:57:26 ---
Exactly the same as here happens in Bug 6801, just another app.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

Hin-Tak Leung  changed:

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

--- Comment #7 from Hin-Tak Leung   2008-03-10 16:46:06 ---
There is a tiny update to patch 11275 in:

http://bugs.winehq.org/attachment.cgi?id=11306

Somebody want to resolve as duplicate?

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

Lionel Debroux  changed:

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

--- Comment #8 from Lionel Debroux   2008-03-31 03:08:08 ---
AJ committed your fix on 2008/03/27, and the problem is solved for me as well.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #9 from Lionel Debroux   2008-03-31 03:28:41 ---
Created an attachment (id=11760)
 --> (http://bugs.winehq.org/attachment.cgi?id=11760)
Screenshot after pressing F1

Actually, it's not fully solved:
* if I launch the help button on the toolbar, the root page of the help is
displayed.
* if I press F1, the root page of the help is not displayed, as shows the
attached screenshot. But clicking on any of the top-level help categories in
the left pane brings a correctly displayed HTML page (and triggers a flurry of
fixmes in the process).

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #10 from Hin-Tak Leung   2008-03-31 10:53:11 ---
I am feeling brave... can you try doing

WINEDEBUG=+htmlhelp wine ... >& your.log

and attach your log? I don't have teach2000...

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #11 from scguy318   2008-04-04 03:01:03 ---
Created an attachment (id=11839)
 --> (http://bugs.winehq.org/attachment.cgi?id=11839)
Teach2000 log with htmlhelp traces

The log I have attached shows the bad behavior when pressing F1. I did not
press the help button on the toolbar, so there are no logs for that.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #12 from Hin-Tak Leung   2008-04-06 20:23:32 ---
Can I see how the log looks like

WINEDEBUG=+htmlhelp wine ... >& your.log

when you are getting correct help?

In the last log, I think

trace:htmlhelp:FindContextAlias returning "html\00201_input_basic.htm"
trace:htmlhelp:navigate_url L"html\00201_input_basic.htm"

should be doing

trace:htmlhelp:navigate_url
L"mk:@MSITStore....chm::html\00201_input_basic.htm"

instead.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #13 from Hin-Tak Leung   2008-04-06 21:36:36 ---
Can you try deleting these three lines

hres = navigate_url(info, surl);
   if(SUCCEEDED(hres))
      return TRUE;

around line 92-94, within NavigateToUrl() in file "dlls/htctrl.ocx/help.c"
to see if F1 works?

NavigateToUrl() is only used by the CONTEXT branch using results from
FindContextAlias(). I don't know if FindContextAlias() ever returns a fully 
formed url, but in this case it obviously doesn't, so the code should drop
through to the part after these three lines trying to find the url within
the chm file.

I think those three lines should either be deleted, or at least spanned by
a check, something like this:

static const WCHAR url_indicator[] = {':', ':', '/'}; 
  if (strstrW(surl, url_indicator)) {
  ...
  }

because navigate_url always succeeds even if url doesn't exist. (the return
value of nagivate_url() is whether *interaction* with the web browser component
succeeds, not about whether the url actually exists, I think?)

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #14 from scguy318   2008-04-06 22:27:24 ---
Created an attachment (id=11899)
 --> (http://bugs.winehq.org/attachment.cgi?id=11899)
Teach2000 log with correct HTML help behavior

This is the log when pressing the Help toolbar button. I will try recompiling
my sources with your change to see whether that helps the issues.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

scguy318  changed:

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

--- Comment #15 from scguy318   2008-04-06 23:13:58 ---
Your suggested modification does the trick. Pressing F1 results in appropriate
behavior.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #16 from Hin-Tak Leung   2008-04-07 08:13:40 ---
That's great. I'll e-mail wine-devel and a few others who were known to have
touched that part of the code and see what they think. git blame is great
for looking up who has last worked on which part of which.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #17 from Hin-Tak Leung   2008-04-07 13:06:48 ---
Created an attachment (id=11924)
 --> (http://bugs.winehq.org/attachment.cgi?id=11924)
proposed patch

proposed patch to match for "://" for url.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #18 from Lionel Debroux   2008-04-08 02:02:13 ---
Your patch works for me as well.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #19 from Hin-Tak Leung   2008-04-08 19:13:12 ---
The first application in bug 10166 is identical to this. The one in bug 10166
comment 2 is a different problem.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #20 from Dirk Mueller   2008-04-10 01:23:46 ---
Created an attachment (id=12034)
 --> (http://bugs.winehq.org/attachment.cgi?id=12034)
As.chm log with htmlhelp traces

I have a similar problem to this.

Fix 11306 is not resolving the problem for me.
Please see attached log for following call:

WINEDEBUG=+htmlhelp wine hh As.chm >& chm.log

I am using wine 0.9.56 patched with 11306

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #21 from Dirk Mueller   2008-04-10 01:31:04 ---
Created an attachment (id=12035)
 --> (http://bugs.winehq.org/attachment.cgi?id=12035)
screenshot windows help file wpa.chm

Viewing a windows help file copied from a windows installation (wpa.chm or
others) is also not working as expected.

The content of the help file is not displayed as you can see on the screen
shot.
The message:
fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
is also displayed.

The gecko browser engine is installed and I am able to browse websites:
wine iexplore.exe http://www.winehq.org
is working as expected.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #22 from Hin-Tak Leung   2008-04-10 12:04:03 ---
(In reply to comment #21)
> Created an attachment (id=12035)

--> (http://bugs.winehq.org/attachment.cgi?id=12035) [details]
> screenshot windows help file wpa.chm
>
> Viewing a windows help file copied from a windows installation (wpa.chm or
> others) is also not working as expected.
>
> The content of the help file is not displayed as you can see on the screen
> shot.
> The message:
> fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
> is also displayed.
>
> The gecko browser engine is installed and I am able to browse websites:
> wine iexplore.exe http://www.winehq.org
> is working as expected.

Those are *two* separate applications, please file separate bugs and attach
*two* separate traces.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #23 from Hin-Tak Leung   2008-04-10 12:13:21 ---
(In reply to comment #21)
> Created an attachment (id=12035)

--> (http://bugs.winehq.org/attachment.cgi?id=12035) [details]
> screenshot windows help file wpa.chm
>
> Viewing a windows help file copied from a windows installation (wpa.chm or
> others) is also not working as expected.
>
> The content of the help file is not displayed as you can see on the screen
> shot.
> The message:
> fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly
> is also displayed.
>
> The gecko browser engine is installed and I am able to browse websites:
> wine iexplore.exe http://www.winehq.org
> is working as expected.

If you had actually *read* what I advised, I wrote file *new* bug, and fill in
all the information - the one critical information is a URL for somebody else
to download what you are using, and what windows softare you are trying to run.
Nobody is going to go around in circles asking for information that you should
have provided in the first place.

Fill two bug reports, and fill in *every* field, and fill in every bit of
information in the *new* bug reports. Please.

(For others: this guy keeps sending private e-mails to me asking if his bugs
are the same, etc, and repeatedly, without any relevant information).

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #24 from Hin-Tak Leung   2008-04-10 12:23:07 ---
(In reply to comment #21)
> Created an attachment (id=12035)

--> (http://bugs.winehq.org/attachment.cgi?id=12035) [details]
<snipped>

The information you have provided is useless - you have a screenshot of a 2nd
application which has nothing to do with the debug trace of the first, and 
you have not provided any information about which *two* applications you are
trying to run.

FWIW, "fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly"

*always* appear whenever somebody try to read a help file -
The message just says the code is incomplete, and that's all it means; and 
"fixme" say the same thing: this code needs more work. There are about 
about a dozen HH cases.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

Hin-Tak Leung  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dirk.mueller@br-
                   |                            |automation.com

--- Comment #25 from Hin-Tak Leung   2008-04-10 12:25:35 ---
And this guy doesn't have the curtesy to add himself to CC. I am speechless.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #26 from Hin-Tak Leung   2008-04-10 12:35:07 ---
I say this guy's first application is a separate bug, the 2nd application of
his
is possibly the same as bug 6801. But without (1) separate trace, (2) separate
information which *which* *two* applications he is trying to run, all his
messages are a complete waste of time.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #27 from Dirk Mueller   2008-04-11 00:32:26 ---
I did not expect so much answers :-)

(In reply to comment #26)
> (For others: this guy keeps sending private e-mails to me asking if his bugs
> are the same, etc, and repeatedly, without any relevant information).

That's not true!
Look, I don't want to bother anyone.

Wine is new to me.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #28 from Hin-Tak Leung   2008-04-11 00:56:22 ---
(In reply to comment #27)
> I did not expect so much answers :-)
>
> (In reply to comment #26)
> > (For others: this guy keeps sending private e-mails to me asking if his bugs
> > are the same, etc, and repeatedly, without any relevant information).
>
> That's not true!
> Look, I don't want to bother anyone.
>
> Wine is new to me.

This is still going around in circles - I have repeatedly written - provide
*full* information. I don't care whether you are new or not, nor why you are
behaving in this way. What is considered full information is every field at the
top of the bug report, and you still have not provided that information, and
just repeating you are new and you are not sure whether you should be here or
not, etc etc etc.

All you have done so far, repeatedly, is to say you have a problem, but refuse
to
provide any relevant information about what your problem is. If you don't have
any thing relevant to write (the first question - *what are the names of the
two applications you are trying to run*?), please do not write - your apologies
and excuses are still wasting people's time to read.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

--- Comment #29 from Hin-Tak Leung   2008-04-17 17:55:39 ---
(In reply to comment #17)
> Created an attachment (id=11924)

--> (http://bugs.winehq.org/attachment.cgi?id=11924) [details]
> proposed patch
> proposed patch to match for "://" for url.

AJ committed the fix for teach2000:
http://www.winehq.org/pipermail/wine-cvs/2008-April/042590.html

Please close if the original poster doesn't see any more problems.

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

bas  changed:

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

--- Comment #30 from bas   2008-04-18 14:00:19 ---
This bug can be closed, as Wine version 0.9.60 is working great with Teach2000.

Thanks,
Bas

[Bug 11690] fixme:htmlhelp:HtmlHelpW Not all HH cases handled correctly

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

Dmitry Timoshkov  changed:

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

--- Comment #31 from Dmitry Timoshkov   2008-04-18 20:34:16 ---
Closing.