Author: jghali
Date: Wed Oct  9 12:59:57 2019
New Revision: 23241

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=23241
Log:
Code style fixes

Modified:
    trunk/Scribus/scribus/plugins/fileloader/scribus12format/scribus12format.cpp
    
trunk/Scribus/scribus/plugins/fileloader/scribus134format/scribus134format.cpp
    trunk/Scribus/scribus/plugins/fileloader/scribus13format/scribus13format.cpp
    
trunk/Scribus/scribus/plugins/fileloader/scribus150format/scribus150format.cpp

Modified: 
trunk/Scribus/scribus/plugins/fileloader/scribus12format/scribus12format.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=23241&path=/trunk/Scribus/scribus/plugins/fileloader/scribus12format/scribus12format.cpp
==============================================================================
--- 
trunk/Scribus/scribus/plugins/fileloader/scribus12format/scribus12format.cpp    
    (original)
+++ 
trunk/Scribus/scribus/plugins/fileloader/scribus12format/scribus12format.cpp    
    Wed Oct  9 12:59:57 2019
@@ -722,11 +722,11 @@
        struct ScribusDoc::BookMa bok;
        int counter;
        bool newVersion = false;
-       QString tmp, tmpf, PgNam, Defont;
+       QString tmp, tmpf, pageName, Defont;
        QMap<int,int> TableID;
        QList<PageItem*> TableItems;
        int x, a;
-       PageItem *Neu;
+       PageItem *newItem;
        groupRemap.clear();
        itemRemap.clear();
        itemNext.clear();
@@ -758,18 +758,18 @@
                return false;
        if (elem.hasAttribute("Version"))
                newVersion = true;
-       QDomNode DOC=elem.firstChild();
+       QDomNode docNode = elem.firstChild();
        if (m_mwProgressBar!=nullptr)
        {
-               m_mwProgressBar->setMaximum(DOC.childNodes().count());
+               m_mwProgressBar->setMaximum(docNode.childNodes().count());
                m_mwProgressBar->setValue(0);
        }
        int ObCount = 0;
        int activeLayer = 0;
-       PrefsManager& prefsManager=PrefsManager::instance();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
+       PrefsManager& prefsManager = PrefsManager::instance();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
        /*
        * Attribute von DOCUMENT auslesen
        */
@@ -814,12 +814,12 @@
                m_Doc->guidesPrefs().gridType = 0;
 //             DoFonts.clear();
                
m_Doc->itemToolPrefs().textSize=qRound(ScCLocale::toDoubleC(dc.attribute("DSIZE"))
 * 10);
-               Defont=dc.attribute("DFONT");
+               Defont = dc.attribute("DFONT");
                m_Doc->itemToolPrefs().textFont = 
prefsManager.appPrefs.itemToolPrefs.textFont;
                m_AvailableFonts->findFont(Defont, m_Doc);
                m_Doc->itemToolPrefs().textFont = Defont;
-               m_Doc->itemToolPrefs().textColumns=dc.attribute("DCOL", 
"1").toInt();
-               
m_Doc->itemToolPrefs().textColumnGap=ScCLocale::toDoubleC(dc.attribute("DGAP"), 
0.0);
+               m_Doc->itemToolPrefs().textColumns = dc.attribute("DCOL", 
"1").toInt();
+               m_Doc->itemToolPrefs().textColumnGap = 
ScCLocale::toDoubleC(dc.attribute("DGAP"), 0.0);
                DocumentInformation di;
                di.setAuthor(dc.attribute("AUTHOR"));
                di.setComments(dc.attribute("COMMENTS"));
@@ -867,13 +867,13 @@
                //m_Doc->setHyphLanguage(dc.attribute("LANGUAGE", ""));
                static const QString LANGUAGE("LANGUAGE");
                QString l(dc.attribute(LANGUAGE, "en"));
-               if (LanguageManager::instance()->langTableIndex(l)!=-1)
+               if (LanguageManager::instance()->langTableIndex(l) != -1)
                        m_Doc->setLanguage(l); //new style storage
                else
                { //old style storage
-                       QString 
lnew=LanguageManager::instance()->getAbbrevFromLang(l, false);
+                       QString lnew = 
LanguageManager::instance()->getAbbrevFromLang(l, false);
                        if (lnew.isEmpty())
-                               
lnew=LanguageManager::instance()->getAbbrevFromLang(l, false);
+                               lnew = 
LanguageManager::instance()->getAbbrevFromLang(l, false);
                        m_Doc->setLanguage(lnew);
                }
 //             m_Doc->setHyphMinimumWordLength(dc.attribute("MINWORDLEN", 
"3").toInt());
@@ -888,16 +888,16 @@
                m_Doc->itemToolPrefs().lineStartArrow = 0;
                m_Doc->itemToolPrefs().lineEndArrow = 0;
                m_Doc->LastAuto = nullptr;
-               QDomNode PAGE=DOC.firstChild();
+               QDomNode pageNode = docNode.firstChild();
                counter = 0;
-               while (!PAGE.isNull())
+               while (!pageNode.isNull())
                {
                        ObCount++;
                        if (m_mwProgressBar!=nullptr)
                                m_mwProgressBar->setValue(ObCount);
-                       QDomElement pg=PAGE.toElement();
+                       QDomElement pg = pageNode.toElement();
                        // 10/25/2004 pv - None is "reserved" color. cannot be 
defined in any file...
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -907,7 +907,7 @@
                                lf.setRegistrationColor(false);
                                m_Doc->PageColors.insert(pg.attribute("NAME"), 
lf);
                        }
-                       if (pg.tagName()=="STYLE")
+                       if (pg.tagName() == "STYLE")
                        {
                                vg.erase();
                                GetStyle(&pg, &vg, nullptr, m_Doc, true);
@@ -915,9 +915,9 @@
                                temp.create(vg);
                                m_Doc->redefineStyles(temp, false);
                        }
-                       if (pg.tagName()=="JAVA")
+                       if (pg.tagName() == "JAVA")
                                m_Doc->JavaScripts[pg.attribute("NAME")] = 
pg.attribute("SCRIPT");
-                       if (pg.tagName()=="LAYERS")
+                       if (pg.tagName() == "LAYERS")
                        {
                                int lId   = pg.attribute("NUMMER").toInt();
                                int level = pg.attribute("LEVEL").toInt();
@@ -926,10 +926,10 @@
                                la.isPrintable = 
pg.attribute("DRUCKEN").toInt();
                                m_Doc->Layers.append(la);
                        }
-                       if (pg.tagName()=="MultiLine")
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -945,7 +945,7 @@
                                }
                                m_Doc->MLineStyles.insert(pg.attribute("Name"), 
ml);
                        }
