geocoder-php / geocoder-php/Geocoder
IpInfoDb - Uncaught TypeError: Argument 11 passed to Geocoder\Model\Address
Personne n'a encore pris cette issue.
- #1095 par @rtmotiondev — fermée sans fusion
- Langage dominant
- PHP
- Étoiles
- 4k
- Forks
- 525
- Merge moyen
- 8 min
- PR mergées (30 j)
- 1
Description
While testing IpInfoDb with IP 1.143.27.182 I got Uncaught TypeError: Argument 11 passed to Geocoder\Model\Address::__construct() must be of the type string or null, bool given
IpInfoDB response:
'statusCode' => string 'OK' (length=2)
'statusMessage' => string '' (length=0)
'ipAddress' => string '1.143.27.182' (length=12)
'countryCode' => string 'AU' (length=2)
'countryName' => string 'Australia' (length=9)
'regionName' => string 'Victoria' (length=8)
'cityName' => string 'Geelong' (length=7)
'zipCode' => string '3220' (length=4)
'latitude' => string '-38.1471' (length=8)
'longitude' => string '144.361' (length=7)
'timeZone' => string '+11:00' (length=6)
$timezone = timezone_name_from_abbr('', (int) substr($data['timeZone'], 0, strpos($data['timeZone'], ':')) * 3600, 0);
timezone_name_from_abbr('', 39600, 0) // Melbourne Australia is observing DST so this does not work
The proper fix is to handle DST time or at the very minimum handle the case of timezone returning false which breaks Geocoder\Model\Address.
Guide de contribution
Ouvrir 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.
Piste de recherche
Reproduisez la recherche IpInfoDb avec l’adresse IP 1.143.27.182 et examinez la conversion de fuseau horaire autour de timezone_name_from_abbr(). Confirmez le comportement lorsque DST fait que la recherche renvoie false, puis vérifiez que la construction résultante de Address ne reçoit plus de valeur booléenne.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 25/100