Review Request: autocorrect config file lookup change
C. Boemann
cbr at boemann.dk
Wed Nov 7 08:59:51 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
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.
- C.
-----------------------------------------------------------
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/c4ab1bfa/attachment.htm>
More information about the calligra-devel
mailing list