-                       if (pg.tagName()=="PAGE")
+                       if (pg.tagName() == "PAGE")
                        {
                                TableItems.clear();
                                TableID.clear();
@@ -953,11 +953,11 @@
                        * Attribute von PAGE auslesen
                        */
                                a = pg.attribute("NUM").toInt();
-                               PgNam = "";
-                               PgNam = pg.attribute("NAM", "");
+                               pageName = "";
+                               pageName = pg.attribute("NAM", "");
                                QString Mus = "";
                                Mus = pg.attribute("MNAM","Normal");
-                               if (PgNam.isEmpty())
+                               if (pageName.isEmpty())
                                {
                                        //We store the pages master page but do 
not apply it now
                                        //as it may not exist yet. They are 
applied in scribus.cpp for now.
@@ -968,7 +968,7 @@
                                else
                                {
                                        m_Doc->setMasterPageMode(true);
-                                       
m_Doc->setCurrentPage(m_Doc->addMasterPage(a, PgNam));
+                                       
m_Doc->setCurrentPage(m_Doc->addMasterPage(a, pageName));
                                }
                                //CB: Remove this unnecessarily "slow" slot 
call when we have no gui for the doc yet!
                                //Items dont appear in the right place if we 
just m_Doc->addPage(a); for <=1.2.x docs
@@ -991,12 +991,12 @@
                                                        
GuideManagerCore::Standard,
                                                        
pg.hasAttribute("NumHGuides"));
 
-                               QDomNode OBJ=PAGE.firstChild();
+                               QDomNode objNode = pageNode.firstChild();
                                int pageNo = a;
                                int pageItem = 0;
-                               while (!OBJ.isNull())
-                               {
-                                       QDomElement obj=OBJ.toElement();
+                               while (!objNode.isNull())
+                               {
+                                       QDomElement obj = objNode.toElement();
                                        /*
                                         * Attribute von OBJECT auslesen
                                         */
@@ -1005,7 +1005,7 @@
                                                const long long itemID = 
scribus12itemID(pageItem++, pageNo);
 //                                             qDebug() << QString("1.2 remap: 
%1 -> %2 [%3 on page 
%4]").arg(itemID).arg(m_Doc->Items->count()).arg(pageItem-1).arg(pageNo);
                                                itemRemap[itemID] = 
m_Doc->Items->count();
-                                               if (obj.tagName()=="PAGEOBJECT")
+                                               if (obj.tagName() == 
"PAGEOBJECT")
                                                {
                                                        // member of linked 
chain?
                                                        if 
(obj.attribute("NEXTITEM").toInt() != -1)
@@ -1057,11 +1057,11 @@
                                        }
                                        else
                                                OB.Groups.clear();
-                                       QDomNode IT=OBJ.firstChild();
-                                       while (!IT.isNull())
+                                       QDomNode node = objNode.firstChild();
+                                       while (!node.isNull())
                                        {
-                                               QDomElement it=IT.toElement();
-                                               if (it.tagName()=="CSTOP")
+                                               QDomElement it = 
node.toElement();
+                                               if (it.tagName() == "CSTOP")
                                                {
                                                        QString name = 
it.attribute("NAME");
                                                        double ramp = 
ScCLocale::toDoubleC(it.attribute("RAMP"), 0.0);
@@ -1072,7 +1072,7 @@
                                                        OB.GrColor = "";
                                                        OB.GrColor2 = "";
                                                }
-                                               IT=IT.nextSibling();
+                                               node = node.nextSibling();
                                        }
                                        OB.itemText = "";
                                        /*int docGc = m_Doc->GroupCounter;
@@ -1080,33 +1080,33 @@
                                        uint last = m_Doc->Items->count();
                                        PasteItem(&OB);
                                        /*m_Doc->GroupCounter = docGc;*/
-                                       Neu = m_Doc->Items->at(last);
-                                       Neu->setMasterPageName(PgNam);
-                                       Neu->setOwnerPage(a); //No need to scan 
for OnPage as we know page by page in 1.2.x
-                                       Neu->oldOwnPage = 0;
-                                       Neu->setRedrawBounding();
-                                       IT=OBJ.firstChild();
+                                       newItem = m_Doc->Items->at(last);
+                                       newItem->setMasterPageName(pageName);
+                                       newItem->setOwnerPage(a); //No need to 
scan for OnPage as we know page by page in 1.2.x
+                                       newItem->oldOwnPage = 0;
+                                       newItem->setRedrawBounding();
+                                       node = objNode.firstChild();
                                        LastStyles * lastS = new LastStyles();
-                                       while (!IT.isNull())
+                                       while (!node.isNull())
                                        {
-                                               QDomElement it=IT.toElement();
-                                               if (it.tagName()=="ITEXT")
-                                                       GetItemText(&it, m_Doc, 
true, false, Neu, lastS);
-                                               IT=IT.nextSibling();
+                                               QDomElement it = 
node.toElement();
+                                               if (it.tagName() == "ITEXT")
+                                                       GetItemText(&it, m_Doc, 
true, false, newItem, lastS);
+                                               node = node.nextSibling();
                                        }
                                        delete lastS;
-                                       
Neu->isAutoText=static_cast<bool>(obj.attribute("AUTOTEXT").toInt());
-                                       if (Neu->isAutoText)
-                                               m_Doc->LastAuto = Neu;
-//                                     Neu->NextIt = 
obj.attribute("NEXTITEM").toInt();
-//                                     nextPg[Neu->ItemNr] = 
obj.attribute("NEXTPAGE").toInt();
-                                       if (Neu->isTableItem)
+                                       
newItem->isAutoText=static_cast<bool>(obj.attribute("AUTOTEXT").toInt());
+                                       if (newItem->isAutoText)
+                                               m_Doc->LastAuto = newItem;
+//                                     newItem->NextIt = 
obj.attribute("NEXTITEM").toInt();
+//                                     nextPg[newItem->ItemNr] = 
obj.attribute("NEXTPAGE").toInt();
+                                       if (newItem->isTableItem)
                                        {
-                                               TableItems.append(Neu);
+                                               TableItems.append(newItem);
                                                
TableID.insert(obj.attribute("OwnLINK", "0").toInt(), last);
                                        }
                                        counter++;
-                                       OBJ=OBJ.nextSibling();
+                                       objNode = objNode.nextSibling();
                                }
                                if (TableItems.count() != 0)
                                {
@@ -1132,14 +1132,15 @@
                                        }
                                }
                        }
-                       PAGE=PAGE.nextSibling();
+                       pageNode = pageNode.nextSibling();
                }
                m_Doc->setMasterPageMode(false);
-               PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="Bookmark")
+
+               pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "Bookmark")
                        {
                                int elem = pg.attribute("Element").toInt();
                                if (elem < m_Doc->Items->count())
@@ -1157,7 +1158,7 @@
                                        m_Doc->BookMarks.append(bok);
                                }
                        }
-                       if (pg.tagName()=="PDF")
+                       if (pg.tagName() == "PDF")
                        {
                                m_Doc->pdfOptions().Articles = 
static_cast<bool>(pg.attribute("Articles").toInt());
                                m_Doc->pdfOptions().Thumbnails = 
static_cast<bool>(pg.attribute("Thumbnails").toInt());
@@ -1208,7 +1209,7 @@
                                m_Doc->pdfOptions().UseLPI = 
static_cast<bool>(pg.attribute("UseLpi", "0").toInt());
                                m_Doc->pdfOptions().UseSpotColors = true;
                                m_Doc->pdfOptions().doMultiFile = false;
-                               QDomNode PFO = PAGE.firstChild();
+                               QDomNode PFO = pageNode.firstChild();
                                while (!PFO.isNull())
                                {
                                        QDomElement pdfF = PFO.toElement();
@@ -1244,9 +1245,9 @@
                                        PFO = PFO.nextSibling();
                                }
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        //m_Doc->Pages = &m_Doc->DocPages;
        //m_Doc->pageCount = m_Doc->Pages->count();
@@ -1365,7 +1366,7 @@
        
        setCurrentComboItem(m_ScMW->unitSwitcher, 
unitGetStrFromIndex(m_Doc->unitIndex()));
        if (m_mwProgressBar!=nullptr)
-               m_mwProgressBar->setValue(DOC.childNodes().count());
+               m_mwProgressBar->setValue(docNode.childNodes().count());
 
        ScMessageBox::warning(ScCore->primaryMainWindow(),
                                                  CommonStrings::trWarning,
@@ -1821,7 +1822,7 @@
        struct CopyPasteBuffer OB;
        ParagraphStyle vg;
        struct ScribusDoc::BookMa bok;
-       PageItem *Neu;
+       PageItem *newItem;
        groupRemap.clear();
        itemRemap.clear();
        itemNext.clear();
@@ -1869,19 +1870,19 @@
                return false;
        if (elem.hasAttribute("Version"))
                newVersion = true;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
        /*
        * Attribute von DOCUMENT auslesen
        */
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
                        // 10/25/2004 pv - None is "reserved" color. cannot be 
defined in any file...
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -1891,14 +1892,14 @@
                                lf.setRegistrationColor(false);
                                m_Doc->PageColors.insert(pg.attribute("NAME"), 
lf);
                        }
-                       if (pg.tagName()=="STYLE")
+                       if (pg.tagName() == "STYLE")
                        {
                                GetStyle(&pg, &vg, nullptr, m_Doc, true);
                                VorLFound = true;
                        }
-                       if (pg.tagName()=="JAVA")
+                       if (pg.tagName() == "JAVA")
                                m_Doc->JavaScripts[pg.attribute("NAME")] = 
pg.attribute("SCRIPT");
-                       if (pg.tagName()=="LAYERS")
+                       if (pg.tagName() == "LAYERS")
                        {
                                int lId   = pg.attribute("NUMMER").toInt();
                                int level = pg.attribute("LEVEL").toInt();
@@ -1919,10 +1920,10 @@
                                        m_Doc->Layers.append(la);
                                }
                        }
-                       if (pg.tagName()=="MultiLine")
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -1950,14 +1951,14 @@
                                }
                                m_Doc->MLineStyles.insert(Nam2, ml);
                        }
-                       if ((pg.tagName()=="PAGE") && 
(pg.attribute("NUM").toInt() == pageNumber))
+                       if ((pg.tagName() == "PAGE") && 
(pg.attribute("NUM").toInt() == pageNumber))
                        {
                                a = m_Doc->currentPage()->pageNr();
                                if (Mpage)
                                {
                                        if (pg.attribute("NAM", "").isEmpty())
                                        {
-                                               PAGE=PAGE.nextSibling();
+                                               pageNode = 
pageNode.nextSibling();
                                                continue;
                                        }
                                }
@@ -1965,13 +1966,13 @@
                                {
                                        if (!pg.attribute("NAM", "").isEmpty())
                                        {
-                                               PAGE=PAGE.nextSibling();
+                                               pageNode = 
pageNode.nextSibling();
                                                continue;
                                        }
                                }
 /*                             if ((pg.attribute("NAM", "").isEmpty()) && 
(Mpage))
                                {
-                                       PAGE=PAGE.nextSibling();
+                                       pageNode = pageNode.nextSibling();
                                        continue;
                                } */
                                if (Mpage)
@@ -1995,18 +1996,18 @@
                                                pg.hasAttribute("NumHGuides"));
                                m_Doc->Pages->at(a)->marginPreset = 
pg.attribute("PRESET", "0").toInt();
 
-                               QDomNode OBJ=PAGE.firstChild();
+                               QDomNode objNode = pageNode.firstChild();
                                counter = m_Doc->Items->count();
                        //      baseobj = counter;
                                int pageItem = 0;
