Changes in src/Actions/AtomAction/TranslateAction.cpp [bcf653:34c338]

File:

• src/Actions/AtomAction/TranslateAction.cpp (modified) (6 diffs)

src/Actions/AtomAction/TranslateAction.cpp

@@ -1 @@
-/*
- * Project: MoleCuilder
- * Description: creates and alters molecular systems
- * Copyright (C)  2010 University of Bonn. All rights reserved.
- * Please see the LICENSE file or "Copyright notice" in builder.cpp for details.
- */
-
 /*
  * TranslateAction.cpp
@@ -12 +5 @@
  *      Author: heber
  */
-
-// include config.h
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
 
 #include "Helpers/MemDebug.hpp"
@@ -44 +32 @@
 class AtomTranslateState : public ActionState {
 public:
-  AtomTranslateState(const std::vector<atom*> &_selectedAtoms, const Vector &_v, const bool _periodic) :
+  AtomTranslateState(std::vector<atom*> _selectedAtoms, Vector &_v) :
     selectedAtoms(_selectedAtoms),
-    v(_v),
-    periodic(_periodic)
+    v(_v)
   {}
   std::vector<atom*> selectedAtoms;
   Vector v;
-  bool periodic;
 };
 
@@ -63 +49 @@
 {}
 
-void AtomTranslate(Vector &x, bool periodic = false) {
+void AtomTranslate(Vector &x) {
   ValueStorage::getInstance().setCurrentValue(AtomTranslateAction::NAME, x);
-  ValueStorage::getInstance().setCurrentValue("periodic", periodic);
   ActionRegistry::getInstance().getActionByName(AtomTranslateAction::NAME)->call(Action::NonInteractive);
 };
@@ -73 +58 @@
 
   dialog->queryVector(NAME, false, ValueStorage::getInstance().getDescription(NAME));
-  dialog->queryBoolean("periodic", ValueStorage::getInstance().getDescription("periodic"));
 
   return dialog;
@@ -80 +64 @@
 Action::state_ptr AtomTranslateAction::performCall() {
   Vector v;
-  bool periodic = false;
-  Box &domain = World::getInstance().getDomain();
   std::vector<atom *> selectedAtoms = World::getInstance().getSelectedAtoms();
 
   ValueStorage::getInstance().queryCurrentValue(NAME, v);
-  if (!ValueStorage::getInstance().queryCurrentValue("periodic", periodic, true))
-    periodic = false;
 
   // TODO: use AtomSet::translate
   for (std::vector<atom *>::iterator iter = selectedAtoms.begin(); iter != selectedAtoms.end(); ++iter) {
     *(*iter) += v;
-    if (periodic)
-      (*iter)->setPosition(domain.WrapPeriodically((*iter)->getPosition()));
   }
-
-  return Action::state_ptr(new AtomTranslateState(selectedAtoms, v, periodic));
+  return Action::state_ptr(new AtomTranslateState(selectedAtoms, v));
 }
 
 Action::state_ptr AtomTranslateAction::performUndo(Action::state_ptr _state) {
   AtomTranslateState *state = assert_cast<AtomTranslateState*>(_state.get());
-  Box &domain = World::getInstance().getDomain();
 
   for (std::vector<atom *>::iterator iter = state->selectedAtoms.begin(); iter != state->selectedAtoms.end(); ++iter) {
     *(*iter) -= state->v;
-    if (state->periodic)
-      (*iter)->setPosition(domain.WrapPeriodically((*iter)->getPosition()));
   }
 
-  return Action::state_ptr(_state);
+  return Action::state_ptr(new AtomTranslateState(state->selectedAtoms, state->v));
 }
 
 Action::state_ptr AtomTranslateAction::performRedo(Action::state_ptr _state){
   AtomTranslateState *state = assert_cast<AtomTranslateState*>(_state.get());
-  Box &domain = World::getInstance().getDomain();
 
   for (std::vector<atom *>::iterator iter = state->selectedAtoms.begin(); iter != state->selectedAtoms.end(); ++iter) {
     *(*iter) += state->v;
-    if (state->periodic)
-      (*iter)->setPosition(domain.WrapPeriodically((*iter)->getPosition()));
   }
 
-  return Action::state_ptr(_state);
+  return Action::state_ptr(new AtomTranslateState(state->selectedAtoms, state->v));
 }