NULL_BOOLEAN is not compatible with Value Objects

Offen
  • Stimmt etwas nicht?
#2,185 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
35/100
Issue-Typ
Bug
Klarheit
Muss geklärt werden
Aktivitätsstatus
Ruhig
Tech-Stack
java
Bereich
api

Rechercherichtung

Beginnen Sie in google-http-client/src/main/java/com/google/api/client/util/Data.java, insbesondere bei Data.isNull und NULL_BOOLEAN, und reproduzieren Sie die im Issue gezeigten Assertions mit der Value-Class-Preview von JDK 28. Vergleichen Sie die beiden vorgeschlagenen Ansätze; abgeschlossen ist die Aufgabe, wenn gültige Boolean-, Integer- und Long-Werte von Data.isNull nicht mehr true zurückgeben und die Nachverfolgung von fehlenden und explizit null gesetzten Werten weiterhin korrekt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

google-http-client uses sentinel objects like NULL_BOOLEAN to distinguish between fields being absent and explicitly set to null:

https://github.com/googleapis/google-http-java-client/blame/95fac312dd743a94fb28532ef8a50d374e6481ad/google-http-client/src/main/java/com/google/api/client/util/Data.java#L48-L49

Those sentinel fields are using deprecated constructors like new Boolean(true) (the deprecation warnings were suppressed in https://github.com/googleapis/google-http-java-client/pull/1215).

JDK 28 introduces a preview of the value classes feature (https://openjdk.org/jeps/401). With JEP 401 boxed primitive classes like Boolean and Integer become value classes that don't have identity, so instance are compared by value:

  • new Boolean(true) == Boolean.TRUE will evaluate to true
  • new Integer(0) == Integer.valueOf(0) will evaluate to true

This causes google-http-client's Data.isNull(object) to return true for legitimate values.

With JEP 401, assertions like these will start to fail:

    assertThat(Data.isNull(true)).isFalse(); 
    assertThat(Data.isNull(0)).isFalse();
    assertThat(Data.isNull(0L)).isFalse();

Possible fixes:

  • Introduce a new API like JsonNullable<Boolean> to replace the use of NULL_BOOLEAN
  • Maintain a set of explicitly set fields, that could be queried instead of relying on sentinel values.
Vorherrschende Sprache
Java
Sterne
1.4k
Forks
476
Ø Merge
10 Std. 47 Min.
Gemergte PRs (30 T.)
3

Entwicklungsumgebung

Erste Schritte

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

Mehr aus googleapis/google-http-java-client

Alle Issues in googleapis/google-http-java-client

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.