Review Request: autocorrect config file lookup change

Laurent Montel montel at kde.org
Wed Nov 7 09:11:50 GMT 2012



> On Nov. 6, 2012, 6:04 p.m., Laurent Montel wrote:
> > plugins/textediting/autocorrection/Autocorrect.cpp, line 707
> > <http://git.reviewboard.kde.org/r/107229/diff/1/?file=93886#file93886line707>
> >
> >     You never breaks here ?
> >     So you will read a lot of file no ?
> 
> C. Boemann wrote:
>     yes that is the point - I want to first read the general kde wide file and then the calligra specific file.
>     
>     I could also revert your original patch and just read the calligra one
>     
>     the point is we need af file with the calligra specifics
> 
> Laurent Montel wrote:
>     "The special tags in the calligra supplied config files where never found" which special tags?
>     
>     global file and calligra file is the same (installed from l10n) So What is the problem ?
> 
> C. Boemann wrote:
>     Uhm it's not the same file. It may be so in kde 4.9 onwards or so but we should be compatible with kde 4.5
>     
>     but calligra needs special tags that I for one don't have in my .kde/share/apps/autocorrect/* file
> 
> Laurent Montel wrote:
>     I don't understand because it's installed from same file from l10n...
>     So what is missing tag ?
> 
> C. Boemann wrote:
>     for example:
>     
>     
>      <SuperScript>
>       <superscript find="1st" super="st" />
>       <superscript find="2nd" super="nd" />
>       <superscript find="3rd" super="rd" />
>       <superscript find="othernb" super="th" />
>      </SuperScript>
>
> 
> Laurent Montel wrote:
>     Error is not that file changed, it's that we don't store it.
>     When I looked at calligra code there was not save code => you didn't save it.
>     => I added it, and missing to add this part.
>     
>     
>     I will create a patch for it.
>     (it's right I don't use it in kmail => I didn't see that it was missing)
>     
>     Will fix it soon.
>     
>     When you added this plugins to calligra (don't know who) rewrite it and forgot to add write code...
> 
> Laurent Montel wrote:
>     I fixed save superscript in kmail and calligra
>     regards
> 
> C. Boemann wrote:
>     Ok thanks for that, but it still leaves the problem that it's not loaded in the first place
>     
>     There are more things here:
>     
>      1) the folders << "/"
>         at the very least I think it should be "" For me the found filename was "/autocorrect/en_US.xml" which is a file that doesn't even exist, yet findResource returns as if it did. This may be due to a bug/difference in older versions of findResource.
>     
>      2) even if I fix the above then the file I do load "/home/cbo/.kde/share/apps/autocorrect/autocorrect.xml" doesn't contain the superscript part. This is probably a derivative problem of us not saving in the past, but it also means that the file is still around and will be loaded.
>

1) don't know but if it fixes for you loading change to ""
2) we can't real fix it without remove this "/home/cbo/.kde/share/apps/autocorrect/autocorrect.xml" in the past we didn't save it so it's not a real problem to remove it and recreate it locally now that save/load is ok.
(I just added saving few weeks ago so it will not impact a lot of calligra release I think).


- Laurent


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107229/#review21494
-----------------------------------------------------------


On Nov. 6, 2012, 5:54 p.m., C. Boemann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107229/
> -----------------------------------------------------------
> 
> (Updated Nov. 6, 2012, 5:54 p.m.)
> 
> 
> Review request for Calligra and Laurent Montel.
> 
> 
> Description
> -------
> 
> The special tags in the calligra supplied config files where never found
> 
> So I've changed it so we first look up the general files and then the calligra specific files on top
> 
> A problem I see is that the general contents is never really used with my approach if we afterwards find a calligra special file
> 
> So a better suggestion is appreciated
> 
> 
> Diffs
> -----
> 
>   plugins/textediting/autocorrection/Autocorrect.h 5b76121 
>   plugins/textediting/autocorrection/Autocorrect.cpp 53fdee6 
> 
> Diff: http://git.reviewboard.kde.org/r/107229/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> C. Boemann
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121107/fffb241c/attachment.htm>


More information about the calligra-devel mailing list