* 内容が変更されていないときのファイル更新について [#jf28a519]

- ページ: [[BugTrack2]]
- 投稿者: 名無しさん
- 優先順位: 低
- 状態: 完了
- カテゴリー: その他
- 投稿日: 2007-04-04 (水) 19:00:36
- バージョン: 1.4.7
- リリース予定バージョン: 1.5.1

** メッセージ [#j245980c]
現状、編集画面で内容を一切変更していなくても、~
更新ボタンを押すと、各種更新処理がおこなわれてしまいます。~
これは無駄な処理であるとともに、変更がないにもかかわらず、最終更新が書き換えられるので、~
管理する側にとってもユーザー側にとってもあまり良くないように思います((これが悪意のある攻撃ならば、話は別ですが))。~
ソースを見るためにプラグインのsource を使わずに編集(edit)で代用している人がいる((私もコマンド入力が面倒なので、ソースが必要なときは、基本的に編集ボタンからソースを見ています。人のことを言える立場ではないです。))ので、キャンセルではなく間違って更新を押しているせいだとは思うのですが。~

~
そこで、このような改造はどうでしょうか?~
見出しから見出しまでの編集などの自作系への影響は、まだ確認してないので参考までにどうぞ。
 // $Id: file.php,v 1.78 2007/02/11 05:53:30 henoheno Exp $
 (変更点がないため、中略)
 // Put a data(wiki text) into a physical file(diff, backup, text)
 function page_write($page, $postdata, $notimestamp = FALSE)
 { 
   global $autoalias, $aliaspage;
 
   if (PKWK_READONLY) return; // Do nothing
 
   $postdata = make_str_rules($postdata);
 
   // Create and write diff
   $oldpostdata = is_page($page) ? join('', get_source($page)) : '';
 
   //分岐を追加
   if ($oldpostdata == $postdata) return; // Do nothing
 
   $diffdata    = do_diff($oldpostdata, $postdata);
   file_write(DIFF_DIR, $page, $diffdata);
 (変更点がないため、以降省略)
 

ファイルの更新処理をしてしまうのは、バグと言ってしまう程のものではないので、~
現状のままにするというのもありなのかもしれません((悪意ある攻撃に対するログを取るという意味で))。~
なので、上にも書きましたが参考意見のひとつとしてみてください。

** 実装 [#y53650b5]

- [[commit:a6ed3648d96b764b83c2161a9062c36c8dcb1f00]] 同一内容での更新をキャンセル扱いにする

--------
- 私は差分を消す目的で、変更無しの更新をかけるときがあります (^^; -- [[teanan]] &new{2007-04-04 (水) 19:24:42};
- 差分(diff)コマンドで表示した後、過去の差分を削除するためのリンクをクリックする。&br;認証画面でパスワードを入れる。&br;差分を消す&br;よりは楽ですからね~。 --  &new{2007-04-04 (水) 19:44:47};
- ↑に追加、ただどちらにしても、バックアップのサイズが少なからず増えてしまう気が。&br;それと、変更が何もないのにリンク更新の処理をする((たしか1つのページが更新されるとすべてのページに影響があったような))というのはサーバーへの負荷を増やしてしまうことになると。 --  &new{2007-04-04 (水) 19:57:39};
- 「空更新」というべき行為自体は、現状までにいくつかの役割が認識されていますが、どれもメリットとデメリットがありました。例えばその一つはRecentChangesの上位にそのページを掲載して、注意を促す事(未処理のBugTrackがそうしてピックアップされる事もありました)。Wikiはあらゆるアクションが基本的にはエネルギーに変わるので、ほどほどであればあまり問題ではありませんでした。しかしあまり脈絡のない空postが多くあると、rssやRecentChangesを購読している人が萎えます。もう一つはteananさんが言われている、diffのクリアです。いたずらを直したときにその痕跡を(ある程度)隠すことでWikiに落ち着いた感じを与える事ができますが、いたずらをする方がその痕跡を(ある程度)隠すことにも使えます。実際にURL貼り付けspamに悪用されています。 -- [[henoheno]] &new{2007-04-04 (水) 22:33:59};
-- ここで挙げられているのは、恐らく別の視点のデメリット、というか単純な無駄の指摘ですよね。ツッコミありがとうございます。 -- [[henoheno]] &new{2007-04-04 (水) 22:39:53};
-- いま最初に挙げたとおりで、このアクションにメリットが無いとも言い切れませんが、デメリットの方を強く感じるような環境や、場面があると思います。利用感覚に関する互換性も考慮すると、(今回挙げられたような単純な無駄は省けるようにした上で)どっちがいいかを管理者が選べるようにするのがスマートな道なのではないかと思います。いかがでしょうか。 -- [[henoheno]] &new{2007-04-04 (水) 22:42:47};
- 回答ありがとうございます。返答が遅れてしまい、すみません。&br;henoheno さんの言うとおり、各管理者が変更できる方がいいと思います。&br;あと、上の例ではほぼすべての処理をキャンセルしていますが、実際に実装する時はどうするか、というぐらいでしょうか。&br;同じようにするのか、それともdiffやbackup、wiki textの更新など、それぞれに分岐を作るのか。 --  &new{2007-04-08 (日) 13:39:57};
- 対応しました。「管理者が挙動を選択できるように」が柔軟性は高いですが、簡単のために、一旦「同内容での更新は更新無し扱いとし、何もしない(仕様変更)」とします。不都合が大きいようであれば再度仕様変更を検討します -- [[umorigu]] &new{2016-02-07 (日) 22:46:48};
- 管理者が挙動を選択するにはページごとでは無く全体で管理ならば変数をひとつ増やすだけではいいのでは -- [[->../../]] &new{2016-02-09 (火) 02:12:51};
- define() や globalで定義した$nodiff_refreshの真偽値を確認して処理を進めれば後の布石になりませんかね -- [[->../../]] &new{2016-02-09 (火) 02:14:58};
- 実装はご指摘の通りです。ただ設定値をpukiwiki.ini.phpに追加して切り替えるほどよく使いたい機能でもないように考えましたので、単純に仕様変更としました。ほとんどのケースで「同内容での更新はキャンセル扱い」で問題にならないと思っています -- [[umorigu]] &new{2016-02-09 (火) 23:03:03};

#comment

トップ   編集 差分 バックアップ 添付 複製 名前変更 リロード   新規 一覧 検索 最終更新   ヘルプ   最終更新のRSS
Site admin: PukiWiki Development Team

PukiWiki 1.5.3+ © 2001-2020 PukiWiki Development Team. Powered by PHP 5.6.40-0+deb8u12. HTML convert time: 0.032 sec.

OSDN