Changes in src/Actions/WorldAction/AddEmptyBoundaryAction.cpp [d74077:952f38]

File:

• src/Actions/WorldAction/AddEmptyBoundaryAction.cpp (modified) (5 diffs)

src/Actions/WorldAction/AddEmptyBoundaryAction.cpp

@@ -11 +11 @@
 #include "Actions/ActionRegistry.hpp"
 #include "atom.hpp"
-#include "log.hpp"
-#include "vector.hpp"
+#include "Helpers/Log.hpp"
+#include "LinearAlgebra/Vector.hpp"
 #include "World.hpp"
 
@@ -23 +23 @@
 #include "UIElements/UIFactory.hpp"
 #include "UIElements/Dialog.hpp"
-#include "UIElements/ValueStorage.hpp"
+#include "Actions/ValueStorage.hpp"
 #include "Helpers/Assert.hpp"
 
@@ -40 +40 @@
 };
 
-Dialog* WorldAddEmptyBoundaryAction::createDialog() {
-  Dialog *dialog = UIFactory::getInstance().makeDialog();
+Dialog* WorldAddEmptyBoundaryAction::fillDialog(Dialog *dialog) {
+  ASSERT(dialog,"No Dialog given when filling action dialog");
 
   dialog->queryVector(NAME, false, ValueStorage::getInstance().getDescription(NAME));
@@ -60 +60 @@
   ASSERT(AllAtoms.size() > 0, "There must be atoms present for AddingEmptyBoundary.");
   vector<atom *>::iterator AtomRunner = AllAtoms.begin();
-  Min = (*AtomRunner)->getPosition();
-  Max = (*AtomRunner)->getPosition();
+  Min = (*AtomRunner)->x;
+  Max = (*AtomRunner)->x;
   for (; AtomRunner != AllAtoms.end(); ++AtomRunner) {
     for (int i=0;i<NDIM;i++) {
-      if ((*AtomRunner)->at(i) > Max[i])
-        Max[i] = (*AtomRunner)->at(i);
-      if ((*AtomRunner)->at(i) < Min[i])
-        Min[i] = (*AtomRunner)->at(i);
+      if ((*AtomRunner)->x[i] > Max[i])
+        Max[i] = (*AtomRunner)->x[i];
+      if ((*AtomRunner)->x[i] < Min[i])
+        Min[i] = (*AtomRunner)->x[i];
     }
   }
@@ -84 +84 @@
   AtomRunner = AllAtoms.begin();
   for (; AtomRunner != AllAtoms.end(); ++AtomRunner)
-    *(*AtomRunner) -= Min - boundary;
+    (*AtomRunner)->x -= Min - boundary;
   return Action::success;
 }