Mongoose: Opciones de rupturas de poblado anidado virtual en el poblado principal.

Creado en 27 may. 2018 · 3Comentarios · Fuente: Automattic/mongoose

Creo que hay un error al utilizar opciones y rellenos anidados.

Mi consulta actual es la siguiente:

...
await Post.find(
          { teacher: req.params.id },
          {},
          {
            sort: { createdAt: -1 },
            limit: 5,
            skip: parseInt(req.query.skip, 10)
          }
        ).populate([
        {
          path: 'comments',
          options: {
            limit: 5,
            sort: { createdAt: -1 }
          },
          populate: { path: 'user', select: '-password' }
        }
      ])
...

Cuando se incluye options en el comments poblar (que tiene otro poblar anidado dentro), devuelve más de lo especificado limit y sort no se aplica. Si se quita el poblado anidado, funciona según lo previsto.

Actualmente, usando:

Nodo 10.2.1
Mangosta 5.1.2
Mongo 3.6.5

¿Es esto un error o me falta algo?

Creo que esto podría estar relacionado: https://github.com/Automattic/mongoose/issues/4321

confirmed-bug

Fuente

Mikeysax

Comentario más útil

@Mikeysax Creé esta esencia para replicar esto lo mejor que pude de tu ejemplo.

Se puede clonar localmente en una carpeta llamada 6528 de la siguiente manera:

issues: git clone https://gist.github.com/7dee38490d8c637976d8fb0805aafb1e ./6528
Cloning into './6528'...
remote: Counting objects: 5, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 5 (delta 0), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (5/5), done.
issues: ls !$
ls ./6528
6528.js          models6528.js    populate6528.js
issues:

Producción:

6528>: chmod +x populate6528.js
6528>: chmod +x 6528.js
6528>: ./populate6528.js
created 10 users
created 10 teachers
created 100 posts
created 1000 comments
6528>: ./6528.js
{ AssertionError [ERR_ASSERTION]: 10 === 5
    at run (/Users/lineus/dev/Help/mongoose5/issues/6528/6528.js:38:10)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
  generatedMessage: true,
  name: 'AssertionError [ERR_ASSERTION]',
  code: 'ERR_ASSERTION',
  actual: 10,
  expected: 5,
  operator: '===' }
^C
6528>:

Viajaré la mayor parte del día, pero podré investigarlo esta noche o mañana a primera hora.

lineus en 28 may. 2018

👍2

Todos 3 comentarios

@Mikeysax Creé esta esencia para replicar esto lo mejor que pude de tu ejemplo.

Se puede clonar localmente en una carpeta llamada 6528 de la siguiente manera:

issues: git clone https://gist.github.com/7dee38490d8c637976d8fb0805aafb1e ./6528
Cloning into './6528'...
remote: Counting objects: 5, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 5 (delta 0), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (5/5), done.
issues: ls !$
ls ./6528
6528.js          models6528.js    populate6528.js
issues:

Producción:

6528>: chmod +x populate6528.js
6528>: chmod +x 6528.js
6528>: ./populate6528.js
created 10 users
created 10 teachers
created 100 posts
created 1000 comments
6528>: ./6528.js
{ AssertionError [ERR_ASSERTION]: 10 === 5
    at run (/Users/lineus/dev/Help/mongoose5/issues/6528/6528.js:38:10)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
  generatedMessage: true,
  name: 'AssertionError [ERR_ASSERTION]',
  code: 'ERR_ASSERTION',
  actual: 10,
  expected: 5,
  operator: '===' }
^C
6528>:

Viajaré la mayor parte del día, pero podré investigarlo esta noche o mañana a primera hora.

lineus en 28 may. 2018

👍2

Para que quede claro, mi corrección aplica el objeto de opciones a la consulta como se esperaba, pero no soluciona el problema en # 4321. Pensaré un poco en ese tema por separado y enviaré un PR si tengo alguna idea que valga la pena compartir ( es un hueso duro de roer ).

lineus en 30 may. 2018

Sí, limitar y poblar ha sido una espina clavada en nuestro costado durante mucho tiempo. La idea general es que tendremos que recurrir a la realización de consultas por separado para cada documento si usa populate() con limit y recupera varios documentos.