-                               while (!OBJ.isNull())
-                               {
-                                       QDomElement obj=OBJ.toElement();
+                               while (!objNode.isNull())
+                               {
+                                       QDomElement obj = objNode.toElement();
                                /*
                                * Attribute von OBJECT auslesen
                                */
                                        itemRemap[scribus12itemID(pageItem++, 
pageNumber)] = m_Doc->Items->count();
-                                       if (obj.tagName()=="PAGEOBJECT")
+                                       if (obj.tagName() == "PAGEOBJECT")
                                        {
                                                // member of linked chain?
                                                if 
((obj.attribute("NEXTITEM").toInt() != -1) && 
(obj.attribute("NEXTPAGE").toInt() == pageNumber))
@@ -2061,11 +2062,11 @@
                                        }
                                        else
                                                OB.Groups.clear();
-                                       QDomNode IT=OBJ.firstChild();
-                                       while (!IT.isNull())
+                                       QDomNode node = objNode.firstChild();
+                                       while (!node.isNull())
                                        {
-                                               QDomElement it=IT.toElement();
-                                               if (it.tagName()=="CSTOP")
+                                               QDomElement it = 
node.toElement();
+                                               if (it.tagName() == "CSTOP")
                                                {
                                                        QString name = 
it.attribute("NAME");
                                                        double ramp = 
ScCLocale::toDoubleC(it.attribute("RAMP"), 0.0);
@@ -2076,36 +2077,36 @@
                                                        OB.GrColor = "";
                                                        OB.GrColor2 = "";
                                                }
-                                               IT=IT.nextSibling();
+                                               node = node.nextSibling();
                                        }
                                        OB.itemText = "";
                                        PasteItem(&OB);
-                                       Neu = m_Doc->Items->at(counter);
-                                       IT=OBJ.firstChild();
+                                       newItem = m_Doc->Items->at(counter);
+                                       node = objNode.firstChild();
                                        LastStyles* last = new LastStyles();
-                                       while (!IT.isNull())
+                                       while (!node.isNull())
                                        {
-                                               QDomElement it=IT.toElement();
-                                               if (it.tagName()=="ITEXT")
-                                                       GetItemText(&it, m_Doc, 
VorLFound, true, Neu, last);
-                                               IT=IT.nextSibling();
+                                               QDomElement it = 
node.toElement();
+                                               if (it.tagName() == "ITEXT")
+                                                       GetItemText(&it, m_Doc, 
VorLFound, true, newItem, last);
+                                               node = node.nextSibling();
                                        }
                                        delete last;
 
 //                                     if (obj.attribute("NEXTPAGE").toInt() 
== pageNumber)
 //                                     {
-//                                             Neu->NextIt = baseobj + 
obj.attribute("NEXTITEM").toInt();
-//                                             nextPg[Neu->ItemNr] = a; // 
obj.attribute("NEXTPAGE").toInt();
+//                                             newItem->NextIt = baseobj + 
obj.attribute("NEXTITEM").toInt();
+//                                             nextPg[newItem->ItemNr] = a; // 
obj.attribute("NEXTPAGE").toInt();
 //                                     }
 //                                     else
-//                                             Neu->NextIt = -1;
-                                       if (Neu->isTableItem)
+//                                             newItem->NextIt = -1;
+                                       if (newItem->isTableItem)
                                        {
-                                               TableItems.append(Neu);
+                                               TableItems.append(newItem);
                                                
TableID.insert(obj.attribute("OwnLINK", "0").toInt(), counter);
                                        }
                                        counter++;
-                                       OBJ=OBJ.nextSibling();
+                                       objNode = objNode.nextSibling();
                                }
                                if (TableItems.count() != 0)
                                {
@@ -2213,11 +2214,11 @@
 
                                if (!Mpage)
                                        m_View->reformPages();
-                               PAGE=DOC.firstChild();
-                               while (!PAGE.isNull())
-                               {
-                                       QDomElement pg=PAGE.toElement();
-                                       if (pg.tagName()=="Bookmark")
+                               pageNode = docNode.firstChild();
+                               while (!pageNode.isNull())
+                               {
+                                       QDomElement pg = pageNode.toElement();
+                                       if (pg.tagName() == "Bookmark")
                                        {
                                                int elem = 
pg.attribute("Element").toInt();
                                                if (elem < 
m_Doc->Items->count())
@@ -2235,13 +2236,13 @@
                                                        
m_Doc->BookMarks.append(bok);
                                                }
                                        }
-                                       PAGE=PAGE.nextSibling();
+                                       pageNode = pageNode.nextSibling();
                                }
                                return true;
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return false;
 }
@@ -2319,11 +2320,11 @@
        else
        {
                QList<ParagraphStyle::TabRecord> tbs;
-               QDomNode IT = pg->firstChild();
-               while (!IT.isNull())
-               {
-                       QDomElement it = IT.toElement();
-                       if (it.tagName()=="Tabs")
+               QDomNode node = pg->firstChild();
+               while (!node.isNull())
+               {
+                       QDomElement it = node.toElement();
+                       if (it.tagName() == "Tabs")
                        {
                                ParagraphStyle::TabRecord tb;
                                tb.tabPosition = 
ScCLocale::toDoubleC(it.attribute("Pos"));
@@ -2337,7 +2338,7 @@
                                tbs.append(tb);
                        }
                        vg->setTabValues(tbs);
-                       IT = IT.nextSibling();
+                       node = node.nextSibling();
                }
        }
        for (int i = 0; i < docParagraphStyles.count(); ++i)
@@ -2407,25 +2408,25 @@
                return false;
        if (!docu.setContent(f))
                return false;
-       QDomElement elem=docu.documentElement();
+       QDomElement elem = docu.documentElement();
        if ((elem.tagName() != "SCRIBUS") && (elem.tagName() != "SCRIBUSUTF8"))
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="STYLE")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "STYLE")
                        {
                                pstyle.erase();
                                GetStyle(&pg, &pstyle, &docParagraphStyles, 
doc, false);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
@@ -2441,18 +2442,18 @@
        QDomElement elem=docu.documentElement();
        if ((elem.tagName() != "SCRIBUS") && (elem.tagName() != "SCRIBUSUTF8"))
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="MultiLine")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -2480,9 +2481,9 @@
                                }
                                Sty->insert(Nam2, ml);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
@@ -2501,16 +2502,16 @@
        QDomElement elem=docu.documentElement();
        if ((elem.tagName() != "SCRIBUS") && (elem.tagName() != "SCRIBUSUTF8"))
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
                        // 10/25/2004 pv - None is "reserved" color. cannot be 
defined in any file...
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -2526,16 +2527,16 @@
                                        lf.setRegistrationColor(false);
                                colors.insert(pg.attribute("NAME"), lf);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
 
 bool Scribus12Format::readPageCount(const QString& fileName, int *num1, int 
*num2, QStringList & masterPageNames)
 {
-       QString PgNam;
+       QString pageName;
        int counter = 0;
        int counter2 = 0;
        QDomDocument docu("scridoc");
@@ -2547,27 +2548,27 @@
        QDomElement elem=docu.documentElement();
        if ((elem.tagName() != "SCRIBUS") && (elem.tagName() != "SCRIBUSUTF8"))
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="PAGE")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "PAGE")
                        {
-                               PgNam = pg.attribute("NAM", "");
-                               if (PgNam.isEmpty())
+                               pageName = pg.attribute("NAM", "");
+                               if (pageName.isEmpty())
                                        counter++;
                                else
                                {
                                        counter2++;
-                                       masterPageNames.append(PgNam);
-                               }
-                       }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                                       masterPageNames.append(pageName);
+                               }
+                       }
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        *num1 = counter;
        *num2 = counter2;

Modified: 
trunk/Scribus/scribus/plugins/fileloader/scribus134format/scribus134format.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=23241&path=/trunk/Scribus/scribus/plugins/fileloader/scribus134format/scribus134format.cpp
==============================================================================
--- 
trunk/Scribus/scribus/plugins/fileloader/scribus134format/scribus134format.cpp  
    (original)
+++ 
trunk/Scribus/scribus/plugins/fileloader/scribus134format/scribus134format.cpp  
    Wed Oct  9 12:59:57 2019
