Filter values for related resources are wrapped in nested arrays?

Offen

#1,110 erstellt am 08.9.2017 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: rails, ruby
Bereich: api, backend

Rechercherichtung

Beginne in lib/jsonapi/resource.rb ungefähr bei Zeile 753 und vergleiche die Behandlung von Filterwerten mit Commit 3a691b2. Reproduziere den im Issue beschriebenen Fall mit einem benutzerdefinierten Filter und prüfe, ob Resource#verify_resource verschachtelte Arrays verursacht. Erledigt ist die Aufgabe, wenn Filterwerte für verknüpfte Ressourcen nicht unnötig verpackt werden und das Parsen von Zeichenketten weiterhin funktioniert; im Issue wird keine Testdatei genannt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

I'm a little confused as to what's going on, but upgrading from 0.8.3 to 0.9 broke some filters I have in place. e.g.:

VERIFY_RADIUS_FILTER = ->(value, context) {
  return NoLimit.new unless context[:current_location].present?

  value
}

FILTER_RADIUS_FROM_LOCATION = ->(records, value, options) {
  location = options.dig(:context, :current_location)
  limit = Array(value).first

  return records unless location.present? && limit.present?

  records.near(location, Float(limit))
}

filter :radius, default: 100.0,
                verify: VERIFY_RADIUS_FILTER,
                apply: FILTER_RADIUS_FROM_LOCATION

I think this commit my be the culprit? https://github.com/cerebris/jsonapi-resources/commit/3a691b29adb3e7aff0743ef59e44556bedb260c9

From what I can tell, this results in Resource#verify_resource being called twice. For a custom filter, at least (which is what I've got), that means the raw value gets doubly-nested in an array: https://github.com/cerebris/jsonapi-resources/blob/19f4d7b59f92b05fa28c4f925391cf5998123308/lib/jsonapi/resource.rb#L753

Instead I wonder if that line should be something like:

filter_values += raw.is_a?(String) ? CSV.parse_line(raw) : Array(raw)

To make sure we don't wrap an array?

/cc @hidde-jan

Vorherrschende Sprache: Ruby
Sterne: 2.3k
Forks: 546
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus JSONAPI-Resources/jsonapi-resources

Fix incorrect translation key for parameter_not_allowed.detail Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100

JSONAPI-Resources/jsonapi-resources#1468 · Erstellt am 28.2.2025
ArgumentError with Rails 8.1 Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 50/100

JSONAPI-Resources/jsonapi-resources#1488 · 1 Kommentar · Erstellt am 12.5.2026
Rails 8.1: jsonapi_resources DSL fails to generate relationship routes due to Resource#initialize signature change Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100

JSONAPI-Resources/jsonapi-resources#1479 · 2 Kommentare · Erstellt am 08.11.2025
Getting invalid status code in Rails 8.0.2 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100

JSONAPI-Resources/jsonapi-resources#1477 · 3 Kommentare · 1 Reaktion · Erstellt am 29.10.2025
Fields filtering with include Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

JSONAPI-Resources/jsonapi-resources#1475 · Erstellt am 22.9.2025

Alle Issues in JSONAPI-Resources/jsonapi-resources