gajus/liqe

Parsing of Field Grouping not working

Open

#30 geöffnet am 17. Feb. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (27 Forks)github user discovery
help wanted

Repository-Metriken

Stars
 (675 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Behandlung von Feldgruppierungen und Präfixen im Parser Code. Fokussiere auf die Lexer oder Parser Regeln für 'field:(...)' und '+/ Präfixe'. Füge Testfälle für die fehlschlagenden Beispiele hinzu und verfolge den Fehlerpfad.
Tech Stack
typescript
Domain
api
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65