プライベートルームの作成者が、本人でなければ本人に更新する処理のバグ

[akagane99]

不具合内容

• 同処理はプライベートルーム自体は更新してくれるけど、プライベートルームのページは更新してくれないバグ。

https://github.com/netcommons/NetCommons2/blob/develop/html/webapp/modules/module/update/Update.class.php#L358-L365

影響範囲

• 同処理が実装されたのは2.2.x.xの頃位と古く、現在NC2を新規で運用している方には、影響ないと思われる。
• NC1⇒NC2のデータ移行に関わる不具合

  前提

• NC1⇒NC2になった時、プライベートルームの作成者は、各個人としてＤＢに保存されるように変わった。
• そのため、モジュール管理の Update.class.php にプライベートルームのページデータ作成者が本人でなければ、本人に更新する処理が含まれている。
• 同処理は、2.4.2.1でも、モジュールの一括アップデート時に動作している。

[akagane99]

プルリク投げられないので、修正ベタ書き。

修正

https://github.com/netcommons/NetCommons2/blob/develop/html/webapp/modules/module/update/Update.class.php#L365

        //
        // private_spaceのinsert_user_idを自分自身に更新
        //
        $private_pages = $this->db->execute("SELECT {pages}.*, {pages_users_link}.user_id,{users}.handle " .
                                    " FROM {pages},{pages_users_link} " .
                                    " LEFT JOIN {users} ON {users}.user_id = {pages_users_link}.user_id ".
                                    " WHERE {pages}.default_entry_flag = "._OFF." AND {pages}.private_flag = "._ON.
                                    " AND {pages}.thread_num = 0 AND {pages}.room_id = {pages_users_link}.room_id AND ({pages}.insert_user_id != {pages_users_link}.user_id OR {pages}.insert_user_name != {users}.handle) ");

↓

                                    " AND {pages}.room_id = {pages_users_link}.room_id AND ({pages}.insert_user_id != {pages_users_link}.user_id OR {pages}.insert_user_name != {users}.handle) ");

この条件いらない {pages}.thread_num = 0 （プライベートルームのトップだけ対象にする条件）

[akagane99]

NC1⇒NC2のデータ移行に関わる不具合をもう１つ

症状

• システム管理者で、パブリックメニューのページを追加できるけど、削除、順番の入れ替えが出来なくなる
• memuの不具合
• NC2新規インストールであれば、下記修正しなくても正常に動く。

修正箇所

https://github.com/netcommons/NetCommons2/blob/e89bfb7c3e9253c07c34ac16999b44428473b51e/html/webapp/modules/menu/components/View.class.php#L274 https://github.com/netcommons/NetCommons2/blob/e89bfb7c3e9253c07c34ac16999b44428473b51e/html/webapp/modules/menu/components/View.class.php#L288

$parent_page =& $pre_buf_row[$row['parent_id']];

↓

$parent_page = $pre_buf_row[$row['parent_id']];

参照渡し、いらない。

[akagane99]

@kteraguchi のアドバイスにより解決できた。感謝!