細かい記載の修正を行う

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/controllers/TodosController.php#L17-L20

phpdocはパラメータ含めて書いた方が良いと思います。

    /**
     * タスク一覧表示
     * @param string $statusparam パラメータ説明文
     */

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657216649_

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/controllers/TodosController.php#L106-L107

1番目の$todoへの代入は消し忘れですよね？

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657216864_

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/controllers/TodosController.php#L186-L187

1番目の$todoへの代入は消し忘れですよね？

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657219195_

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/controllers/TodosController.php#L254

細かい所なのですが、下記２点言われる可能性が高いです。・文字列はシングルクォーテーションで括ってください。　（ダブルクォーテーションだと変数展開する関係上わずかに速度が落ちるからとか。。）・文字列結合のピリオド前後には半角スペースを入れてください。


$this->flash->success('id:' . $id . ' was deleted');

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657219553_

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/library/Elements.php#L19-L20

他のところは配列の終わり？にカンマつけていましたので、こちらもあわせましょうか。 (現場により、入れないでくれと言われる場合もありますが。。）

                'caption' => 'Home',
                'action' => 'index',

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657220076_

UedaKouta commented 4 years ago

https://github.com/UedaKouta/phalcon_work/blob/0c7225433f8c650f2920f60e6500eea75ef5ee47/todo/app/plugins/SecurityPlugin.php#L76-L84

細かい所なのですが、他のところは配列を「[]」で書いていますので、ここもあわせましょうか。

_Originally posted by @mokawa-rosso in https://github.com/UedaKouta/phalcon_work/pull/8#issuecomment-657220198_

UedaKouta / phalcon_work

細かい記載の修正を行う #14