Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

andWhere query is not working #199

Open
sohamdodia opened this issue Nov 7, 2019 · 11 comments
Open

andWhere query is not working #199

sohamdodia opened this issue Nov 7, 2019 · 11 comments

Comments

@sohamdodia
Copy link

sohamdodia commented Nov 7, 2019

andWhere query is throwing an error: Errors converted to json-api Result TypeError: Cannot read property '0' of undefined.

   
queryFactory: async (opts) => {
      const origQuery = await opts.makeQuery(opts);
      return origQuery.andWhere({ field: "user_name", operator: "eq", value: "somename" });
    }


I have also downloaded the example provided in https://github.com/ethanresnick/json-api-example/tree/v3-wip repo, but the same error has occurred in this example as well.

@ethanresnick
Copy link
Owner

Can you provide a full stack trace for the error? This really isn't enough to debug with...

@sohamdodia
Copy link
Author

Errors converted to json-api Result TypeError: Cannot read property '0' of undefined
at toMongoCriteria (/json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/lib.js:71:38)
at Array.map ()
at Object.toMongoCriteria (/json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/lib.js:62:50)
at MongooseAdapter. (/json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/MongooseAdapter.js:63:43)
at Generator.next ()
at /json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/MongooseAdapter.js:7:71
at new Promise ()
at __awaiter (/json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/MongooseAdapter.js:3:12)
at MongooseAdapter.find (/json-api-example-3-wip/node_modules/json-api/build/src/db-adapters/Mongoose/MongooseAdapter.js:59:16)
at Object.runQuery [as default] (/json-api-example-3-wip/node_modules/json-api/build/src/steps/run-query.js:28:19)
at Object.APIController.runQuery (/json-api-example-3-wip/node_modules/json-api/build/src/controllers/API.js:58:55)
at /json-api-example-3-wip/node_modules/json-api/build/src/controllers/API.js:227:43
at Generator.next ()
at fulfilled (/json-api-example-3-wip/node_modules/json-api/build/src/controllers/API.js:4:58)

@ethanresnick
Copy link
Owner

@sohamdodia Does this error only occur when using AWS Serverless, or are you also experiencing it locally?

@sohamdodia
Copy link
Author

sohamdodia commented Nov 8, 2019

This issue is also occurring with local server as well.

@ethanresnick
Copy link
Owner

@sohamdodia Can you paste a log of the opts object that your query factory function is receiving, and a log of the origQuery variable

@emac08
Copy link

emac08 commented Nov 8, 2019

@ethanresnick if you send me an email to ericmacdougall03@gmail.com id be happy to set up a contract to help us get this library working with serverless and solve this issue (and of course assume any code updates/docs would be shared open source)

@sohamdodia
Copy link
Author

sohamdodia commented Nov 8, 2019

opts object:

