Hi, I will start looking at this.  I have been learning python for about a year and this will be my first task in Libreoffice project, if I can finish it.  This showed up as one of the tasks for easyHack/Python.

Even though the described problem shows up at https://help.libreoffice.org/Draw/Shortcut_Keys_for_Drawing_Objects, when I run help/help-to-wiki.py and inspect wiki/Draw/Shortcut_Keys_for_Drawing_Objects/MAIN, the problem is not there.  Please let me know if I am not looking in the right place.

Also, it took me a long time to find out why I was not finding these files when I built core as per the instructions.  Much later, I realized that help is separate and I may have to download it.

I will try to keep studying these programs, if there is something else I can get into.  It seems one way to quickly replicate the problem is to keep just one file in help/alltitles.csv and comment out running to-wiki/getalltitles.py.  Please let me know if there is a better way, if I have to continue to work on this.

If I am right, should I mark it as RESOLVED?  If the problem is still there, please do let me know how I can continue to investigate.

Thank You

Oh. I'm really sorry. I had to mentioned that. You have to build code + help files. 

so you need to your autogen
--with-help=de

or the like. best ask at the irc channel at reenode #libreoffice-dev (me is on the way to be)

the problem is still there: everything you see on the page help.libo.org is generated from the help files (xhp) converted by the named script.


> It seems one way to quickly replicate the problem is to keep just one file in
> help/alltitles.csv and comment out running to-wiki/getalltitles.py.  Please let me 
> know if there is a better way, if I have to continue to work on this.
Wthout checing the files (sry, I'm tired): no: the alltitles is simply a list of files which is uploaded.

Please try to generated the xhd files and then to debug with a phyton debugger where we do the conversation wrong.

Thank you for your answer.  I will keep investigating.  I hope it's not urgent.  I am traveling and will continue from Sunday evening.  I will spend 1-2 hours per day, more in the weekend and hoping to take more tasks in the summer.  If it is urgent, please feel free to assign to someone else and I can look for another assignment.

I am still learning git, git-review etc.

Take your time in fixing this ticket. If you realize that you cannot solve that ticket, or you simply have not enough time, please remove yourself from the assignment field.

Thanks for your help and comments.  I rebuilt the code, I now have helpcontent2 folder and I am now ABLE to reproduce the problem.

The confusion I had/have is that even though the wiki html page has the problem, the wiki page generated by the driver program help-to-wiki.py (which is not html, I guess the conversion happens somewhere else) does not show the problem.

For example, I am looking at the following page (one of three mentioned in the ticket 62292):
source/text/scalc/01/06130000.xhp;Calc/AutoInput;AutoInput

The following input:
<switchinline select="sys"><caseinline select="MAC"><item 
type="keycode">Command</item></caseinline><defaultinline><item type="keycode">Ctrl</item></defaultinline></switchinline><item type="keycode">+Tab</item> 
produces:
{{System|default={{KeyCode|Ctrl}}|mac={{KeyCode|Command}}}}{{KeyCode|+Tab}}

which I guess looks ok.  But, as I said, the wiki WEB page is bad.

I then edited the input file and pasted the exact content you mentioned is not handled properly, and I get the problem output {{KeyCode|CommandCtrl+Tab}}

I will focus on fixing this and study the code in detail, over the next few days.

I have submitted a patch for this: https://gerrit.libreoffice.org/#/c/25000/

I made a custom input file with the content as mentioned in the comment by Dennis to replicate the problem and made the changes.

Input:
<item type="keycode"><switchinline select="sys"><caseinline select="MAC">Command</caseinline><defaultinline>Ctrl</defaultinline></switchinline>+Tab</item> 

Now generates:
{{KeyCode|{{System|default=Ctrl|mac=Command}}+Tab}}

Since the Paragraph class handles switchinline and other objects properly, I subclassed Item from Paragraph to reuse its start_element method.  If this is not appropriate, please let me know.  I will be happy to rework as necessary.

A polite ping, still working on this issue ?

Hi Jan,

Thank you for the ping.  I am waiting for the review.  It's my first commit and it would be awesome to get it approved!  In case there is any feedback that needs to be addressed, I am busy with some exams for the next few days but will certainly be available after 29th or so.  I was also looking at 'making some unittests more pythonic' and studying git in more detail.. Akash

It seems this bug was solved, but the issue was not closed.

Your work however was good, and I look forward to see more work from you after your exams. One of my "duties" is to help contributors, so feel free to contact me.

Apart from the unit tests, we have other python tasks, and even a little project, where you need to do a bit of research and not only write predefined code

cross fungers for your exam.
rgda
jan i

@jani why wontfix? the wanted wikimarkup is still desired as the output on the helpwiki is horrible and confuses the normal user.

(In reply to Dennis Roczek from comment #11)
> @jani why wontfix? the wanted wikimarkup is still desired as the output on
> the helpwiki is horrible and confuses the normal user.

@Dennis, the output of the conversion script is correct, as I mentioned on the mailing list. help.libreoffice.org should be updated (which I'm doing right now). See for example https://help.libreoffice.org/Draw/Shortcut_Keys_for_Drawing_Objects. The CommandCtrl is not there. In fact the bug was solved by Christian Lohmaier, who fixed the invalid xhp files, instead of changing the conversion script. 

commit f9cb7f4660c039f7bf18bf60f6fb9c77f7e4f54b
Author: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
Date:   Mon Jan 25 15:26:12 2016 +0100

    switchinline is no valid child of "item" either
    
    sed command used:
    's#\(<item[^>]*>\)\(<switchinline[^>]*><caseinline[^>]*>\)\([^>]*\)\(</caseinline><defaultinline>\)\([^<]*\)\(</defaultinline></switchinline>\)\([
^<]*</item>\)#\2\1\3</item>\4\1\5</item>\6\1\7#g'