[Hiki-dev:00817] Re: ちょこっと更新(タイムスタンプ更新無)

Back to archive index

Kazuhiko kazuh****@fdiar*****
2005年 6月 17日 (金) 15:52:06 JST


かずひこです。

At Thu, 16 Jun 2005 21:56:15 +0900 (JST),
hadai****@kd6***** wrote:

> 以前投稿した"ちょこっと更新"のパッチを添付します。

反応遅くてすみません。編集画面にインタフェイスが増えるのは多少抵抗がない
わけじゃないのですが、もとの掲示板に

| 2003-07-17 (木) 21:24:07 たけうち : ご意見ありがとうございます。サポー
| トする予定で検討中ですのでお待ち下さい。

とあるので採用しようと思います。:)

で、パッチについて一点確認させてください。

> -    def store( page, text, md5 )
> +    def store( page, text, md5, un_update_timestamp=nil )
>        backup( page )
>        filename = textdir( page )
>  
>        if exist?( page )
>          return nil if md5 != md5hex( page )
> -        FileUtils.copy( filename, backupdir( page ) )
> +        if ( un_update_timestamp == nil)
> +          FileUtils.copy( filename, backupdir( page ) )
> +        end
>        end
>        create_info_default( page ) unless info_exist?( page )

ここの意図がわかりませんでした。タイムスタンプは更新しなくても、バックアッ
プには移した方がいいんじゃないかと思うのですがどうでしょう?

> -      set_last_update( page, Time::now )
> +      if ( un_update_timestamp == nil )
> +        set_last_update( page, Time::now )
> +      end

あと、ファイルのタイムスタンプを見るケースもあると思うので、マージする際
にそのへんも古いタイムスタンプをセットするように変えるつもりです。

よろしくお願いします。
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命!」



Hiki-dev メーリングリストの案内
Back to archive index