From b0d24be58fc854da97ee94625a6dad2dc864b5ed Mon Sep 17 00:00:00 2001
From: Seth Alves <seth.alves@gmail.com>
Date: Fri, 13 Nov 2015 16:02:39 -0800
Subject: [PATCH] add a way to get a list of all actions of a certain type from
 an entity.  hold actions average their positional targets.

---
 interface/src/avatar/AvatarActionHold.cpp     | 29 +++++++++++++++++--
 .../entities/src/EntityActionInterface.h      | 10 +++++--
 libraries/entities/src/EntityItem.cpp         | 15 ++++++++++
 libraries/entities/src/EntityItem.h           |  5 +++-
 libraries/entities/src/EntityTypes.h          |  4 +--
 libraries/physics/src/ObjectAction.h          |  1 -
 6 files changed, 56 insertions(+), 8 deletions(-)

diff --git a/interface/src/avatar/AvatarActionHold.cpp b/interface/src/avatar/AvatarActionHold.cpp
index ca7fcfb187..8e13fa8385 100644
--- a/interface/src/avatar/AvatarActionHold.cpp
+++ b/interface/src/avatar/AvatarActionHold.cpp
@@ -66,9 +66,34 @@ std::shared_ptr<Avatar> AvatarActionHold::getTarget(glm::quat& rotation, glm::ve
 void AvatarActionHold::updateActionWorker(float deltaTimeStep) {
     glm::quat rotation;
     glm::vec3 position;
-    std::shared_ptr<Avatar> holdingAvatar = getTarget(rotation, position);
+    bool valid = false;
+    int holdCount = 0;
+
+    auto ownerEntity = _ownerEntity.lock();
+    if (!ownerEntity) {
+        return;
+    }
+    QList<EntityActionPointer> holdActions = ownerEntity->getActionsOfType(ACTION_TYPE_HOLD);
+    foreach (EntityActionPointer action, holdActions) {
+        std::shared_ptr<AvatarActionHold> holdAction = std::static_pointer_cast<AvatarActionHold>(action);
+        glm::quat rotationForAction;
+        glm::vec3 positionForAction;
+        std::shared_ptr<Avatar> holdingAvatar = holdAction->getTarget(rotationForAction, positionForAction);
+        if (holdingAvatar) {
+            holdCount ++;
+            if (holdAction.get() == this) {
+                // only use the rotation for this action
+                valid = true;
+                rotation = rotationForAction;
+            }
+
+            position += positionForAction;
+        }
+    }
+
+    if (valid && holdCount > 0) {
+        position /= holdCount;
 
-    if (holdingAvatar) {
         bool gotLock = withTryWriteLock([&]{
             _positionalTarget = position;
             _rotationalTarget = rotation;
diff --git a/libraries/entities/src/EntityActionInterface.h b/libraries/entities/src/EntityActionInterface.h
index 01292e3840..a192661e52 100644
--- a/libraries/entities/src/EntityActionInterface.h
+++ b/libraries/entities/src/EntityActionInterface.h
@@ -12,11 +12,14 @@
 #ifndef hifi_EntityActionInterface_h
 #define hifi_EntityActionInterface_h
 
+#include <memory>
 #include <QUuid>
+#include <glm/glm.hpp>
 
-#include "EntityItem.h"
-
+class EntityItem;
 class EntitySimulation;
+using EntityItemPointer = std::shared_ptr<EntityItem>;
+using EntityItemWeakPointer = std::weak_ptr<EntityItem>;
 
 enum EntityActionType {
     // keep these synchronized with actionTypeFromString and actionTypeToString
@@ -34,6 +37,8 @@ public:
     const QUuid& getID() const { return _id; }
     EntityActionType getType() const { return _type; }
 
+    bool isActive() { return _active; }
+
     virtual void removeFromSimulation(EntitySimulation* simulation) const = 0;
     virtual EntityItemWeakPointer getOwnerEntity() const = 0;
     virtual void setOwnerEntity(const EntityItemPointer ownerEntity) = 0;
@@ -81,6 +86,7 @@ protected:
 
     QUuid _id;
     EntityActionType _type;
+    bool _active { false };
 };
 
 
diff --git a/libraries/entities/src/EntityItem.cpp b/libraries/entities/src/EntityItem.cpp
index f032dcd347..100f6dfe22 100644
--- a/libraries/entities/src/EntityItem.cpp
+++ b/libraries/entities/src/EntityItem.cpp
@@ -1844,3 +1844,18 @@ bool EntityItem::shouldSuppressLocationEdits() const {
 
     return false;
 }
+
+QList<EntityActionPointer> EntityItem::getActionsOfType(EntityActionType typeToGet) {
+    QList<EntityActionPointer> result;
+
+    QHash<QUuid, EntityActionPointer>::const_iterator i = _objectActions.begin();
+    while (i != _objectActions.end()) {
+        EntityActionPointer action = i.value();
+        if (action->getType() == typeToGet && action->isActive()) {
+            result += action;
+        }
+        i++;
+    }
+
+    return result;
+}
diff --git a/libraries/entities/src/EntityItem.h b/libraries/entities/src/EntityItem.h
index 5b47198e97..5ceccef4b1 100644
--- a/libraries/entities/src/EntityItem.h
+++ b/libraries/entities/src/EntityItem.h
@@ -30,6 +30,7 @@
 #include "EntityTypes.h"
 #include "SimulationOwner.h"
 #include "SimulationFlags.h"
+#include "EntityActionInterface.h"
 
 class EntitySimulation;
 class EntityTreeElement;
@@ -419,7 +420,9 @@ public:
 
     void setSourceUUID(const QUuid& sourceUUID) { _sourceUUID = sourceUUID; }
     const QUuid& getSourceUUID() const { return _sourceUUID; }
-    bool matchesSourceUUID(const QUuid& sourceUUID) const  { return _sourceUUID == sourceUUID; }
+    bool matchesSourceUUID(const QUuid& sourceUUID) const { return _sourceUUID == sourceUUID; }
+
+    QList<EntityActionPointer> getActionsOfType(EntityActionType typeToGet);
 
 protected:
 
diff --git a/libraries/entities/src/EntityTypes.h b/libraries/entities/src/EntityTypes.h
index 30b6edbc07..3536327d18 100644
--- a/libraries/entities/src/EntityTypes.h
+++ b/libraries/entities/src/EntityTypes.h
@@ -20,8 +20,8 @@
 #include <OctreeRenderer.h> // for RenderArgs
 
 class EntityItem;
-typedef std::shared_ptr<EntityItem> EntityItemPointer;
-typedef std::weak_ptr<EntityItem> EntityItemWeakPointer;
+using EntityItemPointer = std::shared_ptr<EntityItem>;
+using EntityItemWeakPointer = std::weak_ptr<EntityItem>;
 
 inline uint qHash(const EntityItemPointer& a, uint seed) {
     return qHash(a.get(), seed);
diff --git a/libraries/physics/src/ObjectAction.h b/libraries/physics/src/ObjectAction.h
index 45b40a9fb3..afb6745e9c 100644
--- a/libraries/physics/src/ObjectAction.h
+++ b/libraries/physics/src/ObjectAction.h
@@ -67,7 +67,6 @@ protected:
     EntityItemWeakPointer _ownerEntity;
     QString _tag;
     quint64 _expires { 0 }; // in seconds since epoch
-    bool _active { false };
 
 private:
     int getEntityServerClockSkew() const;