dotnet/runtime

Support using any dictionary as JsonExtensionData

Offen

#31.645 geöffnet am 03.02.2020

Auf GitHub ansehenContributor Guide
 (23 Kommentare) (8 Reaktionen) (1 zugewiesene Person)C# (5.445 Forks)batch import
area-System.Text.Jsonenhancementhelp wantedwishlist

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

IMHO, IReadOnlyDictionary should work identical to IDictionary when using with JsonExtensionData attribute.

[JsonExtensionData]
public IReadOnlyDictionary<string, JsonElement> Payload { get; set; }

Currently it fails with error:

System.InvalidOperationException: The data extension property 'Payload' does not match the required signature of IDictionary<string, JsonElement> or IDictionary<string, object>.

This is analogous to collection properties which can be defined as List<T>, IList<T> or IReadOnlyList<T> etc.

Contributor Guide

Research-Richtung
Untersuchen Sie den System.Text.Json Quellcode für die JsonExtensionData Behandlung. Verstehen Sie, wie IDictionary<string, JsonElement> derzeit unterstützt wird, und identifizieren Sie die Änderungen, die erforderlich sind, um auch IReadOnlyDictionary<string, JsonElement> zu unterstützen. Suchen Sie nach der Validierungslogik, die den Eigenschaftstyp überprüft, und erweitern Sie diese entsprechend.
Tech Stack
csharp
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C#.NETJSON serialization
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65