Bug in error handling in Shopify::Rest::Base#save
@mllemango ya está trabajando en esto.
Desde el 5/9/2023.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Issue summary
Since v13.0.0 ShopifyAPI::DiscountCode#errors signature has been changed to Hash, and #errors is initialized as Hashnil for DiscountCode resources.
This causes exception in error handling in ShopifyAPI::Rest::Base#save if ShopifyAPI::Errors::HttpResponseError is raised during save operation for DiscountCode resource. Exception is raised because Hashnil does not implement #errors
shopify_apiversion: 13.1.0- Ruby version: 2.7.7
Expected behavior
Rescuing from ShopifyAPI::Errors::HttpResponseError does not raise exception
Actual behavior
#<NoMethodError: undefined method `errors' for {}:Hash> is raised
#<NoMethodError: undefined method `errors' for nil:NilClass> is raised
Edit: corrected details
- Lenguaje dominante
- Ruby
- Estrellas
- 1.1k
- Forks
- 484
- Merge medio
- 4 h 38 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Shopify/shopify-api-ruby
-
devtools-gardener
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Shopify/shopify-api-ruby#1456 · 3 reacciones ·
-
devtools-gardener
Dificultad 3/5 1-2 días Aptitud para principiantes 67/100
Shopify/shopify-api-ruby#1461 · 1 comentario ·
-
automated
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Shopify/shopify-api-ruby#1460 ·
-
devtools-gardener
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Shopify/shopify-api-ruby#1455 ·
-
devtools-gardener
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Shopify/shopify-api-ruby#1453 · 1 comentario ·
Todos los issues de Shopify/shopify-api-ruby
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
solana-foundation/pay-kit#341 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
Los mantenedores suelen responder en 1 día
-
Sign and read plain-text assetsAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Cask still fails to install: `depends_on macos: :catalina` is now disabled (regression after #58)Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100