From 717436e0fded47ab9e0172393e8aa5e5dc5a3c1e Mon Sep 17 00:00:00 2001
From: Seth Alves <seth.alves@gmail.com>
Date: Mon, 10 Apr 2017 10:54:30 -0700
Subject: [PATCH] use QUrl rather than just munging the url with a
 string-append

---
 .../entities-renderer/src/RenderableModelEntityItem.cpp   | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/libraries/entities-renderer/src/RenderableModelEntityItem.cpp b/libraries/entities-renderer/src/RenderableModelEntityItem.cpp
index 3f2f4308b3..0ff68b4e6a 100644
--- a/libraries/entities-renderer/src/RenderableModelEntityItem.cpp
+++ b/libraries/entities-renderer/src/RenderableModelEntityItem.cpp
@@ -14,6 +14,7 @@
 
 #include <QJsonDocument>
 #include <QtCore/QThread>
+#include <QUrlQuery>
 #include <glm/gtx/transform.hpp>
 
 #include <AbstractViewStateInterface.h>
@@ -621,8 +622,11 @@ void RenderableModelEntityItem::setShapeType(ShapeType type) {
 
 void RenderableModelEntityItem::setCompoundShapeURL(const QString& url) {
     auto currentCompoundShapeURL = getCompoundShapeURL();
-    QString hullURL = url + "?collision-hull";
-    ModelEntityItem::setCompoundShapeURL(hullURL);
+    QUrl hullURL(url);
+    QUrlQuery queryArgs(hullURL);
+    queryArgs.addQueryItem("collision-hull", "");
+    hullURL.setQuery(queryArgs);
+    ModelEntityItem::setCompoundShapeURL(hullURL.toString());
 
     if (getCompoundShapeURL() != currentCompoundShapeURL || !_model) {
         EntityTreePointer tree = getTree();