fix: @_helpers.positional raises TypeError when decorating functions without default arguments
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 88/100
Piste de recherche
L’issue indique googleapiclient/_helpers.py et inclut une reproduction utilisant @_helpers.positional. Lisez l’inférence des arguments de positional() présentée dans l’issue, puis exécutez les tests existants pertinents pour ce helper. Le travail sera terminé lorsque la décoration d’une fonction sans arguments par défaut ou sans paramètres ne déclenchera plus de TypeError et que les tests existants du helper réussiront.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
When @googleapiclient._helpers.positional is used as a bare decorator without arguments (e.g., @positional) on a callable that does not define default argument values (or takes no parameters), it raises TypeError: object of type 'NoneType' has no len() at function definition time.
Current Behavior
In googleapiclient/_helpers.py, positional() infers the maximum number of positional arguments when called without arguments:
if isinstance(max_positional_args, int):
return positional_decorator
else:
args, _, _, defaults, _, _, _ = inspect.getfullargspec(max_positional_args)
return positional(len(args) - len(defaults))(max_positional_args)
When max_positional_args has no default parameters (or takes 0 parameters), inspect.getfullargspec(max_positional_args).defaults returns None.
Evaluating len(defaults) then raises:
TypeError: object of type 'NoneType' has no len()
Expected Behavior
When @positional is used without arguments on a function without default parameters (such as def func(a, b): ...), defaults should default to an empty sequence (e.g., len(defaults or ())), correctly calculating len(args) - 0 = len(args) positional parameters instead of crashing with a TypeError.
Reproduction
from googleapiclient import _helpers
@_helpers.positional
def my_function(a, b):
return a + b
Result:
TypeError: object of type 'NoneType' has no len()
Technical Context
In googleapiclient/_helpers.py, replacing len(defaults) with len(defaults or ()) safely treats None as 0 default arguments:
args, _, _, defaults, _, _, _ = inspect.getfullargspec(max_positional_args)
return positional(len(args) - len(defaults or ()))(max_positional_args)
- Langage dominant
- Python
- Étoiles
- 9k
- Forks
- 2.6k
- Merge moyen
- 8 h 36 min
- PR mergées (30 j)
- 7
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de googleapis/google-api-python-client
-
fix: @_helpers.positional raises TypeError when decorating functions without default argumentsOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
googleapis/google-api-python-client#2844 ·
Les mainteneurs répondent en général sous 1 jour
-
Disable prettyPrint by default for Google Sheets API responsesPeut-être pris @pawciobiel l’a pris il y a 69 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
googleapis/google-api-python-client#2788 ·
Les mainteneurs répondent en général sous 1 jour
-
test: connection leak test test_discovery_http_is_closed is shadowed and syntactically invalidPeut-être pris @Capstan l’a pris il y a 144 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
googleapis/google-api-python-client#2757 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
test: strict positional parameter enforcement is ignored under pytestPeut-être pris @Capstan l’a pris il y a 144 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
googleapis/google-api-python-client#2755 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
googleapis/google-api-python-client#2825 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de googleapis/google-api-python-client
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
CLIMADA-project/climada_python#1337 ·
Les mainteneurs répondent en général sous 3 jours
-
tail without reverseOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
8go/matrix-commander#233 ·
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
-
documentation enhancement priority:high size:M
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Validate non-divisible effective batch sizes in the Unsloth LoRA backendPeut-être pris @debesh-biswas l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100