From b546cec2ac337c287e5a9f87087ee2d9a0922c2e Mon Sep 17 00:00:00 2001
From: Frederik Gladhorn <gladhorn@kde.org>
Date: Sat, 1 Sep 2007 17:43:00 +0000
Subject: [PATCH] Fix tags in old reader (KV_CON_ENTRY/KV_CON_TYPE) mixed up.
 Works again.

svn path=/trunk/KDE/kdeedu/libkdeedu/; revision=707387
---
 keduvocdocument/keduvockvtmlreader.cpp | 81 +++++++++++++-------------
 1 file changed, 40 insertions(+), 41 deletions(-)

diff --git a/keduvocdocument/keduvockvtmlreader.cpp b/keduvocdocument/keduvockvtmlreader.cpp
index 681abed..66bffb0 100644
--- a/keduvocdocument/keduvockvtmlreader.cpp
+++ b/keduvocdocument/keduvockvtmlreader.cpp
@@ -358,7 +358,7 @@ bool KEduVocKvtmlReader::readTranslationConjugations( QDomElement &domElementPar
 
     QString tense;
 
-    QDomElement domElementConjugChild = domElementParent.firstChildElement(KV_CON_ENTRY);
+    QDomElement domElementConjugChild = domElementParent.firstChildElement(KV_CON_TYPE);
     while ( !domElementConjugChild.isNull() )
     {
         // "n" == is the type is the tense
@@ -370,7 +370,7 @@ bool KEduVocKvtmlReader::readTranslationConjugations( QDomElement &domElementPar
         readConjugation(domElementConjugChild, conjugation);
         translation.setConjugation(tense, conjugation);
 
-        domElementConjugChild = domElementConjugChild.nextSiblingElement( KV_CON_ENTRY );
+        domElementConjugChild = domElementConjugChild.nextSiblingElement( KV_CON_TYPE );
     } // while -> next tense, count++
     return true;
 }
@@ -425,55 +425,54 @@ bool KEduVocKvtmlReader::readConjugation( QDomElement &domElementParent, KEduVoc
     p3_common = false;
     s3_common = false;
 
-        // get the individual entries for persons...
-        QDomElement domElementConjugGrandChild = domElementParent.firstChild().toElement();
-        while ( !domElementConjugGrandChild.isNull() ) {
-            if ( domElementConjugGrandChild.tagName() == KV_CON_P1S ) {
-                pers1_sing = domElementConjugGrandChild.text();
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P2S ) {
-                pers2_sing = domElementConjugGrandChild.text();
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SF ) {
-                QDomAttr domAttrCommon = domElementConjugGrandChild.attributeNode( KV_CONJ_COMMON );
-                if ( !domAttrCommon.isNull() )
-                    s3_common = domAttrCommon.value().toInt();  // returns 0 if the conversion fails
-                pers3_f_sing = domElementConjugGrandChild.text();
+    // get the individual entries for persons...
+    QDomElement domElementConjugGrandChild = domElementParent.firstChild().toElement();
+    while ( !domElementConjugGrandChild.isNull() ) {
+        if ( domElementConjugGrandChild.tagName() == KV_CON_P1S ) {
+            pers1_sing = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P2S ) {
+            pers2_sing = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SF ) {
+            QDomAttr domAttrCommon = domElementConjugGrandChild.attributeNode( KV_CONJ_COMMON );
+            if ( !domAttrCommon.isNull() )
+                s3_common = domAttrCommon.value().toInt();  // returns 0 if the conversion fails
+            pers3_f_sing = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SM ) {
-                pers3_m_sing = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SM ) {
+            pers3_m_sing = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SN ) {
-                pers3_n_sing = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3SN ) {
+            pers3_n_sing = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P1P ) {
-                pers1_plur = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P1P ) {
+            pers1_plur = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P2P ) {
-                pers2_plur = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P2P ) {
+            pers2_plur = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PF ) {
-                QDomAttr domAttrCommon = domElementConjugGrandChild.attributeNode( KV_CONJ_COMMON );
-                if ( !domAttrCommon.isNull() )
-                    p3_common = domAttrCommon.value().toInt();  // returns 0 if the conversion fails
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PF ) {
+            QDomAttr domAttrCommon = domElementConjugGrandChild.attributeNode( KV_CONJ_COMMON );
+            if ( !domAttrCommon.isNull() )
+                p3_common = domAttrCommon.value().toInt();  // returns 0 if the conversion fails
 
-                pers3_f_plur = domElementConjugGrandChild.text();
+            pers3_f_plur = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PM ) {
-                pers3_m_plur = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PM ) {
+            pers3_m_plur = domElementConjugGrandChild.text();
 
-            } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PN ) {
-                pers3_n_plur = domElementConjugGrandChild.text();
+        } else if ( domElementConjugGrandChild.tagName() == KV_CON_P3PN ) {
+            pers3_n_plur = domElementConjugGrandChild.text();
 
-            } else {
-                return false;
-            }
-
-            domElementConjugGrandChild = domElementConjugGrandChild.nextSibling().toElement();
-        } // while - probably to be sure, because the persons could be in any order.
-        // I guess this goes over only one set, such as:
-        // <s1>traigo</s1><s2>traes</s2><s3fcommon="1">trae</s3f>
-        // <p1>traemos</p1><p2>traéis</p2><p3f common="1">traen</p3f>
-        // until no elements are left in that soup.
+        } else {
+            return false;
+        }
 
+        domElementConjugGrandChild = domElementConjugGrandChild.nextSibling().toElement();
+    } // while - probably to be sure, because the persons could be in any order.
+    // I guess this goes over only one set, such as:
+    // <s1>traigo</s1><s2>traes</s2><s3fcommon="1">trae</s3f>
+    // <p1>traemos</p1><p2>traéis</p2><p3f common="1">traen</p3f>
+    // until no elements are left in that soup.
 
 
     // now set the data: [count] - number of conjug?
-- 
2.47.3