Changes in src/Actions/MoleculeAction/RotateAroundOriginByAngleAction.cpp [4a06d6:2204b0]

File:

• src/Actions/MoleculeAction/RotateAroundOriginByAngleAction.cpp (modified) (2 diffs)

src/Actions/MoleculeAction/RotateAroundOriginByAngleAction.cpp

@@ -52 +52 @@
 {}
 
-void MoleculeRotateAroundOriginByAngle(const Vector &Axis, double angle) {
+void MoleculeRotateAroundOriginByAngle(double angle) {
   ValueStorage::getInstance().setCurrentValue(MoleculeRotateAroundOriginByAngleAction::NAME, angle);
-  ValueStorage::getInstance().setCurrentValue("position", Axis);
   ActionRegistry::getInstance().getActionByName(MoleculeRotateAroundOriginByAngleAction::NAME)->call(Action::NonInteractive);
 };
@@ -83 +82 @@
   alpha *= M_PI/180.;
 
-  // Creation Line that is the rotation axis
-  Line RotationAxis(Vector(0.,0.,0.), Axis);
-
   DoLog(0) && (Log() << Verbose(0) << "Rotate around origin by " << alpha << " radian, axis from origin to " << Axis << "." << endl);
   for (World::MoleculeSelectionIterator iter = World::getInstance().beginMoleculeSelection(); iter != World::getInstance().endMoleculeSelection(); ++iter) {
     mol = iter->second;
 
+    // Creation Line that is the rotation axis
+    Line RotationAxis(Vector(0.,0.,0.), Axis);
+
     for (molecule::iterator iter = mol->begin(); iter != mol->end(); ++iter) {
       *((*iter)->node) = RotationAxis.rotateVector(*((*iter)->node), alpha);
     }
+    DoLog(0) && (Log() << Verbose(0) << "done." << endl);
   }
-  DoLog(0) && (Log() << Verbose(0) << "done." << endl);
   return Action::state_ptr(new MoleculeRotateAroundOriginByAngleState(World::getInstance().getSelectedMolecules(), Axis, alpha));
 }