microsoftgraph / microsoftgraph/msgraph-sdk-php

ChangeType enum not handling capitalized values

Offen
#1,426 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
PHP
Sterne
669
Forks
150
Ø Merge
15 Std. 21 Min.
Gemergte PRs (30 T.)
3

Beschreibung

We are using v1 (1.109.2) and are seeing problems when handling change notifications: "unhandled enum value Updated"
Model/ChangeType.php is stating that it only supports updated/deleted/created (note the lower case words).
Notifications apparently have Updated/Deleted/Created (since a new update on the Graph servers ?).

We are seeing this when calling ChangeNotification::getChangeType().
The ChangeNotification object has been created when handling the incoming request that we receive, something like this:

$notificationData = $request->getDecodedJsonContent();
$notification = new ChangeNotification($notificationData);
$changeType = $notification->getChangeType();     <===== Throws!

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit Model/ChangeType.php und dem im Bericht genannten Pfad ChangeNotification::getChangeType(). Reproduziere den Fehler mit einer Benachrichtigung, die den großgeschriebenen Wert Updated, Deleted oder Created enthält, und überprüfe anschließend, dass jeder Wert ohne den Fehler aufgrund eines nicht behandelten enum-Werts verarbeitet wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php
Bereich
api
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

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