gajus/liqe

Parsing of Field Grouping not working

Open

#30 aperta il 17 feb 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (27 fork)github user discovery
help wanted

Metriche repository

Star
 (675 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Problem

  • Parsing of Field Grouping throws error
  • Error is thrown as well in case of +/- prefix and missing field name
  • All examples below parse correctly with Java Lucene parser
  • Syntax is in line with:
  • Fixing this would:
    • Increase compatibility with Lucene
    • Provide alternative for providing list of values in format a:(b OR c OR d) or a:(b c d) (later in case of possibility to change default operator to OR)

Example of failing searches

  • +(a b)
  • -(a b)
  • c:(a b)
  • +c:(a b)
  • -c:(a b)

Versions 3.5.0 - on https://npm.runkit.com/liqe 3.6.0 - with Angular 14.0.4

Demonstration

https://runkit.com/embed/msogu7iiwefg

Guida contributor

Direzione di ricerca
Ispeziona la gestione del parser per il raggruppamento dei campi e i prefissi nel codice. Concentrati sulle regole del lexer o parser per 'field:(...)' e prefissi '+/ '. Aggiungi casi di test per gli esempi che falliscono e traccia il percorso dell'errore.
Tech stack
typescript
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65