Code Review #333
closed
permissionの名前を他と被らないような名前にした方が良い
Added by Haru Iida almost 15 years ago.
Updated over 14 years ago.
Description
permissionの名前は"index"だと他のプラグインと重なるかもしれないのでもう少し長くした方が良いと思います。
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
Haru Iida は書きました:
さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。
view_joel_testとanswer_joel_testみたいな感じでしょうか。
活動に表示することなどを考えると、参照、回答権は分けておくのがよいです。
その場合 permission の名前は、 view_[module名] になります。
なので、飯田さんの view_joel_test に一票。
- Status changed from 新規（New） to 担当（Assigned）
- Target version set to 0.0.4
- Estimated time set to 2:00 h
- Status changed from 担当（Assigned） to 解決（Resolved）
- % Done changed from 0 to 100
- % Done changed from 100 to 50
- Status changed from 解決（Resolved） to 担当（Assigned）
- % Done changed from 50 to 80
プラグインチュートリアルを参考に対処致しましたが、以下のバグを組み込んでしまいました。
回避策を考えますが、しばらくかかる見込みです。
- ログインせずに回答ボタンを押下すると、ログイン画面に遷移するが、そこでログインすると、ジョエルテストの回答が全てNo扱いになってしまう。
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
- Status changed from 担当（Assigned） to 解決（Resolved）
- % Done changed from 80 to 100
Haru Iida は書きました:
permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。
permissionが無い場合には回答ボタンを disable にする方向で対応致しました。
- Status changed from 解決（Resolved） to 終了（Closed）
- Estimated time changed from 2:00 h to 2:45 h
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by