{ request:
   { queryParams: { filter: undefined, sort: undefined },
     rawQueryString: undefined,
     id: undefined,
     type: 'media',
     relationship: undefined,
     aboutRelationship: false,
     uri: 'https://localhost/media',
     method: 'get',
     accepts: 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3',
     contentType: undefined,
     body: undefined,
     document: undefined },
  registry:
   ResourceTypeRegistry {
     _errorsConfig: undefined,
     _types:
      { todos: Map { "transformLinkage": false, "pagination": Map {}, "dbAdapter": [object Object], "urlTemplates": Map { "self": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      }, "relationship": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      } } },
        notes: Map { "transformLinkage": false, "pagination": Map {}, "dbAdapter": [object Object], "urlTemplates": Map { "self": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      }, "relationship": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      } } },
        media: Map { "transformLinkage": false, "pagination": Map {}, "dbAdapter": [object Object], "urlTemplates": Map { "self": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      }, "relationship": function (context) {
        return template.replace(/\{([^\{\}]+)\}|([^\{\}]+)/g, function (_, expression, literal) {
          if (expression) {
            var operator = null,
                values = [];

            if (operators.indexOf(expression.charAt(0)) !== -1) {
              operator = expression.charAt(0);
              expression = expression.substr(1);
            }

            expression.split(/,/g).forEach(function (variable) {
              var tmp = /([^:\*]*)(?::(\d+)|(\*))?/.exec(variable);
              values.push.apply(values, that.getValues(context, operator, tmp[1], tmp[2] || tmp[3]));
            });

            if (operator && operator !== '+') {
              var separator = ',';

              if (operator === '?') {
                separator = '&';
              } else if (operator !== '#') {
                separator = operator;
              }
              return (values.length !== 0 ? operator : '') + values.join(separator);
            } else {
              return values.join(',');
            }
          } else {
            return that.encodeReserved(literal);
          }
        });
      } } } },
     _typesMetadata: { nodes: [Array], edges: {}, roots: [Array] } },
  serverReq:
   IncomingMessage {
     _readableState:
      ReadableState {
        objectMode: false,
        highWaterMark: 16384,
        buffer: BufferList { length: 0 },
        length: 0,
        pipes: null,
        pipesCount: 0,
        flowing: true,
        ended: true,
        endEmitted: true,
        reading: false,
        errorEmitted: false,
        sync: true,
        needReadable: false,
        emittedReadable: false,
        readableListening: false,
        resumeScheduled: false,
        emitClose: true,
        destroyed: false,
        defaultEncoding: 'utf8',
        awaitDrain: 0,
        readingMore: true,
        decoder: null,
        encoding: null },
     readable: false,
     domain: null,
     _events: {},
     _eventsCount: 0,
     _maxListeners: undefined,
     socket:
      { encrypted: true,
        readable: false,
        remoteAddress: '127.0.0.1',
        address: [Function: address],
        end: [Function],
        destroy: [Function] },
     connection:
      { encrypted: true,
        readable: false,
        remoteAddress: '127.0.0.1',
        address: [Function: address],
        end: [Function],
        destroy: [Function] },
     httpVersionMajor: '1',
     httpVersionMinor: '1',
     httpVersion: '1.1',
     complete: true,
     headers:
      { host: 'localhost:8080',
        connection: 'keep-alive',
        'cache-control': 'max-age=0',
        'upgrade-insecure-requests': '1',
        'user-agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.87 Safari/537.36',
        'sec-fetch-user': '?1',
        accept: 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3',
        'sec-fetch-site': 'none',
        'sec-fetch-mode': 'navigate',
        'accept-encoding': 'gzip, deflate, br',
        'accept-language': 'en-US,en;q=0.9',
        cookie: '_ga=GA1.1.182961221.1554040199; token=77eb5c36-9438-4303-8bbd-fdd6cb4ab8c5',
        'if-none-match': 'W/"6c9-sf4l2A9UsRGuIrLzxbG0g4TrLVo"',
        'x-request-id': 'offlineContext_requestId_ck2qikym40001jdrmeslb86v3',
        'content-length': 0 },
     rawHeaders: [],
     trailers: {},
     rawTrailers: [],
     upgrade: null,
     url: '/media',
     method: 'GET',
     statusCode: null,
     statusMessage: null,
     client:
      { encrypted: true,
        readable: false,
        remoteAddress: '127.0.0.1',
        address: [Function: address],
        end: [Function],
        destroy: [Function] },
     _consuming: false,
     _dumped: false,
     ip: '127.0.0.1',
     body: <Buffer >,
     requestContext:
      { accountId: 'offlineContext_accountId',
        apiId: 'offlineContext_apiId',
        authorizer: [Object],
        httpMethod: 'GET',
        identity: [Object],
        protocol: 'HTTP/1.1',
        requestId: 'offlineContext_requestId_ck2qikym40001jdrmeslb86v3',
        requestTimeEpoch: 1573240173998,
        resourceId: 'offlineContext_resourceId',
        resourcePath: '/media',
        stage: 'dev' },
     serverless: { event: [Object], context: [LambdaContext] },
     next: [Function: next],
     baseUrl: '',
     originalUrl: '/media',
     _parsedUrl:
      Url {
        protocol: null,
        slashes: null,
        auth: null,
        host: null,
        port: null,
        hostname: null,
        hash: null,
        search: null,
        query: null,
        pathname: '/media',
        path: '/media',
        href: '/media',
        _raw: '/media' },
     params: { type: 'media' },
     query: {},
     res:
      ServerResponse {
        domain: null,
        _events: [Object],
        _eventsCount: 2,
        _maxListeners: undefined,
        output: [],
        outputEncodings: [],
        outputCallbacks: [],
        outputSize: 0,
        writable: true,
        _last: false,
        chunkedEncoding: false,
        shouldKeepAlive: true,
        useChunkedEncodingByDefault: false,
        sendDate: true,
        _removedConnection: false,
        _removedContLen: false,
        _removedTE: false,
        _contentLength: null,
        _hasBody: true,
        _trailer: '',
        finished: false,
        _headerSent: false,
        socket: [Writable],
        connection: [Writable],
        _header: null,
        _onPendingData: [Function: noopPendingOutput],
        _sent100: false,
        _expect_continue: false,
        write: [Function],
        req: [Circular],
        locals: {},
        _startAt: undefined,
        _startTime: undefined,
        writeHead: [Function: writeHead],
        __onFinished: [Function],
        [Symbol(isCorked)]: false,
        [Symbol(outHeadersKey)]: [Object],
        [Symbol()]: [],
        [Symbol()]: {} },
     _startAt: [ 1119449, 442403521 ],
     _startTime: 2019-11-08T19:09:40.413Z,
     _remoteAddress: '127.0.0.1',
     route: Route { path: '/:type(media)', stack: [Array], methods: [Object] } },
  serverRes:
   ServerResponse {
     domain: null,
     _events: { end: [Array], finish: [Array] },
     _eventsCount: 2,
     _maxListeners: undefined,
     output: [],
     outputEncodings: [],
     outputCallbacks: [],
     outputSize: 0,
     writable: true,
     _last: false,
     chunkedEncoding: false,
     shouldKeepAlive: true,
     useChunkedEncodingByDefault: false,
     sendDate: true,
     _removedConnection: false,
     _removedContLen: false,
     _removedTE: false,
     _contentLength: null,
     _hasBody: true,
     _trailer: '',
     finished: false,
     _headerSent: false,
     socket:
      Writable {
        _writableState: [WritableState],
        writable: true,
        _write: [Function: write],
        domain: null,
        _events: [Object],
        _eventsCount: 2,
        _maxListeners: undefined,
        _httpMessage: [Circular] },
     connection:
      Writable {
        _writableState: [WritableState],
        writable: true,
        _write: [Function: write],
        domain: null,
        _events: [Object],
        _eventsCount: 2,
        _maxListeners: undefined,
        _httpMessage: [Circular] },
     _header: null,
     _onPendingData: [Function: noopPendingOutput],
     _sent100: false,
     _expect_continue: false,
     write: [Function],
     req:
      IncomingMessage {
        _readableState: [ReadableState],
        readable: false,
        domain: null,
        _events: {},
        _eventsCount: 0,
        _maxListeners: undefined,
        socket: [Object],
        connection: [Object],
        httpVersionMajor: '1',
        httpVersionMinor: '1',
        httpVersion: '1.1',
        complete: true,
        headers: [Object],
        rawHeaders: [],
        trailers: {},
        rawTrailers: [],
        upgrade: null,
        url: '/media',
        method: 'GET',
        statusCode: null,
        statusMessage: null,
        client: [Object],
        _consuming: false,
        _dumped: false,
        ip: '127.0.0.1',
        body: <Buffer >,
        requestContext: [Object],
        serverless: [Object],
        next: [Function: next],
        baseUrl: '',
        originalUrl: '/media',
        _parsedUrl: [Url],
        params: [Object],
        query: {},
        res: [Circular],
        _startAt: [Array],
        _startTime: 2019-11-08T19:09:40.413Z,
        _remoteAddress: '127.0.0.1',
        route: [Route] },
     locals: {},
     _startAt: undefined,
     _startTime: undefined,
     writeHead: [Function: writeHead],
     __onFinished: { [Function: listener] queue: [Array] },
     [Symbol(isCorked)]: false,
     [Symbol(outHeadersKey)]:
      { 'x-powered-by': [Array],
        'access-control-allow-origin': [Array] },
     [Symbol()]: [],
     [Symbol()]: {} },
  makeDocument: [Function],
  beforeSave: [Function],
  beforeRender: [Function],
  transformDocument: [Function: transformDocument],
  makeQuery: [Function: makeQuery],
  runQuery: [Function],
  setTypePaths: [Function: setTypePaths] }

OrigQuery: {"query":{"type":"media","criteria":{"where":{"type":"FieldExpression","operator":"and","args":[{"field":"user_name","operator":"eq","value":"somename"}]},"isSingular":false},"populates":[],"ignoreLimitMax":false}}

@gabrielsperezz
Copy link

gabrielsperezz commented Jan 22, 2020

+1

@gabrielsperezz
Copy link

@ethanresnick
Copy link
Owner

@sohamdodia, @emac08, and others in this thread... sorry for my non-responsiveness here. I went traveling shortly after this issue was reported and was not at a computer for a couple weeks; when I got back, life intervened hard. Did any of you figure out the cause of this issue (if you stuck with this library)? If so, I'd love to hear it and to incorporate a fix.

@amitassaraf
Copy link

@ethanresnick This is still happening, even in the example. Is this library maintained? I am starting to wonder whether I should use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants