Returned error in result missing error subject (discovered in metafield create/update)

Offen

#204 erstellt am 12.7.2017 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne in activeresource.py bei der Klasse Errors und ihrer from_hash-Methode, und untersuche dann, wie verschachtelte Fehlerschlüssel entpackt werden. Reproduziere den Fall metafield-on-variant mit der gezeigten metafields.namespace-Antwort. Fertig ist es, wenn der zurückgegebene Fehler den subject-Schlüssel beibehält, anstatt das Feld zu verlieren, das den Fehler verursacht.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

feature request

The shopify API returns the subject of an error as a key in the json error string, and pyactiveresource sometimes adds the value but not the key when it unpacks the string.

This probably shows up for some errors generated when adding or modifying variants or metafields, as they show up as lists of hashes in the object attributes. I hit a problem when adding or updating a metafield on a variant, and decided to dig.

The issue is in class Errors in activeresource.py:

    def from_hash(self, messages):
        attribute_keys = self.base.attributes.keys()
        for key, errors in six.iteritems(messages):
            for message in errors:
                if key in attribute_keys:
                    self.add(key, message)
                else:
                    self.add_to_base(message)

An error return like this:

{
    "errors": {
        "metafields.namespace": ["can't be blank", "is too short (minimum is 3 characters)"]
    }
}

(caused by trying to create or update a metafield with the key "name-space" instead of "namespace" :) will lose the important tidbit, the field causing the issue: "metafields" is a key in the variant attributes, but "metafields.namespace" is not.

Perhaps this might be a more robust approach?

    def from_hash(self, messages):
        attribute_keys = self.base.attributes.keys()
        for key, errors in six.iteritems(messages):
            for message in errors:
                if key.split('.')[0] in attribute_keys:
                    self.add(key, message)
                else:
                    self.add_to_base(' '.join([key, message]))

Thanks!

Rick

Vorherrschende Sprache: Python
Sterne: 1.4k
Forks: 388
Ø Merge: 5 Std. 39 Min.
Gemergte PRs (30 T.): 1

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Shopify/shopify_python_api

A new release of the Shopify API is due soon Offen

automated

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

Shopify/shopify_python_api#799 · Erstellt am 01.9.2026
A new release of the Shopify API occurred Offen

automated

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

Shopify/shopify_python_api#796 · 1 Kommentar · Erstellt am 01.7.2026
A new release of the Shopify API occurred Offen

automated

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

Shopify/shopify_python_api#792 · Erstellt am 01.4.2026
Account Activation URL Offen

feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100

Shopify/shopify_python_api#275 · 4 Kommentare · Erstellt am 09.10.2018
prerequisite_customer_ids array is ignored Offen

feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 38/100

Shopify/shopify_python_api#233 · Erstellt am 04.2.2018

Alle Issues in Shopify/shopify_python_api