Project

General

Profile

Code Review #333

permissionの名前を他と被らないような名前にした方が良い

Added by Haru Iida over 9 years ago. Updated over 9 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Target version:
Start date:
04/15/2010
Due date:
% Done:

100%

Estimated time:
2.75 h
Spent time:
Code review: /init.rb@21:line 27

Description

permissionの名前は"index"だと他のプラグインと重なるかもしれないのでもう少し長くした方が良いと思います。

さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。

view_joel_testとanswer_joel_testみたいな感じでしょうか。


Related issues

Related to Joel Test - Defect #331: プロジェクトの設定内のモジュールタブにJoel testと表示されている。終了(Closed)04/15/2010

Actions
Has duplicate Joel Test - Feature #332: ジョエルテストに答えられるのはログインユーザに限定する。終了(Closed)04/15/2010

Actions

Associated revisions

Revision 23:18870c109e8e (diff)
Added by Takashi Takebayashi over 9 years ago

fixes #333
permission index -> view_joel_test へ変更

Revision 24:685e010af094 (diff)
Added by Takashi Takebayashi over 9 years ago

refs #333
permission answer_joel_test 追加
permission に合わせて、action を変更
不要な""部分を''に変更

Revision 25:4ead65aee1a0 (diff)
Added by Takashi Takebayashi over 9 years ago

refs #333
permission のラベル追加
permission チェックを追加

Revision 26:60cc2c90dac9 (diff)
Added by Takashi Takebayashi over 9 years ago

fixes #333
permission answer_joel_test が無い場合は、回答ボタンを disable に変更

History

#1

Updated by Toshiyuki Ando over 9 years ago

Haru Iida は書きました:

さらに言うと参照権と回答権を分けると良いかもしれません。ゲストは参照だけで回答させたくない場合があるかも。

view_joel_testとanswer_joel_testみたいな感じでしょうか。

活動に表示することなどを考えると、参照、回答権は分けておくのがよいです。
その場合 permission の名前は、 view_[module名] になります。
なので、飯田さんの view_joel_test に一票。

#2

Updated by Takashi Takebayashi over 9 years ago

  • Status changed from 新規(New) to 担当(Assigned)
  • Target version set to 0.0.4
  • Estimated time set to 2.00 h
#3

Updated by Takashi Takebayashi over 9 years ago

  • Status changed from 担当(Assigned) to 解決(Resolved)
  • % Done changed from 0 to 100

更新履歴 r23 で適用されました。

#4

Updated by Takashi Takebayashi over 9 years ago

  • % Done changed from 100 to 50

コミットキーワード不正の為、進捗変更

#5

Updated by Takashi Takebayashi over 9 years ago

  • Status changed from 解決(Resolved) to 担当(Assigned)
  • % Done changed from 50 to 80

プラグインチュートリアルを参考に対処致しましたが、以下のバグを組み込んでしまいました。
回避策を考えますが、しばらくかかる見込みです。

  • ログインせずに回答ボタンを押下すると、ログイン画面に遷移するが、そこでログインすると、ジョエルテストの回答が全てNo扱いになってしまう。
#6

Updated by Haru Iida over 9 years ago

permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。

#7

Updated by Takashi Takebayashi over 9 years ago

  • Status changed from 担当(Assigned) to 解決(Resolved)
  • % Done changed from 80 to 100

Haru Iida は書きました:

permissionが無い場合には回答ボタンを表示しないようにすればよいのではないでしょうか。

permissionが無い場合には回答ボタンを disable にする方向で対応致しました。

#8

Updated by Takashi Takebayashi over 9 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)
#9

Updated by Takashi Takebayashi over 9 years ago

  • Estimated time changed from 2.00 h to 2.75 h

Also available in: Atom PDF