[Bug]: Unknown properties return None
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 48/100
Línea de trabajo
Comienza en Adyen/services/base.py, en las líneas 13-16, la ubicación de la causa raíz identificada en el issue. Reproduce los ejemplos de atributos ausentes en las subclases de AdyenBase e inspecciona cómo se gestionan las claves desconocidas. Se considera terminado cuando los atributos inexistentes generan AttributeError en lugar de devolver None, incluidos los objetos de API anidados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
Accessing a non-existent property on an API returns None instead of raising an AttributeError
Steps to reproduce
Attempt to access any non-existent attribute on any subclass of AdyenBase
>>> from Adyen import Adyen
>>> api = Adyen(xapikey='token', platform='test')
>>> print(api.foobar)
None
>>> print(api.payment.payments_api.foobar)
None
>>> api.payment.payments_api.fizz()
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: 'NoneType' object is not callable
Actual behavior
Attempting to access non-existent attribute returns None. Attempting to call a non-existent method throws a "NoneType object is not callable" error.
Expected behavior
Should return an AttributeError.
Code snippet or screenshots (if applicable)
Root cause is here:
If key doesn't exist in self.client, the method exits with no return statement, which is the same as returning None. This means any attribute that's requested defaults to None. It should instead raise AttributeError
Adyen Python API Library version
14.0.0
Python Language version
Python 3.10
Operating System
macOS
Additional context
No response
- Lenguaje dominante
- Python
- Estrellas
- 68
- Forks
- 51
- Merge medio
- 2 h 3 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 Adyen/adyen-python-api-library
-
[Bug] api_*_version override has no effect on PAL-based service URLsPosiblemente ocupada @cobanfurkanx la tomó hace 8 días. AbiertoBug do not stale
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
[Feature]: Foreign Exchange APIAbiertoFeature
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Adyen/adyen-python-api-library#487 · 1 comentario · 1 reacción ·
-
[Test] Add baseUrl environment tests to all service test filesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodo not stale Test
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
-
chore: consolidate package metadata into pyproject.toml (PEP 621)Posiblemente ocupada @erdemtuna la tomó hace 190 días. Abiertodo not stale
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
do not stale Feature
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Adyen/adyen-python-api-library#395 · 1 reacción ·
Todos los issues de Adyen/adyen-python-api-library
Issues similares
-
customer-reported question
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Azure/azure-sdk-for-python#49548 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
btclib-org/btclib-wallet#409 ·
Los mantenedores suelen responder en 1 día
-
Phase 2 PSA Capability & Career Profile
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
bcgov/digital-journeys#2310 ·
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
fastmachinelearning/hls4ml#1582 ·
Los mantenedores suelen responder en 3 días
-
Sort modelsAbiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día