From: vsr <vsr@opencascade.com>
Date: Thu, 26 Apr 2007 07:34:38 +0000 (+0000)
Subject: Fix a crash if SALOME module sources item does not have a pair of corresponding SALOM... 
X-Git-Tag: V_3_2_9~37
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=d3e9fccefbdffb51434361f957083d2d3a8eb646;p=tools%2Finstall.git

Fix a crash if SALOME module sources item does not have a pair of corresponding SALOME module binaries item.
---

diff --git a/bin/SALOME_InstallWizard b/bin/SALOME_InstallWizard
index b6accf9..c605690 100755
Binary files a/bin/SALOME_InstallWizard and b/bin/SALOME_InstallWizard differ
diff --git a/src/SALOME_InstallWizard.cxx b/src/SALOME_InstallWizard.cxx
index bca0109..583380c 100644
--- a/src/SALOME_InstallWizard.cxx
+++ b/src/SALOME_InstallWizard.cxx
@@ -1835,7 +1835,7 @@ void SALOME_InstallWizard::reject()
 				   tr( "&Yes" ),
 				   tr( "&No" ),
 				   QString::null,
-				   0,
+				   1,
 				   1 ) == 1 ) {
       return;
     }
diff --git a/src/SALOME_ProductsView.cxx b/src/SALOME_ProductsView.cxx
index 5268830..c68ae02 100644
--- a/src/SALOME_ProductsView.cxx
+++ b/src/SALOME_ProductsView.cxx
@@ -129,6 +129,8 @@ void ProductsView::itemActivated( QCheckListItem* item ) {
 // ================================================================
 bool ProductsView::isBinaries( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -147,6 +149,8 @@ bool ProductsView::isBinaries( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::isSources( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -168,6 +172,8 @@ bool ProductsView::isSources( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::isNative( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -196,6 +202,8 @@ bool ProductsView::isNone(  QCheckListItem* item ) const
  */
 // ================================================================
 void ProductsView::setBinaries( QCheckListItem* item ) {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* srcItem = 0;
@@ -228,6 +236,8 @@ void ProductsView::setBinaries( QCheckListItem* item ) {
  */
 // ================================================================
 void ProductsView::setSources( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* binItem = 0;
@@ -260,6 +270,8 @@ void ProductsView::setSources( QCheckListItem* item )  {
  */
 // ================================================================
 void ProductsView::setNative( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* binItem = 0;
@@ -291,6 +303,8 @@ void ProductsView::setNative( QCheckListItem* item )  {
  */
 // ================================================================
 void ProductsView::setNone( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -314,6 +328,8 @@ void ProductsView::setNone( QCheckListItem* item )  {
 // ================================================================
 bool ProductsView::hasBinaries( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -333,6 +349,8 @@ bool ProductsView::hasBinaries( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::hasSources( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -352,6 +370,8 @@ bool ProductsView::hasSources( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::hasNative( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -371,6 +391,8 @@ bool ProductsView::hasNative( QCheckListItem* item ) const
 // ================================================================
 void ProductsView::setItemEnabled( QCheckListItem* item, const bool enable )
 {
+  if ( !item ) 
+    return;
   item->setEnabled( enable );
   QListViewItem* subItem = item->firstChild();
   while( subItem ) {