History

Fix EntityItemID not being converted to string in ScriptEngine

Ryan Huffman committed 10 years ago

Commit df092541 parent 8e3f4b71 file · +3 −3Browse files

Showing the changes to libraries/script-engine/src/ScriptEngine.cpp (1 of 1 file). Show all

Unified · Split

libraries/script-engine/src/ScriptEngine.cpp+3−3
@@ -1307,7 +1307,7 @@ void ScriptEngine::loadEntityScript(QWeakPointer<ScriptEngine> theEngine, const · 1306 unmodified lines
1307// so that they show up in stacktraces1307// so that they show up in stacktraces
1308//1308//
1309// Extract the url portion of a url that has been encoded with encodeEntityIdIntoEntityUrl(...)1309// Extract the url portion of a url that has been encoded with encodeEntityIdIntoEntityUrl(...)
1310QString extractUrlFromEntityUrl(QString url) {1310QString extractUrlFromEntityUrl(const QString& url) {
1311 auto parts = url.split(' ', QString::SkipEmptyParts);1311 auto parts = url.split(' ', QString::SkipEmptyParts);
1312 if (parts.length() > 0) {1312 if (parts.length() > 0) {
1313 return parts[0];1313 return parts[0];
@@ -1318,7 +1318,7 @@ QString extractUrlFromEntityUrl(QString url) { · 4 unmodified lines
1318 1318
1319// Encode an entity id into an entity url1319// Encode an entity id into an entity url
1320// Example: http://www.example.com/some/path.js [EntityID:{9fdd355f-d226-4887-9484-44432d29520e}]1320// Example: http://www.example.com/some/path.js [EntityID:{9fdd355f-d226-4887-9484-44432d29520e}]
1321QString encodeEntityIdIntoEntityUrl(QString url, QString entityID) {1321QString encodeEntityIdIntoEntityUrl(const QString& url, const QString& entityID) {
1322 return url + " [EntityID:" + entityID + "]";1322 return url + " [EntityID:" + entityID + "]";
1323}1323}
1324 1324
@@ -1348,7 +1348,7 @@ void ScriptEngine::entityScriptContentAvailable(const EntityItemID& entityID, co · 23 unmodified lines
1348 1348
1349 auto scriptCache = DependencyManager::get<ScriptCache>();1349 auto scriptCache = DependencyManager::get<ScriptCache>();
1350 bool isFileUrl = isURL && scriptOrURL.startsWith("file://");1350 bool isFileUrl = isURL && scriptOrURL.startsWith("file://");
1351 auto fileName = isURL ? encodeEntityIdIntoEntityUrl(scriptOrURL, entityID) : "EmbeddedEntityScript";1351 auto fileName = isURL ? encodeEntityIdIntoEntityUrl(scriptOrURL, entityID.toString()) : "EmbeddedEntityScript";
1352 1352
1353 QScriptProgram program(contents, fileName);1353 QScriptProgram program(contents, fileName);
1354 if (!hasCorrectSyntax(program)) {1354 if (!hasCorrectSyntax(program)) {