@@ -1271,7 +1271,7 @@
        if (attrs.hasAttribute(LANGUAGE))
        {
                QString l(attrs.valueAsString(LANGUAGE));
-               if (LanguageManager::instance()->langTableIndex(l)!=-1)
+               if (LanguageManager::instance()->langTableIndex(l) != -1)
                        newStyle.setLanguage(l); //new style storage
                else
                { //old style storage

Modified: 
trunk/Scribus/scribus/plugins/fileloader/scribus13format/scribus13format.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=23241&path=/trunk/Scribus/scribus/plugins/fileloader/scribus13format/scribus13format.cpp
==============================================================================
--- 
trunk/Scribus/scribus/plugins/fileloader/scribus13format/scribus13format.cpp    
    (original)
+++ 
trunk/Scribus/scribus/plugins/fileloader/scribus13format/scribus13format.cpp    
    Wed Oct  9 12:59:57 2019
@@ -176,7 +176,7 @@
        ParagraphStyle vg;
        struct ScribusDoc::BookMa bok;
        int counter;//, Pgc;
-       QString tmp, PgNam, Defont;
+       QString tmp, pageName, Defont;
        QMap<int,int> TableID;
        QList<PageItem*> TableItems;
        QMap<int,int> TableIDM;
@@ -184,7 +184,7 @@
        QMap<int,int> TableIDF;
        QList<PageItem*> TableItemsF;
        int a;
-       PageItem *Neu;
+       PageItem *newItem;
        ScPage* Apage;
        FrameItems.clear();
        itemRemap.clear();
@@ -221,10 +221,10 @@
        QDomElement elem=docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
+       QDomNode docNode = elem.firstChild();
        if (m_mwProgressBar!=nullptr)
        {
-               m_mwProgressBar->setMaximum(DOC.childNodes().count());
+               m_mwProgressBar->setMaximum(docNode.childNodes().count());
                m_mwProgressBar->setValue(0);
        }
        // Stop  autosave timer,it will be started only if doc has autosave 
feature is enabled
@@ -238,9 +238,9 @@
        TableItemsF.clear();
        TableIDF.clear();
        PrefsManager& prefsManager=PrefsManager::instance();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
        /*
        * Attribute von DOCUMENT auslesen
        */
@@ -260,21 +260,21 @@
                        else
                                fp = 1;
                }
-               if (DOC.namedItem("PageSets").isNull())
+               if (docNode.namedItem("PageSets").isNull())
                {
                        m_Doc->setPageSetFirstPage(m_Doc->pagePositioning(), 
fp);
                }
                
m_Doc->setUsesAutomaticTextFrames(dc.attribute("AUTOTEXT").toInt());
                m_Doc->PageSp = dc.attribute("AUTOSPALTEN").toInt();
-               m_Doc->PageSpa= 
ScCLocale::toDoubleC(dc.attribute("ABSTSPALTEN"));
+               m_Doc->PageSpa = 
ScCLocale::toDoubleC(dc.attribute("ABSTSPALTEN"));
                m_Doc->setUnitIndex(dc.attribute("UNITS", "0").toInt());
-               
m_Doc->itemToolPrefs().textSize=qRound(ScCLocale::toDoubleC(dc.attribute("DSIZE"))
 * 10);
-               Defont=dc.attribute("DFONT");
+               m_Doc->itemToolPrefs().textSize = 
qRound(ScCLocale::toDoubleC(dc.attribute("DSIZE")) * 10);
+               Defont = dc.attribute("DFONT");
                //findFont will use that if it doesnt find the font:
                m_Doc->itemToolPrefs().textFont = 
prefsManager.appPrefs.itemToolPrefs.textFont;
                m_AvailableFonts->findFont(Defont, m_Doc);
                m_Doc->itemToolPrefs().textFont = Defont;
-               m_Doc->itemToolPrefs().textColumns= dc.attribute("DCOL", 
"1").toInt();
+               m_Doc->itemToolPrefs().textColumns = dc.attribute("DCOL", 
"1").toInt();
                m_Doc->itemToolPrefs().textColumnGap = 
ScCLocale::toDoubleC(dc.attribute("DGAP"), 0.0);
                DocumentInformation di;
                di.setAuthor(dc.attribute("AUTHOR"));
@@ -328,7 +328,7 @@
                //m_Doc->setHyphLanguage(dc.attribute("LANGUAGE", ""));
                static const QString LANGUAGE("LANGUAGE");
                QString l(dc.attribute(LANGUAGE, "en"));
-               if (LanguageManager::instance()->langTableIndex(l)!=-1)
+               if (LanguageManager::instance()->langTableIndex(l) != -1)
                        m_Doc->setLanguage(l); //new style storage
                else
                { //old style storage
@@ -450,17 +450,17 @@
                        
m_Doc->setCurCheckProfile(dc.attribute("currentProfile"));
                }
                m_Doc->LastAuto = nullptr;
-               QDomNode PAGE=DOC.firstChild();
+               QDomNode pageNode = docNode.firstChild();
                counter = 0;
-               while (!PAGE.isNull())
+               while (!pageNode.isNull())
                {
                        ObCount++;
                        if (m_mwProgressBar!=nullptr)
                                m_mwProgressBar->setValue(ObCount);
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="PageSets")
-                       {
-                               QDomNode PGS = PAGE.firstChild();
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "PageSets")
+                       {
+                               QDomNode PGS = pageNode.firstChild();
                                m_Doc->clearPageSets();
                                while (!PGS.isNull())
                                {
@@ -491,7 +491,7 @@
                                        PGS = PGS.nextSibling();
                                }
                        }
-                       if (pg.tagName()=="CheckProfile")
+                       if (pg.tagName() == "CheckProfile")
                        {
                                struct CheckerPrefs checkerSettings;
                                checkerSettings.ignoreErrors = 
static_cast<bool>(pg.attribute("ignoreErrors", "0").toInt());
@@ -512,7 +512,7 @@
                                m_Doc->set1CheckerProfile(pg.attribute("Name"), 
checkerSettings);
                        }
                        // 10/25/2004 pv - None is "reserved" color. cannot be 
defined in any file...
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -529,7 +529,7 @@
                                QString name = pg.attribute("NAME");
                                m_Doc->PageColors.insert((name.isEmpty()) ? 
lf.name() : name, lf);
                        }
-                       if (pg.tagName()=="STYLE")
+                       if (pg.tagName() == "STYLE")
                        {
                                readParagraphStyle(vg, pg, m_Doc);
                                StyleSet<ParagraphStyle> temp;
@@ -538,9 +538,9 @@
                                DoVorl[VorlC] = vg.name();
                                ++VorlC;
                        }
-                       if (pg.tagName()=="JAVA")
+                       if (pg.tagName() == "JAVA")
                                m_Doc->JavaScripts[pg.attribute("NAME")] = 
pg.attribute("SCRIPT");
-                       if (pg.tagName()=="LAYERS")
+                       if (pg.tagName() == "LAYERS")
                        {
                                int lId   = pg.attribute("NUMMER").toInt();
                                int level = pg.attribute("LEVEL").toInt();
@@ -556,7 +556,7 @@
                                        la.markerColor =  
QColor(pg.attribute("LAYERC","#000000"));
                                m_Doc->Layers.append(la);
                        }
-/*                     if (pg.tagName()=="Bookmark")
+/*                     if (pg.tagName() == "Bookmark")
                        {
                                bok.Title = pg.attribute("Title");
                                bok.Text = pg.attribute("Text");
@@ -571,10 +571,10 @@
                                bok.Parent = pg.attribute("Parent").toInt();
                                m_Doc->BookMarks.append(bok);
                        } */
-                       if (pg.tagName()=="MultiLine")
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -590,7 +590,7 @@
                                }
                                m_Doc->MLineStyles.insert(pg.attribute("Name"), 
ml);
                        }
-                       if (pg.tagName()=="Arrows")
+                       if (pg.tagName() == "Arrows")
                        {
                                struct ArrowDesc arrow;
                                arrow.name = pg.attribute("Name");
@@ -606,7 +606,7 @@
                                }
                                m_Doc->appendToArrowStyles(arrow);
                        }
-                       if (pg.tagName()=="PDF")
+                       if (pg.tagName() == "PDF")
                        {
                                m_Doc->pdfOptions().Articles = 
static_cast<bool>(pg.attribute("Articles").toInt());
                                m_Doc->pdfOptions().Thumbnails = 
static_cast<bool>(pg.attribute("Thumbnails").toInt());
@@ -671,7 +671,7 @@
                                m_Doc->pdfOptions().fitWindow = 
static_cast<bool>(pg.attribute("fitWindow", "0").toInt());
                                m_Doc->pdfOptions().PageLayout = 
pg.attribute("PageLayout", "0").toInt();
                                m_Doc->pdfOptions().openAction = 
pg.attribute("openAction", "");
-                               QDomNode PFO = PAGE.firstChild();
+                               QDomNode PFO = pageNode.firstChild();
                                while (!PFO.isNull())
                                {
                                        QDomElement pdfF = PFO.toElement();
@@ -707,9 +707,9 @@
                                        PFO = PFO.nextSibling();
                                }
                        }
