最終更新者の表示

大塚です。

ページの下の最終更新日を表示している部分に、更新したユーザ名を
表示したいと思い、パッチを作ってみました。いかがでしょうか?

 --- db-backup.rb.org    2006-07-19 13:56:17.000000000 +0900
+++ db-backup.rb        2007-09-22 17:00:28.000000000 +0900
@@ -31,6 +31,21 @@
      return path(k, time).read
    end 

+    def get_user(k, time)
+      begin
+        f = File.open(@backup_path+"#{time}__SiteLog")
+        a = f.readlines
+        l = a[a.length-1].strip.split(',')
+      rescue
+        return 'Unknown'
+      end
+      if (l[2].length > 0)
+        return l[2]
+      else
+        return 'Anonymous'
+      end
+    end
+
    def check(k, v)
    end
}}} 

{{{
 --- act-basic.rb.org    2006-07-19 13:56:16.000000000 +0900
+++ act-basic.rb        2007-09-22 17:02:22.000000000 +0900
@@ -169,8 +169,9 @@
      page = @site[@req.base]
      return if page.nil?
      date = page.mtime
+       user = @site.backupdb.get_user('test',date.to_i.to_s)
      return [:span, {:class=>'attribute'}, _('Last modified'), ': ',
 -       [:em, date.ymd]]
+       [:em, date.ymd], ' by ', [:em, MailAddress.obfuscate(user)]]
    end 

    def plg_generate_time

--

大塚 洋 (OHTSUKA, Hiroshi)
hao@t...

ページの下の最終更新日を表示している部分に、更新したユーザ名を 表示したいと思い、パッチを作ってみました。いかがでしょうか?

最終更新ユーザ名を表示していなかったのは意図的だったのですが，たしかに
機能としてはあってもいいですね．現在の実装だとちょっと重すぎます．
もう少し軽い実装になってテストケースもつけばcommitしてもいいです．

最終更新ユーザ名を表示していなかったのは意図的だったのですが，たしかに

意図的とは、更新は匿名性があった方が良いということですか?

機能としてはあってもいいですね．現在の実装だとちょっと重すぎます． もう少し軽い実装になってテストケースもつけばcommitしてもいいです．

うーん、確かに相当回りくどいことしてます。
日付と同じようにpageに持たせるようにしないとだめでしょうか。

最終更新ユーザ名を表示していなかったのは意図的だったのですが，たしかに

意図的とは、更新は匿名性があった方が良いということですか?

はい，そうです．

機能としてはあってもいいですね．現在の実装だとちょっと重すぎます． もう少し軽い実装になってテストケースもつけばcommitしてもいいです．

うーん、確かに相当回りくどいことしてます。 日付と同じようにpageに持たせるようにしないとだめでしょうか。

今の実装だと毎回SiteLogを全部読んでいるので，メモリが破綻します．本来
は記録する際にpageに情報を持たせるのがいいですね．現在のqwikWebはペー
ジをfilesystemに持たせているので，そのような柔軟な情報を保持することが
できないので，ちょっと実装方法には悩みます．ページ毎にmeta情報を持たせ
るファイルを別に用意すればいいのですが，ファイルが増えるのがいやなので
躊躇しています．