Revert graph filters

[NolanTrem]

Important

Reverts graph filter logic in graphs.py, removing apply_filters and reintroducing custom filter parsing in graph_search() and _build_filters().

• Behavior:
  • Reverts graph filter logic in graph_search() and _build_filters() in graphs.py.
  • Removes apply_filters function usage, reintroducing custom filter parsing.
  • Handles $eq, $ne, $lt, $lte, $gt, $gte, $in, $contains operations in _build_filters().
• Exceptions:
  • Removes UndefinedColumnError handling in graph_search().
  • Continues to handle general exceptions with HTTP 500 error in graph_search().

^{This description was created by for f349869. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f349869 in 2 minutes and 59 seconds

More details

• Looked at 211 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. py/core/database/graphs.py:15

• Draft comment:
  The import for apply_filters is removed, and _build_filters is re-implemented. This aligns with the PR's intent to revert graph filters.
• Reason this comment was not posted:
  Confidence changes required: 20%
  The import statement for apply_filters is removed, but the function _build_filters is re-implemented in this PR. This change is consistent with the PR title 'Revert graph filters', indicating a reversion to a previous implementation. The removal of apply_filters is intentional and aligns with the PR's purpose.

2. py/core/database/graphs.py:2602

• Draft comment:
  The exception handling for UndefinedColumnError is removed. Ensure this aligns with the intended error handling strategy.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The exception handling for UndefinedColumnError is removed, which might lead to unhandled exceptions if such an error occurs. This change should be reviewed to ensure it aligns with the intended behavior of the application.

3. py/core/database/graphs.py:2614

• Draft comment:
  The _build_filters function is re-implemented. Review for correctness in handling various operators and data types.
• Reason this comment was not posted:
  Confidence changes required: 40%
  The _build_filters function is re-implemented, and it includes a detailed parsing of filter conditions. This implementation should be reviewed for correctness and completeness, especially in handling various operators and data types.

Workflow ID: wflow_PbOS5bjMNb7kbEC6

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.