Skip to content

Commit 29babaf

Browse files
committed
Add comments to enable bugfixing in the future
1 parent b33082f commit 29babaf

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

lib/query/woqlQuery.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -425,6 +425,7 @@ WOQLQuery.prototype.from = function (graphRef, query) {
425425
// return ['graph', 'query']
426426
if (this.cursor['@type']) this.wrapCursorWithAnd();
427427
this.cursor['@type'] = 'From';
428+
// FIXME: Fix the typeof check that should be graphRef, not graph
428429
if (!graphRef || typeof graph !== 'string') {
429430
return this.parameterError(
430431
'The first parameter to from must be a Graph Filter Expression (string)',
@@ -1389,6 +1390,7 @@ WOQLQuery.prototype.length = function (inputVarList, resultVarName) {
13891390
if (this.cursor['@type']) this.wrapCursorWithAnd();
13901391
this.cursor['@type'] = 'Length';
13911392
this.cursor.list = this.cleanDataValue(inputVarList);
1393+
// FIXME: Bug - should check typeof resultVarName instead of undefined variable 'vb'
13921394
if (typeof vb === 'number') {
13931395
this.cursor.length = this.cleanObject(resultVarName, 'xsd:nonNegativeInteger');
13941396
} else if (typeof vb === 'string') {

0 commit comments

Comments
 (0)