ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#3205 closed defect (fixed)

公開範囲を「公開しない」としてだれかがTodoにチェックを入れるとTodoが消える

Reported by: imamura623 Owned by: kiwa
Priority: minor Milestone: OpenPNE2.12.9
Component: 指定しない Version: 2.12.x & 2.14.x
Keywords: OpenPNE2.13.7 Cc:

Description (last modified by kudo)

■現象

公開範囲で「公開しない」を選択し、「だれかがTodo」にチェックを入れ編集をするとTodoが消える。

■発生バージョン

  • 2.10.x
  • 2.12.x
  • trunk

■再現手順

  1. Todo一覧から適当なTodoを追加(チェックは入れない)
  2. Todo一覧から手順1で追加したTodoを編集→編集画面に遷移
  3. 公開範囲で「公開しない」を選択し、「だれかがTodo」にチェックを入れ編集
  4. Todo一覧を見ると編集したTodoが表示されていない

■環境

  • FireFox3

■原因

■修正内容

公開しない共有Todoをh系以外にも表示させるようにした。

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=4214より転記

お世話になります。
OpenPNEのバージョン2.12.4にて、
Todo編集画面で
・公開範囲を「公開しない」
・だれかがTodoにチェックを入れる
として編集を行うと、Todo一覧から入力したTodoが消えてしまいます。
データベースの「biz_todo」を確認すると、編集したレコードは残っている為、
データベースからは削除されている訳ではないようです。
そもそも公開範囲を「公開しない」としているのに、「だれかがTodo」を選ぶこと自体がおかしい気がするのですが、上記の様な入力が出来てしまうのは仕方がないのでしょうか?
また、この現象は2.12.4版では、仕様ということになるのでしょうか?
よろしくお願いします。

Change History (16)

comment:1 Changed 11 years ago by nakasone

Keywords: 再現待ち removed

以下のような確認を行ったところ再現できました。

■確認ブラウザ

  • !FireFox3

■確認バージョン

  • 2.10.x
  • 2.12.x
  • trunk

■手順

  1. Todo一覧から適当なTodoを追加(チェックは入れない)
  2. Todo一覧から手順1で追加したTodoを編集→編集画面に遷移
  3. 公開範囲で「公開しない」を選択し、「だれかがTodo」にチェックを入れ編集
  4. Todo一覧を見ると編集したTodoが表示されていない

comment:2 Changed 11 years ago by kiwa

Version: 2.12.x & 2.13.x

comment:3 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.7 added
Milestone: OpenPNE2.12.9

comment:4 Changed 11 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned
公開範囲
公開しない
チェック
だれかがTodo

の場合、Todo作成者にのみ表示させるようにします。

comment:5 Changed 11 years ago by kiwa

Owner: changed from kiwa to nobody
Status: assignednew
  • Todo編集(fh_home_edit_biz_todo)で公開範囲が「公開しない」の場合に「だれかがTodo」が選択されていたら編集を中断してエラーを返す
  • 既存の「公開しない」「誰かがTodo」のTodoへの救済策としてcomment:4 の措置を行う

2つの必要がありそうです。すいません、ひとまず手放します。

comment:6 Changed 11 years ago by kiwa

Todo編集(fh_home_edit_biz_todo)で公開範囲が「公開しない」の場合に「だれかがTodo」が選択されていたら編集を中断してエラーを返す

ひとまずこちらの修正のみ行いました。

既存の「公開しない」「誰かがTodo」のTodoへの救済策としてcomment:4 の措置を行う

こちらの変更は、Todo公開範囲のチェックを行う関数「biz_isPermissionTodo」にて以下の条件を満たす場合にtrueを返す処理を追加するようになると思います。

  1. $biz_todo['c_member_id']が0
  2. $biz_todo['writer_id']が自分
  3. ページ自体のtarget_c_member_idと$uが一致する

現在のbiz_isPermissionTodoでは3の判定を行うことができないため、新たにページ自体のtarget_c_member_idを入れる引数が必要です。

comment:7 Changed 11 years ago by kiwa

Owner: changed from nobody to kiwa
Status: newassigned

やります。

comment:8 Changed 11 years ago by kiwa

Keywords: 確認中 added

以下のリビジョンで非公開の共有Todoをh系にのみ表示するようにしました

comment:9 Changed 11 years ago by ShogoKawahara

Keywords: テスト待ち added; 確認中 removed

チェックしました。

comment:10 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

テスト完了です。問題ありません。

comment:11 Changed 11 years ago by ogawa

Resolution: fixed
Status: closedreopened

権限チェックに一部問題があったので差し戻します。

公開しない誰かがToDoは h系以外でも表示されるように修正した方が影響範囲も小さく、またこの仕様でもそれほど違和感はないと思います(そもそも非公開で共有というのがおかしいので)

comment:12 Changed 11 years ago by ogawa

Keywords: 差し戻し added

comment:13 Changed 11 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで公開しない共有Todoをh系以外にも表示させるようにしました

comment:14 Changed 11 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

OKです。

comment:15 Changed 11 years ago by imamura623

Keywords: テスト待ち removed
Resolution: fixed
Status: reopenedclosed

確認しました。

comment:16 Changed 11 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.