Peut-être une erreur de programmation

karutaproject / karuta-backend_legacy

Data repository for Karuta. Can write into MySQL and Oracle databases.

Other

10 stars 15 forks source link

Closed benjamin-feron closed 2 years ago

benjamin-feron commented 2 years ago

Bonjour,

Dans le fichier /src/com/portfolio/rest/RestServicePortfolio.java à la ligne 4300, on peut lire :

if (ldapvalues[1] != null | ldapvalues[2] != null | ldapvalues[3] != null)

Ne faudrait il pas plutôt doubler les "|" comme ceci ? :

if (ldapvalues[1] != null || ldapvalues[2] != null || ldapvalues[3] != null)

Avec de simples "|", il s'agit d'un OU binaire.

Voici le commit correspondant.

Benjamin Féron

nobry commented 2 years ago

Dans la finalité le code s'execute correctement, mais j'avoue que c'est plus propre avec le OU logique. J'ai fais les changements dans le commit https://github.com/karutaproject/karuta-backend_legacy/commit/c08bd69a96608918a406ca39fc97954b6ff59ad2

Merci du commentaire.

benjamin-feron commented 2 years ago

Avec plaisir