geocoder-php / geocoder-php/Geocoder

Wrong LocationIQ Bounds

オープン 初心者向け
#1,272 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
PHP
スター
4k
フォーク
525
平均マージ
8分
マージ済み PR(30日)
1

説明

In the LocationIQ provider, bounds are passed to the AddressBuilder in the wrong order.

$builder->setBounds((float) $bounds['south'], (float) $bounds['north'], (float) $bounds['west'], (float) $bounds['east']);

should really be

$builder->setBounds((float) $bounds['south'], (float) $bounds['west'], (float) $bounds['north'], (float) $bounds['east']);

according to the function signature of AddressBuilder.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

bounds を AddressBuilder に渡している LocationIQ プロバイダーのコードを見つけ、呼び出しと AddressBuilder の関数シグネチャを比較してください。Issue に示されている引数の順序を修正し、その後、プロバイダーの bounds 処理で south、west、north、east が期待される位置で使用されていることを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php
領域
backend
issue の種類
バグ
難易度
1/5
見積もり時間
1時間未満
活発さ
活発
明瞭さ
明確に書かれている
初心者へのやさしさ
78/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。