<div dir="ltr">Hi Andras,<div><br></div><div>This was the bug I was trying to work on: <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=98110">https://bugs.documentfoundation.org/show_bug.cgi?id=98110</a></div><div><br></div><div>As you can see, I had very similar comments such as yours, asking if the problem was still there.  Then, after getting a confirmation from Dennis and noting that there may be multiple tasks before the problem is fixed, I decided to mock up input as described in the ticket.</div><div><br></div><div>Also, sorry for introducing a bug.  I will be more careful next time onward in testing.</div><div><br></div><div>In any case, Jan has closed the ticket as WONTFIX.  So I will look for other tasks.</div><div><br></div><div>Thank you all for your time.  I have learnt a lot already and look forward to taking tasks and working on them without asking too many questions.</div><div><br></div><div>Akash</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 17, 2016 at 5:24 AM, Andras Timar <span dir="ltr"><<a href="mailto:andras.timar@collabora.com" target="_blank">andras.timar@collabora.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Akash,<br>
<br>
I ran the script on actual help source files. What problem did you try<br>
to solve? If it was an artificial, made-up problem, then it's possible<br>
that you solved it. But the net result of the patch was a regression.<br>
%PRODUCTNAME strings from source has to be replaced with {{ProductName}}<br>
in wiki text, otherwise the replacement to LibreOffice will not be made.<br>
<br>
Let's see the original report: CommandCtrl on pages.<br>
History of Draw/Shortcut Keys for Drawing Objects shows, that the last<br>
update of this page was in 2011.<br>
<a href="https://help.libreoffice.org/WikiAction/history/Draw/Shortcut_Keys_for_Drawing_Objects" rel="noreferrer" target="_blank">https://help.libreoffice.org/WikiAction/history/Draw/Shortcut_Keys_for_Drawing_Objects</a><br>
<br>
Example:<br>
#Press {{KeyCode|CommandCtrl+F6}} to enter the document.<br>
<br>
But in my recently generated wiki text file I see (with or without your<br>
patch):<br>
<br>
#Press<br>
{{System|default={{KeyCode|Ctrl}}|mac={{KeyCode|Command}}}}{{KeyCode|+F6}}<br>
to enter the document.<br>
<br>
So, the bug was fixed earlier, but wiki was not updated. Can you please<br>
repeat my experiment on actual help files, with and without your patch,<br>
and check, if you get the same as me?<br>
<br>
Regards,<br>
Andras<br>
<div class="HOEnZb"><div class="h5"><br>
On 06/17/2016 01:04 AM, Akash Deshpande wrote:<br>
> Hi Andras,<br>
><br>
> Thank you for the review.<br>
><br>
> Actually, as I noted in my comments, I made a custom input file and checked<br>
> to make sure the output was as described by Dennis in the ticket as<br>
> requirement:  <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=98110#c7" rel="noreferrer" target="_blank">https://bugs.documentfoundation.org/show_bug.cgi?id=98110#c7</a><br>
><br>
> In a related/parent ticket, Dennis mentioned there may be two parts<br>
> (easyHacks): <a href="https://bugs.documentfoundation.org/show_bug.cgi?id=62292#c5" rel="noreferrer" target="_blank">https://bugs.documentfoundation.org/show_bug.cgi?id=62292#c5</a><br>
><br>
> So I wasn't expecting this to fix the wiki pages by itself.<br>
><br>
> Also, sorry for a silly question: does 'regression' mean adverse impact or<br>
> side effect?  I will take a look in the next few days for sure.<br>
><br>
> If this explanation is not good and if you or Dennis have other ideas on<br>
> how to tackle this task, please share some high level details.  I would<br>
> love to work on it.<br>
><br>
> I am studying other aspects of the project (python related) and also<br>
> learning git.  Norbert and others helped me a lot with my first submission<br>
> (on IRC) and I feel like I have forgotten some of the commands already :(<br>
><br>
> Thank you all again and I am looking forward to addressing this issue or<br>
> looking for other tickets in the summer.  Feel free to suggest anything<br>
> python related (I am looking at 'making unittests more pythonic' etc.)<br>
><br>
> Akash<br>
><br>
> On Thu, Jun 16, 2016 at 3:57 AM, Andras Timar (via Code Review) <<br>
> <a href="mailto:gerrit@gerrit.libreoffice.org">gerrit@gerrit.libreoffice.org</a>> wrote:<br>
><br>
>> Andras Timar has posted comments on this change.<br>
>><br>
>> Change subject: tdf#98110: fix to <item ..> handling<br>
>> ......................................................................<br>
>><br>
>><br>
>> Patch Set 3: Code-Review-1<br>
>><br>
>> I generated help wiki pages with ./help-to-wiki.py -n command with and<br>
>> without the patch and I compared the results.<br>
>><br>
>> I did not see improvements, but I saw a regression: {{ProductName}} turned<br>
>> into %PRODUCTNAME in 219 occasions.<br>
>><br>
>> --<br>
>> To view, visit <a href="https://gerrit.libreoffice.org/25000" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/25000</a><br>
>> To unsubscribe, visit <a href="https://gerrit.libreoffice.org/settings" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/settings</a><br>
>><br>
>> Gerrit-MessageType: comment<br>
>> Gerrit-Change-Id: I841bf638d9975d78ac0ffeba40697bbc614cde4a<br>
>> Gerrit-PatchSet: 3<br>
>> Gerrit-Project: help<br>
>> Gerrit-Branch: master<br>
>> Gerrit-Owner: Akash Deshpande <<a href="mailto:fastakash@gmail.com">fastakash@gmail.com</a>><br>
>> Gerrit-Reviewer: Andras Timar <<a href="mailto:andras.timar@collabora.com">andras.timar@collabora.com</a>><br>
>> Gerrit-Reviewer: dennisroczek <<a href="mailto:dennisroczek@libreoffice.org">dennisroczek@libreoffice.org</a>><br>
>> Gerrit-Reviewer: jan iversen <<a href="mailto:jani@documentfoundation.org">jani@documentfoundation.org</a>><br>
>> Gerrit-HasComments: No<br>
>><br>
><br>
><br>
><br>
--<br>
</div></div><span class="HOEnZb"><font color="#888888">Andras Timar<br>
Product Development Manager<br>
Collabora Productivity Ltd.<br>
<a href="https://collaboraoffice.com/" rel="noreferrer" target="_blank">https://collaboraoffice.com/</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">akash</div>
</div>