CyberSource / CyberSource/cybersource-rest-samples-python
__dict__ vs to_dict()
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 29
- Forks
- 34
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hey, this might be a strange thing to open an issue for, but I'm using the SDK to transition from Authorize.net AIM to CyberSource, and I noticed that all the examples use things like:
request.payment_information = payment_info.__dict__
# etc
message_body = json.dumps(request.__dict__)
instead of:
request.payment_information = payment_info
# etc
message_body = json.dumps(request.do_dict())
...even though the SDK provides to_dict() for a whole lot of things, so it looks like it was designed for that (and it avoids including unwanted attributes by accident). It looks like both ways produce the same output and work on the process_payment.py sample.
Is there a reason to prefer __dict__ each time over a single to_dict() at the end?
Thanks
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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
Beginnen Sie mit dem process_payment.py-Beispiel und den SDK-Objekten, die to_dict() bereitstellen; vergleichen Sie die beiden im Issue beschriebenen Serialisierungspfade. Als abgeschlossen gilt die Arbeit erst mit einer von einem Maintainer bestätigten Empfehlung und, falls sich Beispiele ändern sollten, einer klar identifizierten Dokumentations- oder Beispielaktualisierung; im Issue ist keine Zieldatei und kein Test genannt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100