Fix crash when trying to display open text dialog when there are no modules in configured directory

1 files changed, 19 insertions, 16 deletions

diff --git a/noncore/apps/dagger/opentextdlg.cpp b/noncore/apps/dagger/opentextdlg.cpp
index f7893a9..8c94a4b 100644
--- a/noncore/apps/dagger/opentextdlg.cpp
+++ b/noncore/apps/dagger/opentextdlg.cpp
@@ -32,67 +32,70 @@ OpenTextDlg::OpenTextDlg( QWidget *parent, sword::SWMgr *swordMgr, QPixmap *bibl
 
     QVBoxLayout *layout = new QVBoxLayout( this );
     layout->setMargin( 4 );
     layout->addWidget( &m_textList );
 
     m_textList.setRootIsDecorated( true );
     m_textList.addColumn( tr( "Icon" ),35 );
     m_textList.addColumn( tr( "Text" ) );
     m_textList.header()->hide();
     m_textList.setAllColumnsShowFocus( true );
     m_textList.setSorting( 1 );
 
     m_commentaries = new QListViewItem( &m_textList, QString::null, tr( "Commentaries" ) );
     m_commentaries->setPixmap( 0, *commentaryIcon );
     m_textList.insertItem( m_commentaries );
     m_lexicons = new QListViewItem( &m_textList, QString::null, tr( "Lexicons/Dictionaries" ) );
     m_lexicons->setPixmap( 0, *lexiconIcon );
     m_textList.insertItem( m_lexicons );
     m_bibles = new QListViewItem( &m_textList, QString::null, tr( "Biblical Texts" ) );
     m_bibles->setPixmap( 0, *bibleIcon );
     m_textList.insertItem( m_bibles );
     connect( &m_textList, SIGNAL(clicked(QListViewItem*)), this, SLOT(slotItemClicked(QListViewItem*)) );
 
     if ( swordMgr )
     {
         sword::ModMap::iterator it;
         QString type;
         QPixmap *icon = 0x0;
         QListViewItem *parent = 0x0;
 
         for ( it = swordMgr->Modules.begin(); it != swordMgr->Modules.end(); it++ )
         {
-            type = it->second->Type();
-            if ( type == "Biblical Texts" )
+            if ( it->second )
             {
-                icon = bibleIcon;
-                parent = m_bibles;
+                type = it->second->Type();
+                if ( type == "Biblical Texts" )
+                {
+                    icon = bibleIcon;
+                    parent = m_bibles;
+                }
+                else if ( type == "Commentaries" )
+                {
+                    icon = commentaryIcon;
+                    parent = m_commentaries;
+                }
+                else if ( type == "Lexicons / Dictionaries" )
+                {
+                    icon = lexiconIcon;
+                    parent = m_lexicons;
+                }
+    
+                parent->insertItem( new QListViewItem( parent, QString::null, it->first.c_str() ) );
             }
-            else if ( type == "Commentaries" )
-            {
-                icon = commentaryIcon;
-                parent = m_commentaries;
-            }
-            else if ( type == "Lexicons / Dictionaries" )
-            {
-                icon = lexiconIcon;
-                parent = m_lexicons;
-            }
-
-            parent->insertItem( new QListViewItem( parent, QString::null, it->first.c_str() ) );
         }
     }
 
     m_textList.sort();
 }
 
 void OpenTextDlg::slotItemClicked( QListViewItem *item )
 {
     if ( item == m_bibles || item == m_lexicons || item == m_commentaries )
     {
         m_textList.clearSelection();
         if ( item->childCount() > 0 )
         {
             item->setOpen( !item->isOpen() );
         }
     }
 }