vadimdemedes/mongorito

query.lte overwrites query.gte and vice versa

Open

#104 aberto em 10 de abr. de 2016

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (1.397 stars) (104 forks)batch import
bughelp wanted

Description

It seems like query.lte overwrites query.gte and vice versa. Steps to reproduce:

const Model = require('mongorito').Model; let query = Model.gte('updated_at', new Date()); query = Model.lte('updated_at', new Date()); console.log(query.query); // { created_at: { '$lte': Sun Apr 10 2016 11:09:32 GMT+0200 (CEST) } }

Guia do colaborador

Pilha de tecnologia
javascriptnodejsmongodb
Domain
backend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScript basicsMongoDB query syntax
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
The bug is in the query builder methods `gte` and `lte` in mongorito, likely in `lib/model.js` or similar. Currently, each method overwrites the other's condition in the query object. The fix should ensure that both conditions are stored separately, such as merging into an existing `$and` array or using separate keys. Review the source code in the repository to understand the query construction and implement a solution that correctly combines both `$gte` and `$lte` operators in the same query.