KnpLabs / KnpLabs/php-github-api

Do not "help" the user by changing parameters

オープン
#566 コメント 2 件 リアクション 4 件 担当者 0 名 GitHub で見る
enhancement
主要言語
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

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

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