microsoftgraph / microsoftgraph/msgraph-sdk-php
ChangeType enum not handling capitalized values
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
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Ö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