Bug 228708 - Synonyms are not checked in a transitiv way
Summary: Synonyms are not checked in a transitiv way
Status: RESOLVED FIXED
Alias: None
Product: parley
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Parley Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-27 06:43 UTC by Benjamin Schleinzer
Modified: 2014-03-16 10:56 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Check synonyms in a transitive way (3.00 KB, patch)
2010-02-27 06:45 UTC, Benjamin Schleinzer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Schleinzer 2010-02-27 06:43:46 UTC
Version:            (using KDE 4.4.0)
Compiler:          gcc-4.3 
OS:                Linux
Installed from:    Gentoo Packages

When parley checks for correct synonyms it only checks the direct synonyms. Because of bug 212306 we cannot save multiple synonyms if there is a circular connection between these synonyms, e.g. A=B=C=A .

I propose a patch to check synonyms in a transitive way. So if we have something like the following A=B and B=C than entering C if A is asked should still be okay.
Comment 1 Benjamin Schleinzer 2010-02-27 06:45:12 UTC
Created attachment 41156 [details]
Check synonyms in a transitive way

A patch that does the things I described in the bug
Comment 2 Daniel Laidig 2010-04-02 21:43:25 UTC
Checking synonyms is tricky and I don't think that checking them in a transitive way is a good idea. Sure, adding all synonyms is more work if you have lots of them, but thanks to your other patch for bug 212306 this should be fixed soon. :)

The reason for why I'm against this is that it might accept more as correct than it should since synonyms almost never have exactly the same meaning and there are words with different meanings. For example, "fix" can mean both "attach" and "repair", but we don't want to accept "attach" if you are asked to enter "repair".

So, are you okay if I close this bug as WONTFIX? If there are other important reasons why we need this, I'd be happy to hear them.
Comment 3 Benjamin Schleinzer 2010-04-03 07:42:37 UTC
If the patch for bug #212306 is accepted it would be okay to close this.

The main reason we added the patch was that we used synonyms as multiple translations for the same word. So for example in Japanese language there are 4 or more translations for thank you. Its just a big hassle to add them all. Also I wrote this patch first as it was easier :)

What I would think is a good idea is that the editor checks upon entering a new word, if a word with the same spelling is already in another lesson and asks if the new word should be added as a synonym ?
Comment 4 Inge Wallin 2014-03-16 10:56:56 UTC
Since the bug 212306 is closed and per Benjamins comment "If the patch for bug #212306 is accepted it would be okay to close this" I will hereby close this bug.

And for the record, I also agree with Daniels comment that checking synonyms transitively is tricky and not always a good idea.