Changeset 361 in PSPA for Interface_Web

Timestamp:

Mar 5, 2013, 11:35:57 AM (12 years ago)

Author:

garnier

Message:

correction d'un plantage au run sur un element fit

Location:

Interface_Web/trunk/pspaWT

Files:

• History (modified) (2 diffs)
• sources/controler/src/abstractElement.cc (modified) (2 diffs)
• sources/userInterface/src/GWt_ligneFaisceau.cc (modified) (1 diff)
• workingArea/philFit.save (added)

Interface_Web/trunk/pspaWT/History

@@ -5 +5 @@
      PSPA - Plateforme de simulation en physique des accélérateurs
      =========================================================
+
+5 Mars : Laurent Garnier
+- GWt_LigneFaisceau : protection contre un pointer NULL
+- abstractElement : Ajout d'un cas oublié dans la version précédente
+- ajoute du cas test philFit.save
 
 4 Mars : Laurent Garnier
@@ -36 +41 @@
 - softwareParmela  : ajout d'une protection sur NULL pointer dans createInputFile()
 - trivaluedBool.h : Changement des noms (confusions possibles) en TBoolError, TBoolIgnore, TBoolOk
+
 1er Mars : Laurent Garnier
 - Fix le bug sur le serveur : Mauvaise variable utilisée. Introduit le 26 février

Interface_Web/trunk/pspaWT/sources/controler/src/abstractElement.cc

@@ -111 +111 @@
     }
     
+    trivaluedBool tb = TBoolError;
     if ( std::find(acceptableSoftware_.begin(), acceptableSoftware_.end(), prog) != acceptableSoftware_.end()) {
+        tb = TBoolOk;
+    } else if ( std::find(ignoreSoftware_.begin(), ignoreSoftware_.end(), prog) != ignoreSoftware_.end()) {
+        tb =TBoolIgnore;
+    } else {
+        tb = TBoolError;
+    }
+    if ((tb == TBoolOk) || (tb == TBoolIgnore)) {
         if (prog == nomDeLogiciel::parmela){
             abstractSoftware_ = new softwareParmela();
@@ -123 +131 @@
             abstractSoftware_ = NULL;
         }
-    } else if ( std::find(ignoreSoftware_.begin(), ignoreSoftware_.end(), prog) != ignoreSoftware_.end()) {
-        return TBoolIgnore;
-    } else {
-        return TBoolError;
     }
-    return TBoolOk;
+    return tb;
 }
 

Interface_Web/trunk/pspaWT/sources/userInterface/src/GWt_ligneFaisceau.cc

@@ -266 +266 @@
         if (GWt_elementLigneFaisceau* elem = dynamic_cast <GWt_elementLigneFaisceau*> (ligneFaisceauLayout_->itemAt(a)->widget ())) {
             if (index == i) {
-                elem->setBGColor(WColor(elem->getGWt_AbstractElement()->getAbstractElement()->getAbstractSoftware()->getColor()));
+                if (elem->getGWt_AbstractElement()->getAbstractElement()->getAbstractSoftware()) {
+                    elem->setBGColor(WColor(elem->getGWt_AbstractElement()->getAbstractElement()->getAbstractSoftware()->getColor()));
+                }
             }
             index ++;