Apollo-link-rest: يطرح الاستعلام خطأ عندما تكون قيمة المعلمة خاطئة (0 ، خطأ ، فارغ ، غير محدد)

تم إنشاؤها على ٢٦ فبراير ٢٠١٨ · 4تعليقات · مصدر: apollographql/apollo-link-rest

أتلقى الخطأ التالي عند تمرير قيمة 0 لمتغير.
خطأ: خطأ في الشبكة: معلمات مفقودة لتشغيل الاستعلام ، حددها في معلمات الاستعلام أو استخدم توجيه تصدير

إذا استخدمت بدلاً من ذلك سلسلة فإنها تعمل.

لقد قمت بتصحيح الكود ووجدت المشكلة في الوظيفة replaceParam التي تتحقق من !value حيث (أعتقد 😃) يجب أن يتحقق من null / undefined قيم

var replaceParam = function (endpoint, name, value) {
    if (!value || !name) {
        return endpoint;
    }
    return endpoint.replace(":" + name, value);
};

سأرسل العلاقات العامة

bug 🐛

مصدر

gaguirre

ال 4 كومينتر

صيد جيد ، الطريقة المطبوعة لذلك هي

if (value != null || name != null)

إنني أتطلع إلى دمج العلاقات العامة الخاصة بك! :)

الرجاء إضافة اختبار وحدة حتى نتمكن من رصد أي تراجع في المستقبل.

fbartho في ٢٦ فبراير ٢٠١٨

👍1

من الواضح بالنسبة لي أن 0 و false يمثلان قيمًا حقيقية و undefined قيمة مفقودة ، لكن لست متأكدًا من قيم null .
ما رأيكfbartho؟

gaguirre في ٢٦ فبراير ٢٠١٨

gaguirre آسف فاتني ردك! لقد شحنت الرمز الخاص بك كما هو. أعتقد أننا ربما يجب أن ندعم تمرير null ، والسماح لطبقة مختلفة بالتعامل معه.

هل رأيت وظيفة pathBuilder ؟ هذا هو الكود الذي قدمه المستخدم والذي يمكن أن يهتم بالتمييز بين undefined و null ، أو يمكنه بالفعل استخدام null بشكل مفيد.

هل تعتقد أننا يجب أن نفتح تذكرة جديدة لتتبع هذه الحالة؟

fbartho في ٢٧ فبراير ٢٠١٨

fbartho لا مشكلة! أوافق على أن null حالة صالحة وقد اعتبرت ذلك في PR.
الشيء الوحيد المفقود هو حالات الاختبار لحالة undefined ، لكن السلوك الذي أعتقد أنه صحيح: سيؤدي إلى ظهور خطأ _معلمة مفقودة_. 👍

gaguirre في ٢٧ فبراير ٢٠١٨

👍1

هل كانت هذه الصفحة مفيدة؟

0 / 5 - 0 التقييمات