IPAL 20025 4.x: wrong presentation is created with checked "Logarithmic Scaling"...

diff --git a/src/VISUGUI/VisuGUI_Table3dDlg.cxx b/src/VISUGUI/VisuGUI_Table3dDlg.cxx
index 2b49f5c93cac789bf1418b0c67ca78ca38d79b23..dd5046eb636f7622e1c4fc65940c6154b72f70e0 100644 (file)
--- a/src/VISUGUI/VisuGUI_Table3dDlg.cxx
+++ b/src/VISUGUI/VisuGUI_Table3dDlg.cxx
@@ -402,17 +402,13 @@ void VisuGUI_TableScalarBarPane::initFromPrsObject(VISU::PointMap3d_i* thePrs)
   if( !myBarPrs )
     return;
 
-  if (myBarPrs->IsPositiveTable()) {
-    switch(myBarPrs->GetScaling()){
-    case VISU::LOGARITHMIC :
-      CBLog->setChecked( true );
-      break;
-    default:
-      CBLog->setChecked( false );
-    }
+  switch(myBarPrs->GetScaling()){
+  case VISU::LOGARITHMIC :
+    CBLog->setChecked( true );
+    break;
+  default:
+    CBLog->setChecked( false );
   }
-  else
-    CBLog->setEnabled(false);
 
   setRange( myBarPrs->GetMin(), myBarPrs->GetMax(), myBarPrs->IsRangeFixed() );
 
@@ -713,13 +709,53 @@ VisuGUI_Table3DDlg::VisuGUI_Table3DDlg ( SalomeApp_Module* theModule )
 VisuGUI_Table3DDlg::~VisuGUI_Table3DDlg()
 {}
 
+//=======================================================================
+//function : Check
+//purpose  : Called when <OK> button is clicked, validates data and closes dialog
+//=======================================================================
+bool VisuGUI_TableScalarBarPane::check()
+{
+  double minVal = MinEdit->text().toDouble();
+  double maxVal = MaxEdit->text().toDouble();
+  if ( RBIrange->isChecked() ) {
+    if (minVal >= maxVal) {
+      SUIT_MessageBox::warn1( this,tr("WRN_VISU"),
+                            tr("MSG_MINMAX_VALUES"),
+                            tr("BUT_OK"));
+      return false;
+    }
+  }
+
+  // check if logarithmic mode is on and check imposed range to not contain negative values
+  if ( CBLog->isChecked() ) {
+    if ( minVal <= 0.0 || maxVal <= 0.0 ) {
+      if ( RBIrange->isChecked() ) {
+       SUIT_MessageBox::warn1( this,
+                               tr("WRN_VISU"),
+                               tr("WRN_LOGARITHMIC_RANGE"),
+                               tr("BUT_OK"));
+      } else {
+       SUIT_MessageBox::warn1( this,
+                               tr("WRN_VISU"),
+                               tr("WRN_LOGARITHMIC_FIELD_RANGE"),
+                               tr("BUT_OK"));
+       RBIrange->setChecked(true);
+       changeRange(1);
+      }
+      return false;
+    }
+  }
+  return true;
+}
+
 //=======================================================================
 //function : accept
 //purpose  :
 //=======================================================================
 void VisuGUI_Table3DDlg::accept()
 {
-  QDialog::accept();
+  if (myScalarBarPane->check())
+    QDialog::accept();
 }
 
 //=======================================================================
@@ -737,8 +773,10 @@ void VisuGUI_Table3DDlg::reject()
 //=======================================================================
 void VisuGUI_Table3DDlg::onApply()
 {
-  storeToPrsObject( myPrsCopy );
-  myPrsCopy->UpdateActors();
+  if (myScalarBarPane->check()) {
+    storeToPrsObject( myPrsCopy );
+    myPrsCopy->UpdateActors();
+  }
 }
 
 //=======================================================================

diff --git a/src/VISU_I/VISU_PointMap3d_i.cc b/src/VISU_I/VISU_PointMap3d_i.cc
index 95b0766c14f6a80896c2a1a275ec4fd72c25631b..a36cc8cad54d0597857295448037b6bb06abe643 100644 (file)
--- a/src/VISU_I/VISU_PointMap3d_i.cc
+++ b/src/VISU_I/VISU_PointMap3d_i.cc
@@ -736,12 +736,38 @@ CORBA::Float VISU::PointMap3d_i::GetMemorySize()
 //------------------ ColoredPrs3dBase Methods --------------------
 CORBA::Double VISU::PointMap3d_i::GetMin()
 {
-  return myTablePL->GetScalarRange()[0];
+  SALOMEDS::GenericAttribute_var anAttr;
+  mySObj->FindAttribute(anAttr, "AttributeTableOfReal");
+  SALOMEDS::AttributeTableOfReal_var aTableOfReal = SALOMEDS::AttributeTableOfReal::_narrow(anAttr);
+  double aMin = aTableOfReal->GetValue(1, 1);
+
+  for (int i=1; i<=aTableOfReal->GetNbColumns(); i++)
+    for (int j=1; j<=aTableOfReal->GetNbRows(); j++) {
+      double aVal = aTableOfReal->GetValue(j, i);
+      if (aVal < aMin)
+       aMin = aVal;
+    }
+
+  return aMin;
+  //  return myTablePL->GetScalarRange()[0];
 }
 
 CORBA::Double VISU::PointMap3d_i::GetMax()
 {
-  return myTablePL->GetScalarRange()[1];
+  SALOMEDS::GenericAttribute_var anAttr;
+  mySObj->FindAttribute(anAttr, "AttributeTableOfReal");
+  SALOMEDS::AttributeTableOfReal_var aTableOfReal = SALOMEDS::AttributeTableOfReal::_narrow(anAttr);
+  double aMax = aTableOfReal->GetValue(1, 1);
+
+  for (int i=1; i<=aTableOfReal->GetNbColumns(); i++)
+    for (int j=1; j<=aTableOfReal->GetNbRows(); j++) {
+      double aVal = aTableOfReal->GetValue(j, i);
+      if (aVal > aMax)
+       aMax = aVal;
+    }
+
+  return aMax;
+  //  return myTablePL->GetScalarRange()[1];
 }
 
 void VISU::PointMap3d_i::SetRange(CORBA::Double theMin, CORBA::Double theMax)