From 4f5727e6b0c143a400c6e0be063fd91eb6eb8c4a Mon Sep 17 00:00:00 2001
From: SamGondelman <samuel_gondelman@alumni.brown.edu>
Date: Mon, 8 Jan 2018 15:25:16 -0800
Subject: [PATCH] add _needsRenderUpdate flag

---
 libraries/entities-renderer/src/RenderableEntityItem.cpp | 9 ++++++++-
 libraries/entities-renderer/src/RenderableEntityItem.h   | 1 +
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/libraries/entities-renderer/src/RenderableEntityItem.cpp b/libraries/entities-renderer/src/RenderableEntityItem.cpp
index 1f11802405..fb9aba636b 100644
--- a/libraries/entities-renderer/src/RenderableEntityItem.cpp
+++ b/libraries/entities-renderer/src/RenderableEntityItem.cpp
@@ -141,7 +141,10 @@ std::shared_ptr<T> make_renderer(const EntityItemPointer& entity) {
 }
 
 EntityRenderer::EntityRenderer(const EntityItemPointer& entity) : _entity(entity) {
-    connect(entity.get(), &EntityItem::requestRenderUpdate, this, &EntityRenderer::requestRenderUpdate);
+    connect(entity.get(), &EntityItem::requestRenderUpdate, this, [&] {
+        _needsRenderUpdate = true;
+        emit requestRenderUpdate();
+    });
 }
 
 EntityRenderer::~EntityRenderer() { }
@@ -312,6 +315,9 @@ void EntityRenderer::setSubRenderItemIDs(const render::ItemIDs& ids) {
 // Returns true if the item needs to have updateInscene called because of internal rendering 
 // changes (animation, fading, etc)
 bool EntityRenderer::needsRenderUpdate() const {
+    if (_needsRenderUpdate) {
+        return true;
+    }
     if (_prevIsTransparent != isTransparent()) {
         return true;
     }
@@ -360,6 +366,7 @@ void EntityRenderer::doRenderUpdateSynchronous(const ScenePointer& scene, Transa
 
         _moving = entity->isMovingRelativeToParent();
         _visible = entity->getVisible();
+        _needsRenderUpdate = false;
     });
 }
 
diff --git a/libraries/entities-renderer/src/RenderableEntityItem.h b/libraries/entities-renderer/src/RenderableEntityItem.h
index 5deb69711d..8eb82e2c6e 100644
--- a/libraries/entities-renderer/src/RenderableEntityItem.h
+++ b/libraries/entities-renderer/src/RenderableEntityItem.h
@@ -125,6 +125,7 @@ protected:
     bool _prevIsTransparent { false };
     bool _visible { false };
     bool _moving { false };
+    bool _needsRenderUpdate { false };
     // Only touched on the rendering thread
     bool _renderUpdateQueued{ false };