KnpLabs / KnpLabs/php-github-api
Do not "help" the user by changing parameters
- 主要言語
- PHP
- スター
- 2.2k
- フォーク
- 587
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
On 30+ places we have code like this:
```php
if (!in_array($state, array('open', 'closed'))) {
$state = 'open';
}
```
If the user misspelled the `$state` we should not help them. That may cause unpredicted actions. We should throw an exception or let the API call fail.
This is a BC break. I've marked it in milestone 3.0.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
Search the PHP source for the repeated in_array parameter-normalization pattern and review how invalid API parameters currently fail. Determine the consistent failure behavior and identify all affected locations; done means misspelled parameters are no longer silently changed and the existing test suite passes.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- api, backend
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100