Japanese Moodle: ユーザ画像の表示範囲（全般的日本語フォーラム）からの続き

ユーザ画像の表示範囲（全般的日本語フォーラム）からの続き

2010年 06月 1日(火曜日) 17:28 - Tatsuya Shirai の投稿

本件は全般的日本語フォーラムに投稿された「ユーザ画像の表示範囲」から派生したディスカッションです．
http://moodle.org/mod/forum/discuss.php?d=139243
2010/06/03にディスカッションを分割

すみません，この件を完全に忘れていました＾＾；；；

　いま大雑把に対策の骨子ができましたので，方針のみを報告します．修正はuser/pix.phpのみで，修正は最小限に抑えようと思います．

    if (count($args) == 2) {
        $userid   = (integer)$args[0];
        // do not serve images of deleted users
        if ($user = get_record('user', 'id', $userid, 'deleted', 0, 'picture', 1)) {
// (Shirai190): ユーザ画像の表示範囲を制限可能とする機能の追加 (2010/06/01)
// (Shirai190): ここから追加
            if (fs_function_enable('FS_ENABLE_PRINTUSERPICTURELEVEL')) {
                if (!fs_can_display_userpicture($user)) redirect($CFG->pixpath.'/u/f1.png');
            }
// (Shirai190): ここまで追加
            $image    = $args[1];
            $pathname = make_user_directory($userid, true) . "/$image";
            if (file_exists($pathname) and !is_dir($pathname)) {
                send_file($pathname, $image);
            }
        }
    }

　これはfs_moodleにおける修正方法です．fs_function_enable()という関数は，引数で指定された定数が定義されているか，定義されていた場合にそれが機能を有効とする値に設定されているか（このENABLE系のスイッチの場合はtrueが設定されている場合）かを調べています．つまりできるだけオリジナルのままに使いたい場合には影響を最小限に抑えるための仕組みです．

　そのif文のブロックの中のfs_can_display_userpicture()関数（現在作成中）で，ユーザの画像を表示するか，それともデフォルトの黄色い顔アイコンを表示するかを分けています．表示しないのではなく，黄色い顔アイコンを表示してしまえばプライバシーは守られますウインク．このfs_can_display_userpicture()関数がtrueを返す場合は学生のアイコンを表示し，falseの場合はデフォルトの顔アイコンを表示するようにURLをリダイレクトします．

　ユーザのアイコン画像が発見されてしまった（というのも何ですが）場合のみ，この画像を表示してしまって良いかどうかを考えることにしています．

＃仕様はさらに拡張されて，こちらのWikiにまとめたようになっています．

Re: ユーザ画像の表示範囲

2010年 06月 2日(水曜日) 17:29 - Tatsuya Shirai の投稿

　少々，面倒でした．冷静に考えてみたら，「このユーザは学生か否か？」をどう判断すれば良いのかを整理していませんでした．Moodle1.8からロールの定義をいくらでも増やすことが可能になりました．Moodle1.7まではadmin，coursecreator，editingteacher，teacher，student，guestの６種のロールがあり，Moodle1.8からはさらにuserが追加されたと記憶しています．なお，userは認証済みユーザ（authenticated user）です．そしてこれらのロールはレガシーロールと呼ばれていますね．

　新しいロールにしろ，Moodle1.8/Moodle1.9で用意されているロールにしろ，ベースとなるロールがどのレガシーロールなのかを指定します．ですので，このレガシーロールによってユーザが大雑把に分けてどの立場なのかはある程度分かります．
　でも，もう一つ状況を混乱させる概念があります．ユーザのロールは単一ではありません．状況によってコロコロと変えることが可能です．システムロール，カテゴリーロール，コースロール，そしてモジュールやブロックなどのロールの４つ．それぞれ後に指定されているものほど優先度が上がります．状況によってユーザは教師であったり学生であったりします．ここはではシステムロールで判断することとしましょう．
　でも，怠け者の私は手抜きをして，ユーザのシステムロールを設定していませんでした^^; その場合，ユーザのシステムロールはサイト管理ブロックの［ユーザ］-［パーミッション］-［ユーザポリシー］のすべてのユーザのデフォルトロール（デフォルト：認証済みユーザ）が適用されます．ですので大半のユーザは教師も学生もシステムロールは認証済みユーザ！　以下の改良を適用するには，まずシステムロールをきちんと設定する必要があります．

　ユーザ画像を表示するかどうかをレガシーロールに基づいて判別する関数群を貼付します．でも，あくまで参考程度に考えて下さい．何箇所かfs_moodleに依存した部分があります．

貼付したライブラリにミスがありましたので，2010/06/02 18:57に新しいファイルと差し替えました．

fs_lib_userpicture.php

Re: ユーザ画像の表示範囲

2010年 06月 2日(水曜日) 17:38 - Tatsuya Shirai の投稿

　先ほど貼付した関数群の中にprint_legacy_role_table()という関数が含まれています．この関数は指定されたユーザIDのユーザのシステムロールのレガシーロールを調べて表形式で表示する関数です．使い方は，user/view.phpに，以下のように組み込むことで管理者がユーザのユーザプロファイルを表示した際に画面下部に貼付画像のような表を出力するものです．これで「あれ？なんでこのユーザの画像が表示される／されないんだ？」という時の確認に使えます．

