Apollo-link-rest: Kueri melempar kesalahan saat nilai parameter salah (0, salah, nol, tidak terdefinisi)

Dibuat pada 26 Feb 2018 · 4Komentar · Sumber: apollographql/apollo-link-rest

Saya mendapatkan kesalahan berikut saat memberikan nilai 0 pada variabel.
Kesalahan: Kesalahan jaringan: Parameter tidak ada untuk menjalankan kueri, tentukan dalam parameter kueri, atau gunakan perintah ekspor

Jika sebaliknya saya menggunakan string itu berfungsi.

Saya telah men-debug kode dan menemukan masalah dalam fungsi replaceParam yang memeriksa !value mana (saya kira ) harus memeriksa null / undefined nilai.

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

Saya akan mengirimkan PR

bug 🐛

Sumber

gaguirre

Semua 4 komentar

Tangkapan yang bagus, cara TypeScript untuk ini adalah

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

Saya berharap untuk menggabungkan PR Anda! :)

Harap tambahkan uji unit sehingga kami dapat menangkap regresi di masa mendatang.

fbartho pada 26 Feb 2018

👍1

Jelas bagi saya bahwa 0 dan false mewakili nilai nyata dan undefined yang hilang, tetapi tidak yakin tentang nilai null .
Bagaimana menurutmu @fbartho?

gaguirre pada 26 Feb 2018

@gaguirre Maaf saya melewatkan balasan Anda! Saya mengirimkan kode Anda apa adanya. Saya pikir kita mungkin harus mendukung melewati null , dan membiarkan lapisan yang berbeda menanganinya.

Pernahkah Anda melihat fungsi pathBuilder ? Itu adalah kode yang disediakan pengguna yang peduli dengan perbedaan antara undefined dan null , atau sebenarnya dapat menggunakan null bermanfaat.

Apakah Anda pikir kita harus membuka tiket baru untuk melacak kasus itu?

fbartho pada 27 Feb 2018

@fbartho tidak masalah! Saya setuju bahwa null adalah kasus yang valid dan saya telah mempertimbangkannya di PR.
Satu-satunya hal yang hilang adalah kasus uji untuk kasus undefined , tetapi perilaku yang menurut saya benar: akan menimbulkan kesalahan _Parameter yang hilang_. 👍

gaguirre pada 27 Feb 2018

👍1

Apakah halaman ini membantu?

0 / 5 - 0 peringkat