NULL_BOOLEAN is not compatible with Value Objects

Ouverte
  • Quelque chose ne va pas ?
#2,185 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
35/100
Type d'issue
Bug
Clarté
À clarifier
Activité
Calme
Stack technique
java
Domaine
api

Piste de recherche

Commencez dans google-http-client/src/main/java/com/google/api/client/util/Data.java, en particulier avec Data.isNull et NULL_BOOLEAN, et reproduisez les Assertions présentées dans l’issue avec la préversion des value classes de JDK 28. Comparez les deux approches proposées ; le travail est terminé lorsque les valeurs Boolean, Integer et Long légitimes ne renvoient plus true depuis Data.isNull et que le suivi des valeurs absentes et explicitement nulles reste correct.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.
Langage dominant
Java
Étoiles
1.4k
Forks
476
Merge moyen
10 h 47 min
PR mergées (30 j)
3

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de googleapis/google-http-java-client

Toutes les issues de googleapis/google-http-java-client

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.