r22651 by jghali - refactor scripter's GetUniqueItem()

[scribus-commit]

Author: jghali
Date: Mon Aug 27 18:56:37 2018
New Revision: 22651

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=22651
Log:
refactor scripter's GetUniqueItem()

Modified:
    trunk/Scribus/scribus/plugins/scriptplugin/cmdutil.cpp

Modified: trunk/Scribus/scribus/plugins/scriptplugin/cmdutil.cpp
URL: 
http://scribus.net/websvn/diff.php?repname=Scribus&rev=22651&path=/trunk/Scribus/scribus/plugins/scriptplugin/cmdutil.cpp
==============================================================================
--- trunk/Scribus/scribus/plugins/scriptplugin/cmdutil.cpp      (original)
+++ trunk/Scribus/scribus/plugins/scriptplugin/cmdutil.cpp      Mon Aug 27 
18:56:37 2018
@@ -101,16 +101,14 @@
 /* 04/07/10 returns selection if is not name specified  pv  */
 PageItem* GetUniqueItem(const QString& name)
 {
-       if (name.length()==0)
+       if (name.length() == 0)
+       {
                if (ScCore->primaryMainWindow()->doc->m_Selection->count() != 0)
                        return 
ScCore->primaryMainWindow()->doc->m_Selection->itemAt(0);
-               else
-               {
-                       PyErr_SetString(NoValidObjectError, QString("Cannot use 
empty string for object name when there is no 
selection").toLocal8Bit().constData());
-                       return nullptr;
-               }
-       else
-               return getPageItemByName(name);
+               PyErr_SetString(NoValidObjectError, QString("Cannot use empty 
string for object name when there is no selection").toLocal8Bit().constData());
+               return nullptr;
+       }
+       return getPageItemByName(name);
 }
 
 PageItem* getPageItemByName(const QString& name)


_______________________________________________
scribus-commit mailing list
scribus-commit@lists.scribus.net
http://lists.scribus.net/mailman/listinfo/scribus-commit