/// Printing Interests
if( !empty($CFG->usetags)) {
     if ( $interests = tag_get_tags_csv('user', $user->id) ) {
            print_row(get_string('interests') .": ", $interests);
        }
    }
/// End of Printing Interests

    echo "</table>";

    echo "</td></tr></table>";

    $userauth = get_auth_plugin($user->auth);

    $passwordchangeurl = false;
    if ($currentuser and $userauth->can_change_password() and !isguestuser() and has_capability('moodle/user:changeownpassword', $systemcontext)) {
        if (!$passwordchangeurl = $userauth->change_password_url()) {
            if (empty($CFG->loginhttps)) {
                $passwordchangeurl = "$CFG->wwwroot/login/change_password.php";
            } else {
                $passwordchangeurl = str_replace('http:', 'https:', $CFG->wwwroot.'/login/change_password.php');
            }
        }
    }

// Print other functions
// (Shirai190): ユーザ画像の表示範囲を制限可能とする機能の追加 (2010/06/01)
// (Shirai190): ここから追加
    if (!empty($USER->id) and is_siteadmin($USER->id)) {
        echo '<p align="center">';
        print_legacy_role_table($user->id);
        echo '<br />';
        echo '</p>';
    }
// (Shirai190): ここまで追加
    echo '<div class="buttons">';

    if ($passwordchangeurl) {
        $params = array('id'=>$course->id);

これらの機能は次に公開予定のfs_moodle4.06.00に組み込む予定です．

Re: ユーザ画像の表示範囲

2010年 06月 2日(水曜日) 18:03 - Tatsuya Shirai の投稿

　少々，不安なのはレガシーロールの設定を読み出す関数，

function isteacher($courseid=0, $userid=0, $obsolete_includeadmin=true)
function isteacherinanycourse($userid=0, $includeadmin=true)
function isteacheredit($courseid, $userid=0, $obsolete_ignorestudentview=false)
function iscreator ($userid=0)
function isstudent($courseid=0, $userid=0)
function isstudent($courseid=0, $userid=0)

辺りが次のメジャーバージョンアップで削除される候補としてlib/deprecatedlib.phpに移されていることです．今回のライブラリでは，function isstudent($courseid=0, $userid=0)をベースに開発しています．このアクセス方法が廃止されてしまったらまた別の方法を考えないといけないかも知れませんね．

＃消える候補にあるから自分で関数を作ったのですが．．．

Re: ユーザ画像の表示範囲

2010年 06月 3日(木曜日) 10:01 - Takayuki ISHIKAWA の投稿

本筋とは無関係ですが…話題が code 改造を含むようになってきましたら開発者 forum へ移行するのはいかがでしょうか。forum 毎に mail 購読するかどうか選べますので、そうしていただけると助かります。

Re: ユーザ画像の表示範囲

2010年 06月 3日(木曜日) 11:16 - hideki ichihashi の投稿

Ishikawa様

まだフォーラムの空気が読めておらず、申し訳ありません。。
今後はそのようにさせていただきます。
取り急ぎご返信までにて失礼します。

Re: ユーザ画像の表示範囲

2010年 06月 3日(木曜日) 22:22 - Takayuki ISHIKAWA の投稿

いえ、forum の整理は最近行われたものでして、実際のところ私も全部を把握してはいません (mail 購読している forums だけ把握しています)。どなたが悪い、というような話ではないので、お気になさらないで下さい。

白井先生、移行ありがとうございました。code 改造に手を付けることで更なる前進が望める、という点はどの forums にもありうることなので、具体的な改造点が出てきたら開発者 forum へ移行する (というか fork する)、ということでいかがでしょうか。

Re: ユーザ画像の表示範囲

2010年 06月 4日(金曜日) 09:06 - Tatsuya Shirai の投稿

　そうですね．皆さんも意図的に「ここから先の話は技術的な話になりますので＊＊＊へ移行します」という感じで，この投稿を例に話を分岐してください．

　もし既に始まってしまった話が技術的傾向に徐々に偏ってきて，いまから話を分けると混乱する！という場合はご相談下さい．facilitator権限で分割や移動ができます（ただし合体はできないので，一度，切ったディスカッションは戻せない）．

Re: ユーザ画像の表示範囲

2010年 06月 3日(木曜日) 14:34 - Tatsuya Shirai の投稿

　ここが移行期の微妙なところでして^^;

　最初の投稿が昨年の１２月中ですのでフォーラムを丸ごと移動する訳にもいかず，分割するべきかとも一瞬考えたのですが（リンクを張れば良いので無茶ではないですね），一連の発言のどこにどういう返信が付くか予想ができません．．．

　そうですね，丁度，私が思い出して投稿をした辺りから雰囲気が変わっていますので，そこから先を分割させましょう．

　どうですか？　フォーラム分割は失敗したらどうしよう...と思うと躊躇してしまうのですが，なかなかうまく行ったようです．

Moodle開発者フォーラム

ユーザ画像の表示範囲（全般的日本語フォーラム）からの続き