Revision: 6848
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=6848&view=rev
Author:   gulp21-1
Date:     2012-05-07 16:56:48 +0000 (Mon, 07 May 2012)
Log Message:
-----------
[de] fixed some false positives in AgreementRule (bug 3524295)

Modified Paths:
--------------
    trunk/JLanguageTool/src/java/org/languagetool/rules/de/AgreementRule.java
    
trunk/JLanguageTool/src/java/org/languagetool/rules/de/GermanWordRepeatRule.java

Modified: 
trunk/JLanguageTool/src/java/org/languagetool/rules/de/AgreementRule.java
===================================================================
--- trunk/JLanguageTool/src/java/org/languagetool/rules/de/AgreementRule.java   
2012-05-07 09:06:40 UTC (rev 6847)
+++ trunk/JLanguageTool/src/java/org/languagetool/rules/de/AgreementRule.java   
2012-05-07 16:56:48 UTC (rev 6848)
@@ -89,6 +89,7 @@
     REL_PRONOUN.add("deren");
     REL_PRONOUN.add("dem");
     REL_PRONOUN.add("den");
+    REL_PRONOUN.add("denen");
     REL_PRONOUN.add("welche");
     REL_PRONOUN.add("welcher");
     REL_PRONOUN.add("welchen");
@@ -104,6 +105,9 @@
     PREPOSITIONS.add("ab");
     PREPOSITIONS.add("für");
     PREPOSITIONS.add("zu");
+    PREPOSITIONS.add("bei");
+    PREPOSITIONS.add("nach");
+    PREPOSITIONS.add("über");
     // TODO: add more
   }
 
@@ -113,8 +117,11 @@
     "er", "sie", "es",
     //"wir",
     "mir",
+    "uns",
+    "ihnen",
     "euch",
     "ihm",
+    "ihr",
     "ihn",
     "dessen",
     "deren",
@@ -162,7 +169,7 @@
         continue;
       //AnalyzedGermanToken analyzedToken = new AnalyzedGermanToken(tokens[i]);
       
-       final AnalyzedGermanTokenReadings analyzedToken = 
(AnalyzedGermanTokenReadings)tokens[i];
+      final AnalyzedGermanTokenReadings analyzedToken = 
(AnalyzedGermanTokenReadings)tokens[i];
       final boolean relevantPronoun = isRelevantPronoun(tokens, i);
      
       boolean ignore = couldBeRelativeClause(tokens, i);

Modified: 
trunk/JLanguageTool/src/java/org/languagetool/rules/de/GermanWordRepeatRule.java
===================================================================
--- 
trunk/JLanguageTool/src/java/org/languagetool/rules/de/GermanWordRepeatRule.java
    2012-05-07 09:06:40 UTC (rev 6847)
+++ 
trunk/JLanguageTool/src/java/org/languagetool/rules/de/GermanWordRepeatRule.java
    2012-05-07 16:56:48 UTC (rev 6848)
@@ -50,7 +50,7 @@
       if (position >= 2 && ",".equals(tokens[position - 2].getToken())) {
         return true;
       }
-      if (position >= 3 && ",".equals(tokens[position - 3].getToken()) &&  
tokens[position - 2].getToken().matches("auf|bei|für|in|nach|über")) {
+      if (position >= 3 && ",".equals(tokens[position - 3].getToken()) &&  
tokens[position - 2].getToken().matches("ab|an|auf|bei|für|in|nach|über|zu")) {
         return true;
       }
       return false;

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Languagetool-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-cvs

Reply via email to