複数の admin を指定できるようにする

[GoogleCodeExporter]

現状 admin として動けるのは property で指定した人のみ。
User クラスに admin 
属性を追加し、そのフラグが立っている人は admin 
扱いにするなどして複数人の admin 権限を可能にする。

Original issue reported on code.google.com by kawanaka.shinya@gmail.com on 1 May 2011 at 10:55

[GoogleCodeExporter]

カンマ区切りで複数指定できる、とかどうでしょう？属性��
�増やす必要性はあまり感じていません。
とりあえずパッチ書いてみました。

Original comment by skypencil@gmail.com on 14 Jul 2011 at 2:30

• Changed state: Started

Attachments:

• issue165.patch

[GoogleCodeExporter]

次の２点を直してもらえますか？

1. getTwitterAdminName() の null check 
を入れる。現状も入ってませんが、あったほうが better 
2. split したあと trim する。もしなっていれば OK

Original comment by kawanaka.shinya@gmail.com on 15 Jul 2011 at 2:57

[GoogleCodeExporter]

いいですね。
ついでに大文字小文字の同一視も入れようと思いますが問��
�ないですか？
trimした後でtoLowerCasesするイメージです。

Original comment by skypencil@gmail.com on 15 Jul 2011 at 6:58

[GoogleCodeExporter]

書きました。ご確認のほどよろしくお願いします。

Original comment by skypencil@gmail.com on 15 Jul 2011 at 2:21

Attachments:

• issue165_2nd.patch

[GoogleCodeExporter]

毎回splitするのはもったいないなーと思い、あらかじめ分割
・加工したものをHashSetに入れて保持するようにしています�
��
Collections#unmodifiableSetでラップした方がいいかも。

Original comment by skypencil@gmail.com on 15 Jul 2011 at 2:23

[GoogleCodeExporter]

unmodifiableSet でラップしたほうがよいと思います。
あと、getTwitterAdminName を getTwitterAdminNames 
に名前を変更してください。それで commit 
してもらってかまいません。

Original comment by kawanaka.shinya@gmail.com on 15 Jul 2011 at 5:38

[GoogleCodeExporter]

This issue was closed by revision r569.

Original comment by skypencil@gmail.com on 16 Jul 2011 at 9:50

• Changed state: Fixed

[GoogleCodeExporter]

ありがとうございました。commitしました。

Original comment by skypencil@gmail.com on 16 Jul 2011 at 9:54

[GoogleCodeExporter]

Migrated to http://github.com/partakein/partake/issues/165

Original comment by skypencil@gmail.com on 27 Jun 2012 at 9:39

• Changed state: Migrated