-                       if (pg.tagName()=="DocItemAttributes")
-                       {
-                               QDomNode DIA = PAGE.firstChild();
+                       if (pg.tagName() == "DocItemAttributes")
+                       {
+                               QDomNode DIA = pageNode.firstChild();
                                m_Doc->clearItemAttributes();
                                while (!DIA.isNull())
                                {
@@ -729,9 +729,9 @@
                                        DIA = DIA.nextSibling();
                                }
                        }
-                       if (pg.tagName()=="TablesOfContents")
-                       {
-                               QDomNode TOC = PAGE.firstChild();
+                       if (pg.tagName() == "TablesOfContents")
+                       {
+                               QDomNode TOC = pageNode.firstChild();
                                m_Doc->clearTocSetups();
                                while (!TOC.isNull())
                                {
@@ -756,9 +756,9 @@
                                        TOC = TOC.nextSibling();
                                }
                        }
-                       if (pg.tagName()=="Sections")
-                       {
-                               QDomNode Section = PAGE.firstChild();
+                       if (pg.tagName() == "Sections")
+                       {
+                               QDomNode Section = pageNode.firstChild();
                                while (!Section.isNull())
                                {
                                        QDomElement sectionElem = 
Section.toElement();
@@ -769,15 +769,15 @@
                                                
newSection.name=sectionElem.attribute("Name");
                                                
newSection.fromindex=sectionElem.attribute("From").toInt();
                                                
newSection.toindex=sectionElem.attribute("To").toInt();
-                                               if 
(sectionElem.attribute("Type")=="Type_1_2_3")
+                                               if 
(sectionElem.attribute("Type") == "Type_1_2_3")
                                                        
newSection.type=Type_1_2_3;
-                                               if 
(sectionElem.attribute("Type")=="Type_i_ii_iii")
+                                               if 
(sectionElem.attribute("Type") == "Type_i_ii_iii")
                                                        
newSection.type=Type_i_ii_iii;
-                                               if 
(sectionElem.attribute("Type")=="Type_I_II_III")
+                                               if 
(sectionElem.attribute("Type") == "Type_I_II_III")
                                                        
newSection.type=Type_I_II_III;
-                                               if 
(sectionElem.attribute("Type")=="Type_a_b_c")
+                                               if 
(sectionElem.attribute("Type") == "Type_a_b_c")
                                                        
newSection.type=Type_a_b_c;
-                                               if 
(sectionElem.attribute("Type")=="Type_A_B_C")
+                                               if 
(sectionElem.attribute("Type") == "Type_A_B_C")
                                                        
newSection.type=Type_A_B_C;
                                                
newSection.sectionstartindex=sectionElem.attribute("Start").toInt();
                                                
newSection.reversed=static_cast<bool>(sectionElem.attribute("Reversed").toInt());
@@ -788,25 +788,25 @@
                                        Section = Section.nextSibling();
                                }
                        }
-                       if ((pg.tagName()=="PAGE") || 
(pg.tagName()=="MASTERPAGE"))
+                       if ((pg.tagName() == "PAGE") || (pg.tagName() == 
"MASTERPAGE"))
                        {
                                a = pg.attribute("NUM").toInt();
-                               PgNam = "";
-                               PgNam = pg.attribute("NAM", "");
-                               if (pg.tagName()=="MASTERPAGE" && 
PgNam.isEmpty())
+                               pageName = "";
+                               pageName = pg.attribute("NAM", "");
+                               if (pg.tagName() == "MASTERPAGE" && 
pageName.isEmpty())
                                {
                                        qDebug() << "scribus13format: corrupted 
masterpage with empty name detected";
-                                       PAGE=PAGE.nextSibling();
+                                       pageNode = pageNode.nextSibling();
                                        continue;
                                }
-                               if (PgNam.isEmpty())
+                               if (pageName.isEmpty())
                                        m_Doc->setMasterPageMode(false);
                                else
                                        m_Doc->setMasterPageMode(true);
-                               if (PgNam.isEmpty())
+                               if (pageName.isEmpty())
                                        Apage = m_Doc->addPage(a);
                                else
-                                       Apage = m_Doc->addMasterPage(a, PgNam);
+                                       Apage = m_Doc->addMasterPage(a, 
pageName);
                                Apage->LeftPg=pg.attribute("LEFT", "0").toInt();
                                QString Mus = "";
                                Mus = pg.attribute("MNAM","Normal");
@@ -852,20 +852,20 @@
                                                GuideManagerCore::Standard,
                                                pg.hasAttribute("NumHGuides"));
                        }
-                       if ((pg.tagName()=="PAGEOBJECT") || 
(pg.tagName()=="MASTEROBJECT") || (pg.tagName()=="FRAMEOBJECT"))
-                       {
-                               if ((pg.tagName()=="PAGEOBJECT") || 
(pg.tagName()=="FRAMEOBJECT"))
+                       if ((pg.tagName() == "PAGEOBJECT") || (pg.tagName() == 
"MASTEROBJECT") || (pg.tagName() == "FRAMEOBJECT"))
+                       {
+                               if ((pg.tagName() == "PAGEOBJECT") || 
(pg.tagName() == "FRAMEOBJECT"))
                                        m_Doc->setMasterPageMode(false);
                                else
                                        m_Doc->setMasterPageMode(true);
                                int pagenr = -1;
-                               if ((!pg.attribute("OnMasterPage").isEmpty()) 
&& (pg.tagName()=="MASTEROBJECT"))
+                               if ((!pg.attribute("OnMasterPage").isEmpty()) 
&& (pg.tagName() == "MASTEROBJECT"))
                                {
                                        
m_Doc->setCurrentPage(m_Doc->MasterPages.at(m_Doc->MasterNames[pg.attribute("OnMasterPage")]));
                                        pagenr = -2;
                                }
 
-                               if (pg.tagName()=="PAGEOBJECT")
+                               if (pg.tagName() == "PAGEOBJECT")
                                {
                                        itemRemap[itemCount++] = 
m_Doc->Items->count();
                                        // member of linked chain?
@@ -877,106 +877,106 @@
                                PageItem::ItemKind itemKind = 
PageItem::StandardItem;
                                if (pg.tagName() == "FRAMEOBJECT")
                                        itemKind = PageItem::InlineItem;
-                               Neu = PasteItem(&pg, m_Doc, fileDir, itemKind, 
pagenr);
-                               Neu->setRedrawBounding();
-                               if (pg.tagName()=="MASTEROBJECT")
-                                       Neu->setOwnerPage(m_Doc->OnPage(Neu));
-                               else
-                                       
Neu->setOwnerPage(pg.attribute("OwnPage").toInt());
+                               newItem = PasteItem(&pg, m_Doc, fileDir, 
itemKind, pagenr);
+                               newItem->setRedrawBounding();
+                               if (pg.tagName() == "MASTEROBJECT")
+                                       
newItem->setOwnerPage(m_Doc->OnPage(newItem));
+                               else
+                                       
newItem->setOwnerPage(pg.attribute("OwnPage").toInt());
                                if (pg.tagName() == "PAGEOBJECT")
-                                       Neu->setMasterPageName(QString());
-                               QDomNode IT=pg.firstChild();
+                                       newItem->setMasterPageName(QString());
+                               QDomNode node = pg.firstChild();
                                LastStyles * last = new LastStyles();
-                               while (!IT.isNull())
-                               {
-                                       QDomElement it=IT.toElement();
-                                       if (it.tagName()=="CSTOP")
+                               while (!node.isNull())
+                               {
+                                       QDomElement it = node.toElement();
+                                       if (it.tagName() == "CSTOP")
                                        {
                                                QString name = 
it.attribute("NAME");
                                                double ramp = 
ScCLocale::toDoubleC(it.attribute("RAMP"), 0.0);
                                                int shade   = 
it.attribute("SHADE", "100").toInt();
                                                double opa  = 
ScCLocale::toDoubleC(it.attribute("TRANS"), 1.0);
                                                handleOldColorShade(m_Doc, 
name, shade);
-                                               
Neu->fill_gradient.addStop(SetColor(m_Doc, name, shade), ramp, 0.5, opa, name, 
shade);
+                                               
newItem->fill_gradient.addStop(SetColor(m_Doc, name, shade), ramp, 0.5, opa, 
name, shade);
                                        }
-                                       if (it.tagName()=="ITEXT")
-                                               GetItemText(&it, m_Doc, Neu, 
last);
+                                       if (it.tagName() == "ITEXT")
+                                               GetItemText(&it, m_Doc, 
newItem, last);
 
                                        //CB PageItemAttributes
-                                       if (it.tagName()=="PageItemAttributes")
+                                       if (it.tagName() == 
"PageItemAttributes")
                                        {
-                                               QDomNode PIA = it.firstChild();
+                                               QDomNode pia = it.firstChild();
                                                ObjAttrVector 
pageItemAttributes;
-                                               while (!PIA.isNull())
+                                               while (!pia.isNull())
                                                {
-                                                       QDomElement itemAttr = 
PIA.toElement();
+                                                       QDomElement itemAttr = 
pia.toElement();
                                                        if (itemAttr.tagName() 
== "ItemAttribute")
                                                        {
                                                                ObjectAttribute 
objattr;
-                                                               
objattr.name=itemAttr.attribute("Name");
-                                                               
objattr.type=itemAttr.attribute("Type");
-                                                               
objattr.value=itemAttr.attribute("Value");
-                                                               
objattr.parameter=itemAttr.attribute("Parameter");
-                                                               
objattr.relationship=itemAttr.attribute("Relationship");
-                                                               
objattr.relationshipto=itemAttr.attribute("RelationshipTo");
-                                                               
objattr.autoaddto=itemAttr.attribute("AutoAddTo");
+                                                               objattr.name = 
itemAttr.attribute("Name");
+                                                               objattr.type = 
itemAttr.attribute("Type");
+                                                               objattr.value = 
itemAttr.attribute("Value");
+                                                               
objattr.parameter = itemAttr.attribute("Parameter");
+                                                               
objattr.relationship = itemAttr.attribute("Relationship");
+                                                               
objattr.relationshipto = itemAttr.attribute("RelationshipTo");
+                                                               
objattr.autoaddto = itemAttr.attribute("AutoAddTo");
                                                                
pageItemAttributes.append(objattr);
                                                        }
-                                                       PIA = PIA.nextSibling();
+                                                       pia = pia.nextSibling();
                                                }
-                                               
Neu->setObjectAttributes(&pageItemAttributes);
+                                               
newItem->setObjectAttributes(&pageItemAttributes);
                                        }
-                                       IT=IT.nextSibling();
+                                       node = node.nextSibling();
                                }
                                delete last;
-                               if (Neu->fill_gradient.stops() == 0)
+                               if (newItem->fill_gradient.stops() == 0)
                                {
                                        const ScColor& col1 = 
m_Doc->PageColors[m_Doc->itemToolPrefs().shapeFillColor];
                                        const ScColor& col2 = 
m_Doc->PageColors[m_Doc->itemToolPrefs().shapeLineColor];
-                                       
Neu->fill_gradient.addStop(ScColorEngine::getRGBColor(col1, m_Doc), 0.0, 0.5, 
1.0, m_Doc->itemToolPrefs().shapeFillColor, 100);
-                                       
Neu->fill_gradient.addStop(ScColorEngine::getRGBColor(col2, m_Doc), 1.0, 0.5, 
1.0, m_Doc->itemToolPrefs().shapeLineColor, 100);
-                               }
-                               Neu->isAutoText = 
static_cast<bool>(pg.attribute("AUTOTEXT").toInt());
-                               Neu->isEmbedded = 
static_cast<bool>(pg.attribute("isInline", "0").toInt());
-                               Neu->gXpos = 
ScCLocale::toDoubleC(pg.attribute("gXpos"), 0.0);
-                               Neu->gYpos = 
ScCLocale::toDoubleC(pg.attribute("gYpos"), 0.0);
-                               Neu->gWidth  = 
ScCLocale::toDoubleC(pg.attribute("gWidth"), Neu->width());
-                               Neu->gHeight = 
ScCLocale::toDoubleC(pg.attribute("gHeight"), Neu->height());
-                               if (Neu->isAutoText)
-                                       m_Doc->LastAuto = Neu;
-                               if (pg.tagName()=="FRAMEOBJECT")
-                               {
-                                       
FrameItems.append(m_Doc->Items->takeAt(m_Doc->Items->indexOf(Neu)));
-                                       Neu->setLayer(m_Doc->firstLayerID());
-                               }
-                               if (Neu->isTableItem)
-                               {
-                                       if (pg.tagName()=="PAGEOBJECT")
+                                       
newItem->fill_gradient.addStop(ScColorEngine::getRGBColor(col1, m_Doc), 0.0, 
0.5, 1.0, m_Doc->itemToolPrefs().shapeFillColor, 100);
+                                       
newItem->fill_gradient.addStop(ScColorEngine::getRGBColor(col2, m_Doc), 1.0, 
0.5, 1.0, m_Doc->itemToolPrefs().shapeLineColor, 100);
+                               }
+                               newItem->isAutoText = 
static_cast<bool>(pg.attribute("AUTOTEXT").toInt());
+                               newItem->isEmbedded = 
static_cast<bool>(pg.attribute("isInline", "0").toInt());
+                               newItem->gXpos = 
ScCLocale::toDoubleC(pg.attribute("gXpos"), 0.0);
+                               newItem->gYpos = 
ScCLocale::toDoubleC(pg.attribute("gYpos"), 0.0);
+                               newItem->gWidth  = 
ScCLocale::toDoubleC(pg.attribute("gWidth"), newItem->width());
+                               newItem->gHeight = 
ScCLocale::toDoubleC(pg.attribute("gHeight"), newItem->height());
+                               if (newItem->isAutoText)
+                                       m_Doc->LastAuto = newItem;
+                               if (pg.tagName() == "FRAMEOBJECT")
+                               {
+                                       
FrameItems.append(m_Doc->Items->takeAt(m_Doc->Items->indexOf(newItem)));
+                                       
newItem->setLayer(m_Doc->firstLayerID());
+                               }
+                               if (newItem->isTableItem)
+                               {
+                                       if (pg.tagName() == "PAGEOBJECT")
                                        {
-                                               TableItems.append(Neu);
-                                               
TableID.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->DocItems.indexOf(Neu));
+                                               TableItems.append(newItem);
+                                               
TableID.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->DocItems.indexOf(newItem));
                                        }
-                                       else if (pg.tagName()=="FRAMEOBJECT")
+                                       else if (pg.tagName() == "FRAMEOBJECT")
                                        {
-                                               TableItemsF.append(Neu);
-                                               
TableIDF.insert(pg.attribute("OwnLINK", "0").toInt(), FrameItems.indexOf(Neu));
+                                               TableItemsF.append(newItem);
+                                               
TableIDF.insert(pg.attribute("OwnLINK", "0").toInt(), 
FrameItems.indexOf(newItem));
                                        }
                                        else
                                        {
-                                               TableItemsM.append(Neu);
-                                               
TableIDM.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->MasterItems.indexOf(Neu));
+                                               TableItemsM.append(newItem);
+                                               
TableIDM.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->MasterItems.indexOf(newItem));
                                        }
                                }
                                m_Doc->setMasterPageMode(false);
                                counter++;
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="Bookmark")
+                       pageNode = pageNode.nextSibling();
+               }
+               pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "Bookmark")
                        {
                                int elem = pg.attribute("Element").toInt();
                                if (elem < m_Doc->Items->count())
@@ -994,9 +994,9 @@
                                        m_Doc->BookMarks.append(bok);
                                }
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        if (TableItemsF.count() != 0)
        {
@@ -1190,7 +1190,7 @@
 //             m_Doc->autoSaveTimer->start(m_Doc->autoSaveTime());
 
        if (m_mwProgressBar!=nullptr)
-               m_mwProgressBar->setValue(DOC.childNodes().count());
+               m_mwProgressBar->setValue(docNode.childNodes().count());
 
        ScMessageBox::warning(ScCore->primaryMainWindow(),
                                                  CommonStrings::trWarning,
@@ -1449,11 +1449,11 @@
        {
                QList<ParagraphStyle::TabRecord> tbs;
                vg.setTabValues(tbs);
-               QDomNode IT = pg.firstChild();
-               while (!IT.isNull())
-               {
-                       QDomElement it = IT.toElement();
-                       if (it.tagName()=="Tabs")
+               QDomNode node = pg.firstChild();
+               while (!node.isNull())
+               {
+                       QDomElement it = node.toElement();
+                       if (it.tagName() == "Tabs")
                        {
                                ParagraphStyle::TabRecord tb;
                                tb.tabPosition = 
ScCLocale::toDoubleC(it.attribute("Pos"));
@@ -1467,7 +1467,7 @@
                                tbs.append(tb);
                        }
 //                     vg.setTabValues(tbs);
-                       IT=IT.nextSibling();
+                       node = node.nextSibling();
                }
                if (tbs.count() > 0)
                        vg.setTabValues(tbs);
@@ -1504,7 +1504,7 @@
        QString tmp;
        double xf, yf, xf2;
        QString clPath;
-       QDomNode IT;
+       QDomNode node;
        switch (pt)
        {
        // OBSOLETE CR 2005-02-06
@@ -1532,18 +1532,18 @@
                currItem->IRender   = (eRenderIntent) obj->attribute("IRENDER", 
"1").toInt();
                currItem->UseEmbedded = obj->attribute("EMBEDDED", "1").toInt();
                currItem->pixm.imgInfo.lowResType = obj->attribute("ImageRes", 
"1").toInt();
-               IT = obj->firstChild();
-               while (!IT.isNull())
-               {
-                       QDomElement it = IT.toElement();
-                       if (it.tagName()=="ImageEffect")
+               node = obj->firstChild();
+               while (!node.isNull())
+               {
+                       QDomElement it = node.toElement();
+                       if (it.tagName() == "ImageEffect")
                        {
                                struct ImageEffect ef;
                                ef.effectParameters = it.attribute("Param");
                                ef.effectCode = it.attribute("Code").toInt();
                                currItem->effectsInUse.append(ef);
                        }
-                       IT=IT.nextSibling();
+                       node = node.nextSibling();
                }
                if (!currItem->Pfile.isEmpty())
                        doc->loadPict(currItem->Pfile, currItem, false);
@@ -1569,10 +1569,10 @@
                if (currItem->pixm.imgInfo.layerInfo.count() != 0)
                {
                        bool found = false;
-                       IT = obj->firstChild();
-                       while (!IT.isNull())
-                       {
-                               QDomElement it = IT.toElement();
+                       node = obj->firstChild();
+                       while (!node.isNull())
+                       {
+                               QDomElement it = node.toElement();
                                if (it.tagName() == "PSDLayer")
                                {
                                        found = true;
@@ -1582,7 +1582,7 @@
                                        loadingInfo.useMask = 
static_cast<bool>(it.attribute("useMask", "1").toInt());
                                        
currItem->pixm.imgInfo.RequestProps.insert(it.attribute("Layer").toInt(), 
loadingInfo);
                                }
-                               IT=IT.nextSibling();
+                               node = node.nextSibling();
                        }
                        if (found)
                        {
@@ -1724,11 +1724,11 @@
        }
        else
        {
-               IT = obj->firstChild();
-               while (!IT.isNull())
-               {
-                       QDomElement it = IT.toElement();
-                       if (it.tagName()=="Tabs")
+               node = obj->firstChild();
+               while (!node.isNull())
+               {
+                       QDomElement it = node.toElement();
+                       if (it.tagName() == "Tabs")
                        {
                                ParagraphStyle::TabRecord tb;
                                tb.tabPosition = 
ScCLocale::toDoubleC(it.attribute("Pos"));
@@ -1741,7 +1741,7 @@
                                        tb.tabFillChar = tbCh[0];
                                tbValues.append(tb);
                        }
-                       IT=IT.nextSibling();
+                       node = node.nextSibling();
                }
        }
        if (!tbValues.empty())
@@ -2033,7 +2033,7 @@
        }
        ParagraphStyle vg;
        struct ScribusDoc::BookMa bok;
-       PageItem *Neu;
+       PageItem *newItem;
        ScPage* Apage = nullptr;
        FrameItems.clear();
        itemRemap.clear();
@@ -2080,19 +2080,19 @@
        }
        QString fileDir = QFileInfo(fileName).absolutePath();
        ScColor lf = ScColor();
-       QDomElement elem=docu.documentElement();
+       QDomElement elem = docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
+       QDomNode docNode = elem.firstChild();
        counter = m_Doc->Items->count();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -2109,14 +2109,14 @@
                                QString name = pg.attribute("NAME");
                                m_Doc->PageColors.insert((name.isEmpty()) ? 
lf.name() : name, lf);
                        }
-                       if (pg.tagName()=="STYLE")
+                       if (pg.tagName() == "STYLE")
                        {
                                GetStyle(&pg, &vg, nullptr, m_Doc, true);
                                VorLFound = true;
                        }
-                       if (pg.tagName()=="JAVA")
+                       if (pg.tagName() == "JAVA")
                                m_Doc->JavaScripts[pg.attribute("NAME")] = 
pg.attribute("SCRIPT");
-                       if (pg.tagName()=="LAYERS")
+                       if (pg.tagName() == "LAYERS")
                        {
                                int lId   = pg.attribute("NUMMER").toInt();
                                int level = pg.attribute("LEVEL").toInt();
@@ -2143,10 +2143,10 @@
                                        m_Doc->Layers.append(la);
                                }
                        }
-                       if (pg.tagName()=="MultiLine")
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -2174,7 +2174,7 @@
                                }
                                m_Doc->MLineStyles.insert(Nam2, ml);
                        }
-                       if (pg.tagName()=="Arrows")
+                       if (pg.tagName() == "Arrows")
                        {
                                struct ArrowDesc arrow;
                                arrow.name = pg.attribute("Name");
@@ -2190,13 +2190,13 @@
                                }
                                m_Doc->appendToArrowStyles(arrow);
                        }
-                       if (((pg.tagName()=="PAGE") || 
(pg.tagName()=="MASTERPAGE")) && (pg.attribute("NUM").toInt() == pageNumber))
+                       if (((pg.tagName() == "PAGE") || (pg.tagName() == 
"MASTERPAGE")) && (pg.attribute("NUM").toInt() == pageNumber))
                        {
                                a = m_Doc->currentPage()->pageNr();
                                Apage = m_Doc->Pages->at(a);
-                               if ((pg.tagName()!="MASTERPAGE") && (Mpage))
-                               {
-                                       PAGE=PAGE.nextSibling();
+                               if ((pg.tagName() != "MASTERPAGE") && (Mpage))
+                               {
+                                       pageNode = pageNode.nextSibling();
                                        continue;
                                }
                                if (Mpage)
@@ -2239,13 +2239,13 @@
                                                                                
                GuideManagerCore::Standard,
                                                                                
                pg.hasAttribute("NumHGuides"));
                        }
-                       if ((pg.tagName()=="PAGEOBJECT") || 
(pg.tagName()=="MASTEROBJECT") || (pg.tagName()=="FRAMEOBJECT"))
+                       if ((pg.tagName() == "PAGEOBJECT") || (pg.tagName() == 
"MASTEROBJECT") || (pg.tagName() == "FRAMEOBJECT"))
                        {
                                if (Mpage)
                                {
                                        if (pg.tagName() != "MASTEROBJECT")
                                        {
-                                               PAGE=PAGE.nextSibling();
+                                               pageNode = 
pageNode.nextSibling();
                                                continue;
                                        }
                                }
@@ -2253,20 +2253,20 @@
                                {
                                        if (pg.tagName() == "MASTEROBJECT")
                                        {
-                                               PAGE=PAGE.nextSibling();
+                                               pageNode = 
pageNode.nextSibling();
                                                continue;
                                        }
                                }
                                
                                if (pg.attribute("OwnPage").toInt() != 
pageNumber)
                                {                       
-                                       if (pg.tagName()=="PAGEOBJECT")
+                                       if (pg.tagName() == "PAGEOBJECT")
                                                itemRemap[itemCount++] = -1;
                                }
                                else
                                {
                                        // first of linked chain?
-                                       if (pg.tagName()=="PAGEOBJECT")
+                                       if (pg.tagName() == "PAGEOBJECT")
                                        {
                                                itemRemap[itemCount++] = 
m_Doc->Items->count();
                                                if 
(pg.attribute("NEXTITEM").toInt() != -1)
@@ -2279,99 +2279,99 @@
                                                itemKind = PageItem::InlineItem;
                                        /*int docGc = m_Doc->GroupCounter;
                                        m_Doc->GroupCounter = 0;*/
-                                       Neu = PasteItem(&pg, m_Doc, fileDir, 
itemKind);
-                                       Neu->moveBy(-pageX + Apage->xOffset(), 
- pageY + Apage->yOffset());
-                                       Neu->setRedrawBounding();
+                                       newItem = PasteItem(&pg, m_Doc, 
fileDir, itemKind);
+                                       newItem->moveBy(-pageX + 
Apage->xOffset(), - pageY + Apage->yOffset());
+                                       newItem->setRedrawBounding();
                                        //CB Must run onpage as we cant use 
pagetoload if the page has been renamed. 
                                        //CB TODO Make this accept a page to 
place onto.
-                                       //Neu->OwnPage = m_Doc->OnPage(Neu);
-                                       //Neu->OwnPage = PageToLoad;
-                                       
Neu->setOwnerPage(m_Doc->currentPageNumber());
+                                       //newItem->OwnPage = 
m_Doc->OnPage(newItem);
+                                       //newItem->OwnPage = PageToLoad;
+                                       
newItem->setOwnerPage(m_Doc->currentPageNumber());
                                        if (pg.tagName() == "PAGEOBJECT")
-                                               
Neu->setMasterPageName(QString());
+                                               
newItem->setMasterPageName(QString());
                                        else if (Mpage && 
!renamedPageName.isEmpty())
-                                               
Neu->setMasterPageName(renamedPageName);
-                                       
Neu->setLayer(layerTrans.value(Neu->m_layerID, Neu->m_layerID));
+                                               
newItem->setMasterPageName(renamedPageName);
+                                       
newItem->setLayer(layerTrans.value(newItem->m_layerID, newItem->m_layerID));
                                        /*m_Doc->GroupCounter = docGc;*/
-                                       QDomNode IT=pg.firstChild();
+                                       QDomNode node = pg.firstChild();
                                        LastStyles * last = new LastStyles();
-                                       while (!IT.isNull())
+                                       while (!node.isNull())
                                        {
-                                               QDomElement it=IT.toElement();
-                                               if (it.tagName()=="CSTOP")
+                                               QDomElement it = 
node.toElement();
+                                               if (it.tagName() == "CSTOP")
                                                {
                                                        QString name = 
it.attribute("NAME");
                                                        double ramp  = 
ScCLocale::toDoubleC(it.attribute("RAMP"), 0.0);
                                                        int shade  = 
it.attribute("SHADE", "100").toInt();
                                                        double opa = 
ScCLocale::toDoubleC(it.attribute("TRANS"), 1.0);
                                                        
handleOldColorShade(m_Doc, name, shade);
-                                                       
Neu->fill_gradient.addStop(SetColor(m_Doc, name, shade), ramp, 0.5, opa, name, 
shade);
+                                                       
newItem->fill_gradient.addStop(SetColor(m_Doc, name, shade), ramp, 0.5, opa, 
name, shade);
                                                }
-                                               if (it.tagName()=="ITEXT")
-                                                       GetItemText(&it, m_Doc, 
Neu, last, true, VorLFound);
-                                               if 
(it.tagName()=="PageItemAttributes")
+                                               if (it.tagName() == "ITEXT")
+                                                       GetItemText(&it, m_Doc, 
newItem, last, true, VorLFound);
+                                               if (it.tagName() == 
"PageItemAttributes")
                                                {
-                                                       QDomNode PIA = 
it.firstChild();
+                                                       QDomNode pia = 
it.firstChild();
                                                        ObjAttrVector 
pageItemAttributes;
-                                                       while (!PIA.isNull())
+                                                       while (!pia.isNull())
                                                        {
-                                                               QDomElement 
itemAttr = PIA.toElement();
+                                                               QDomElement 
itemAttr = pia.toElement();
                                                                if 
(itemAttr.tagName() == "ItemAttribute")
                                                                {
                                                                        
ObjectAttribute objattr;
-                                                                       
objattr.name=itemAttr.attribute("Name");
-                                                                       
objattr.type=itemAttr.attribute("Type");
-                                                                       
objattr.value=itemAttr.attribute("Value");
-                                                                       
objattr.parameter=itemAttr.attribute("Parameter");
-                                                                       
objattr.relationship=itemAttr.attribute("Relationship");
-                                                                       
objattr.relationshipto=itemAttr.attribute("RelationshipTo");
-                                                                       
objattr.autoaddto=itemAttr.attribute("AutoAddTo");
+                                                                       
objattr.name = itemAttr.attribute("Name");
+                                                                       
objattr.type = itemAttr.attribute("Type");
+                                                                       
objattr.value = itemAttr.attribute("Value");
+                                                                       
objattr.parameter = itemAttr.attribute("Parameter");
+                                                                       
objattr.relationship = itemAttr.attribute("Relationship");
+                                                                       
objattr.relationshipto = itemAttr.attribute("RelationshipTo");
+                                                                       
objattr.autoaddto = itemAttr.attribute("AutoAddTo");
                                                                        
pageItemAttributes.append(objattr);
                                                                }
-                                                               PIA = 
PIA.nextSibling();
+                                                               pia = 
pia.nextSibling();
                                                        }
-                                                       
Neu->setObjectAttributes(&pageItemAttributes);
+                                                       
newItem->setObjectAttributes(&pageItemAttributes);
                                                }
-                                               IT=IT.nextSibling();
+                                               node = node.nextSibling();
                                        }
                                        delete last;
-                                       if (Neu->fill_gradient.stops() == 0)
+                                       if (newItem->fill_gradient.stops() == 0)
                                        {
                                                const ScColor& col1 = 
m_Doc->PageColors[m_Doc->itemToolPrefs().shapeFillColor];
                                                const ScColor& col2 = 
m_Doc->PageColors[m_Doc->itemToolPrefs().shapeLineColor];
-                                               
Neu->fill_gradient.addStop(ScColorEngine::getRGBColor(col1, m_Doc), 0.0, 0.5, 
1.0, m_Doc->itemToolPrefs().shapeFillColor, 100);
-                                               
Neu->fill_gradient.addStop(ScColorEngine::getRGBColor(col2, m_Doc), 1.0, 0.5, 
1.0, m_Doc->itemToolPrefs().shapeLineColor, 100);
+                                               
newItem->fill_gradient.addStop(ScColorEngine::getRGBColor(col1, m_Doc), 0.0, 
0.5, 1.0, m_Doc->itemToolPrefs().shapeFillColor, 100);
+                                               
newItem->fill_gradient.addStop(ScColorEngine::getRGBColor(col2, m_Doc), 1.0, 
0.5, 1.0, m_Doc->itemToolPrefs().shapeLineColor, 100);
                                        }
-//                                     Neu->Language = 
ScMW->GetLang(pg.attribute("LANGUAGE", m_Doc->Language));
-//                                     Neu->Language = m_Doc->Language;
-                                       Neu->isAutoText = 
static_cast<bool>(pg.attribute("AUTOTEXT").toInt());
-                                       Neu->isEmbedded = 
static_cast<bool>(pg.attribute("isInline", "0").toInt());
-                                       Neu->gXpos = 
ScCLocale::toDoubleC(pg.attribute("gXpos"), 0.0);
-                                       Neu->gYpos = 
ScCLocale::toDoubleC(pg.attribute("gYpos"), 0.0);
-                                       Neu->gWidth  = 
ScCLocale::toDoubleC(pg.attribute("gWidth"), Neu->width());
-                                       Neu->gHeight = 
ScCLocale::toDoubleC(pg.attribute("gHeight"),Neu->height());
-                                       if (Neu->isAutoText)
-                                               m_Doc->LastAuto = Neu;
-                                       if (Neu->isTableItem)
+//                                     newItem->Language = 
ScMW->GetLang(pg.attribute("LANGUAGE", m_Doc->Language));
+//                                     newItem->Language = m_Doc->Language;
+                                       newItem->isAutoText = 
static_cast<bool>(pg.attribute("AUTOTEXT").toInt());
+                                       newItem->isEmbedded = 
static_cast<bool>(pg.attribute("isInline", "0").toInt());
+                                       newItem->gXpos = 
ScCLocale::toDoubleC(pg.attribute("gXpos"), 0.0);
+                                       newItem->gYpos = 
ScCLocale::toDoubleC(pg.attribute("gYpos"), 0.0);
+                                       newItem->gWidth  = 
ScCLocale::toDoubleC(pg.attribute("gWidth"), newItem->width());
+                                       newItem->gHeight = 
ScCLocale::toDoubleC(pg.attribute("gHeight"),newItem->height());
+                                       if (newItem->isAutoText)
+                                               m_Doc->LastAuto = newItem;
+                                       if (newItem->isTableItem)
                                        {
-                                               TableItems.append(Neu);
-                                               
TableID.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->Items->indexOf(Neu));
+                                               TableItems.append(newItem);
+                                               
TableID.insert(pg.attribute("OwnLINK", "0").toInt(), 
m_Doc->Items->indexOf(newItem));
                                        }
-                                       if (pg.tagName()=="FRAMEOBJECT")
+                                       if (pg.tagName() == "FRAMEOBJECT")
                                        {
-                                               
FrameItems.append(m_Doc->Items->takeAt(m_Doc->Items->indexOf(Neu)));
-                                               
Neu->setLayer(m_Doc->firstLayerID());
+                                               
FrameItems.append(m_Doc->Items->takeAt(m_Doc->Items->indexOf(newItem)));
+                                               
newItem->setLayer(m_Doc->firstLayerID());
                                        }
                                }
                                counter++;
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="Bookmark")
+                       pageNode = pageNode.nextSibling();
+               }
+               pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "Bookmark")
                        {
                                int elem = pg.attribute("Element").toInt();
                                if (elem < m_Doc->Items->count())
@@ -2389,9 +2389,9 @@
                                        m_Doc->BookMarks.append(bok);
                                }
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        if (TableItems.count() != 0)
        {
@@ -2617,22 +2617,22 @@
        QDomElement elem=docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="STYLE")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "STYLE")
                        {
                                pstyle.erase();
                                GetStyle(&pg, &pstyle, &docParagraphStyles, 
doc, false);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
@@ -2648,18 +2648,18 @@
        QDomElement elem=docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       if (pg.tagName()=="MultiLine")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       if (pg.tagName() == "MultiLine")
                        {
                                multiLine ml;
-                               QDomNode MuLn = PAGE.firstChild();
+                               QDomNode MuLn = pageNode.firstChild();
                                while (!MuLn.isNull())
                                {
                                        QDomElement MuL = MuLn.toElement();
@@ -2687,9 +2687,9 @@
                                }
                                Sty->insert(Nam2, ml);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
@@ -2707,16 +2707,16 @@
        QDomElement elem=docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomElement dc=DOC.toElement();
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomElement dc = docNode.toElement();
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
                        // 10/25/2004 pv - None is "reserved" color. cannot be 
defined in any file...
-                       if (pg.tagName()=="COLOR" && 
pg.attribute("NAME")!=CommonStrings::None)
+                       if (pg.tagName() == "COLOR" && pg.attribute("NAME") != 
CommonStrings::None)
                        {
                                if (pg.hasAttribute("CMYK"))
                                        lf.setNamedColor(pg.attribute("CMYK"));
@@ -2733,16 +2733,16 @@
                                QString name = pg.attribute("NAME");
                                colors.insert((name.isEmpty()) ? lf.name() : 
name, lf);
                        }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        return true;
 }
 
 bool Scribus13Format::readPageCount(const QString& fileName, int *num1, int 
*num2, QStringList & masterPageNames)
 {
-       QString PgNam;
+       QString pageName;
        int counter = 0;
        int counter2 = 0;
        QDomDocument docu("scridoc");
@@ -2751,27 +2751,27 @@
                return false;
        if (!docu.setContent(f))
                return false;
-       QDomElement elem=docu.documentElement();
+       QDomElement elem = docu.documentElement();
        if (elem.tagName() != "SCRIBUSUTF8NEW")
                return false;
-       QDomNode DOC=elem.firstChild();
-       while (!DOC.isNull())
-       {
-               QDomNode PAGE=DOC.firstChild();
-               while (!PAGE.isNull())
-               {
-                       QDomElement pg=PAGE.toElement();
-                       PgNam = pg.attribute("NAM", "");
-                       if (pg.tagName()=="PAGE")
+       QDomNode docNode = elem.firstChild();
+       while (!docNode.isNull())
+       {
+               QDomNode pageNode = docNode.firstChild();
+               while (!pageNode.isNull())
+               {
+                       QDomElement pg = pageNode.toElement();
+                       pageName = pg.attribute("NAM", "");
+                       if (pg.tagName() == "PAGE")
                                counter++;
-                       if (pg.tagName()=="MASTERPAGE")
+                       if (pg.tagName() == "MASTERPAGE")
                        {
                                counter2++;
-                               masterPageNames.append(PgNam);
-                       }
-                       PAGE=PAGE.nextSibling();
-               }
-               DOC=DOC.nextSibling();
+                               masterPageNames.append(pageName);
+                       }
+                       pageNode = pageNode.nextSibling();
+               }
+               docNode = docNode.nextSibling();
        }
        *num1 = counter;
        *num2 = counter2;

Modified: 
trunk/Scribus/scribus/plugins/fileloader/scribus150format/scribus150format.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=23241&path=/trunk/Scribus/scribus/plugins/fileloader/scribus150format/scribus150format.cpp
==============================================================================
--- 
trunk/Scribus/scribus/plugins/fileloader/scribus150format/scribus150format.cpp  
    (original)
+++ 
trunk/Scribus/scribus/plugins/fileloader/scribus150format/scribus150format.cpp  
    Wed Oct  9 12:59:57 2019
@@ -2577,7 +2577,7 @@
        if (attrs.hasAttribute(LANGUAGE))
        {
                QString l(attrs.valueAsString(LANGUAGE));
-               if (LanguageManager::instance()->langTableIndex(l)!=-1)
+               if (LanguageManager::instance()->langTableIndex(l) != -1)
                        newStyle.setLanguage(l); //new style storage
                else
                { //old style storage


_______________________________________________
scribus-commit mailing list
[email protected]
http://lists.scribus.net/mailman/listinfo/scribus-commit

Reply via email to