tDiary

ただただしです。

06/07/22 に KURODA Hiraku<hiraku@...> さんは書きました:
> *たださんの日記本文に制御記号が混入している
> *それをそのまま取り込んで、makerss.rbが出力するindex.rdfにも制御記号が混
> 入している

おぉ。ブラウザ上の文字列をコピペしただけなのに、そういうモノが
入っちゃうことがあるんですね。失礼しました。あとで直しておきます。

> そこで、makerss.rbをいじって、index.rdfを出力する際に制御記号(\x00から
> \x1fまで)のうち改行(\rと\n)以外を削除するようにしてみました。

あとは"\t"も特別視したほうがいいように思えます。
とりあえずtrunkにだけ入れました。ありがとうございました。

＃2.0はそろそろ2.2を出したいからいいかな、と。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

黒田です。

TADA Tadashi wrote:
> ただただしです。
(中略)
> 
> あとは"\t"も特別視したほうがいいように思えます。
> とりあえずtrunkにだけ入れました。ありがとうございました。
> 
> ＃2.0はそろそろ2.2を出したいからいいかな、と。

うわ、タブのことを忘れていました。
2.0用は
 http://www.hinet.mydns.jp/~hiraku/hiki/?Patches
で公開します。

2.2、楽しみにしています。

-- 
--------------------------------------------------
 黒田 拓(くろだ ひらく)
    hiraku@...
    http://www.hinet.mydns.jp/~hiraku/
--------------------------------------------------

ToshiOkadaです。矢継ぎ早ですみません。

　先の投稿を書いた後に考え直して，仕様をちょっと変えてみました。

　@options=true環境下では，ブラウザ上からは殆ど設定出来ないように変更し
ました。
　セキュアな環境下で日記ユーザーによる全ての設定が出来ても，その必要性を
あまり見いだせそうにない(逆に管理面でデメリットが増しそうな)ので，下記オ
プションを本体tdiary.confへ記述して falseを指定しない限り，画像サイズ情
報の取得動作の有効／無効以外の設定を表示しないように変更しました。

　　@options['image.hideconf'] = false

を指定することで，セキュアな環境下でも全オプションの設定が可能なようにし
てあります(無指定時はtrueがデフォルト)。

　で，image.rb rev1.31へのパッチは

http://okat.no-ip.com/~toshi/hiki/?%B3%A8%C6%FC%B5%AD%A5%D7%A5%E9%A5%B0%A5%A4%A5%F3
(http://okat.no-ip.com/~toshi/tdiary/image_plugin-1.4.2.patch)

に置いておきます。
　相変わらずパッチ当てすると勝手に rev 1.4.2を名乗ります。すみません(汗

# パッチを添付するとMLやISPのメールサーバーなどに "アブナイメール" と判
# 定されて遅配になるっぽいので(笑)，私個人のサーバーにパッチを置くことに
# しました。

-- 
ToshiOkada  <tokada@...>

ただただしです。

反応遅くてすみません。

06/07/12 に NISHIMURA Takashi<ni46rtk4@...> さんは書きました:
> 設定画面「spam フィルタ」で「ツッコミ中で URL を表す文字の
> 占める割合が [ ] % より高いものは spam とみなす」と以前とは
> 表現が変更されましたが、tdiary/filter/spam.rb の該当部分を
> 見ると（358 行目）
>
>  uris.join('').size.to_f/comment_body.gsub(/\s+/, '').size.to_f > @max_rate
>
> となっているので % で指定してはダメなのではありませんか。

いや、勘違いというか、使ってなかったので仕様を知らなかったのは
私の方です。すみませんorz

で、現在の仕様だと、使うほうも難しいし、説明もしづらいので、%に
変えちまった方がいいように思うんですが、どうでしょうね。仕様変更
になっちゃうけど。

ご意見募集。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

NT です。

> で、現在の仕様だと、使うほうも難しいし、説明もしづらいので、%に
> 変えちまった方がいいように思うんですが、どうでしょうね。仕様変更
> になっちゃうけど。
>
> ご意見募集。

確かにたださんのおっしゃる通りだと思うので、仕様変更に賛成です。
私自身、以前の文面ですら最初に読んだときに % で指定するものだと
思い込んでうまく動作してくれないのはなぜか悩みましたから。

--
NISHIMURA Takashi <nt@...>

ただただしです。

06/07/18 に NISHIMURA Takashi<ni46rtk4@...> さんは書きました:
> 確かにたださんのおっしゃる通りだと思うので、仕様変更に賛成です。
> 私自身、以前の文面ですら最初に読んだときに % で指定するものだと
> 思い込んでうまく動作してくれないのはなぜか悩みましたから。

反対意見もないようなので、max_rateの意味するところをパーセントに
変更しました。従来「0.5」のように指定してある場合には、「50」を指定
しなおす必要があります。

同時に、tDiary-users MLで指摘のあった、「a:b」のような文字列がURI
と認識されてしまうという問題に対処するため、URIのschemeとして以
下の4種類を指定しました:

    http, https, ftp, mailto

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

ただただしです。

06/07/04 に Takashi SATO<exocet@...> さんは書きました:
> tDiary開発版2.1.2以後で、Windowsで動作させると、
> 「Permission denied」が発生します。

おお、これはありがたい。似たようなレポートを受けていたんだけど、
よくわからなかったのでした。

> Permission denied - （データパス）/2006/200607.td2 (Errno::EACCES)
>
> （hoge）/tdiary214/tdiary/defaultio.rb:169:in `delete'

たぶん、open中のファイルを削除しようとしているからですね。この
パッチでどうでしょうか。

Index: defaultio.rb
===================================================================
RCS file: /cvsroot/tdiary/core/tdiary/defaultio.rb,v
retrieving revision 1.40
diff -u -r1.40 defaultio.rb
--- defaultio.rb        10 Feb 2006 18:09:05 -0000      1.40
+++ defaultio.rb        4 Jul 2006 12:37:57 -0000
@@ -166,6 +166,10 @@

                                if diaries.empty?
                                        begin
+                                               unless fh then
+                                                       fh.close
+                                                       fh = nil
+                                               end
                                                File::delete( @dfile )
                                        rescue Errno::ENOENT
                                        end

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

えぐぞせ です。

On <f7dffd10607040540w6bf8b18ej2a561f3197efc936@...>
"TADA Tadashi" <sho@...> wrote:

> たぶん、open中のファイルを削除しようとしているからですね。この
> パッチでどうでしょうか。

エラーの行番号が後ろにずれただけで、全く同じエラーとなりました…。

# ちなみに、パッチあては手動で行いました
# この場合は行頭に"+"がある行を追加するだけですよね？

ところで、示していただいたパッチをあてたうえで、私なりにいじって
みました。

（パッチをあてた上での）167行目からの部分ですが、

----ここから----
				if diaries.empty?
					begin
						unless fh then
							fh.close
							fh = nil
						end
						File::delete( @dfile )
					rescue Errno::ENOENT
					end
----ここまで----

これを、

----ここから----
				if diaries.empty?
					begin
						unless fh then
							fh.close
							fh = nil
						end
				#		File::delete( @dfile )
					rescue Errno::ENOENT
					end
----ここまで----

このように、『File::delete( @dfile )』をコメントアウトして、
186行目付近からの

----ここから----
			ensure
				fh.close if fh
			end
----ここまで----

この部分を、

----ここから----
			ensure
				fh.close if fh
				File::delete( @dfile )
			end
----ここまで----

このように、『File::delete( @dfile )』を追加（というよりは移動）
しました。
この結果、見た目には、正常動作するようになりました。

バカ正直に「DeleteするのはCloseした後じゃないのかなぁ」と思って
いじってみたのですが、

これって正しいでしょうか？？？



-- 
Takashi SATO <exocet@...>

えぐぞせ です。

On <20060705023511.3721.EXOCET@...>
Takashi SATO <exocet@...> wrote:

> バカ正直に「DeleteするのはCloseした後じゃないのかなぁ」と思って
> いじってみたのですが、
> 
> これって正しいでしょうか？？？

ブラウザのキャッシュクリアして再表示させたらバリッとおかしくなり
ました…。

# バカの考え休むに似たり orz



-- 
Takashi SATO <exocet@...>

ただただしです。

06/07/05 に Takashi SATO<exocet@...> さんは書きました:
> エラーの行番号が後ろにずれただけで、全く同じエラーとなりました…。

あ、すみません。パッチが間違ってます。unlessじゃなくてifでした。

> # ちなみに、パッチあては手動で行いました
> # この場合は行頭に"+"がある行を追加するだけですよね？

そうですね。追加した行の最初の「unless」を「if」に変えてみてもらえ
ますか?

> このように、『File::delete( @dfile )』を追加（というよりは移動）
> しました。
> この結果、見た目には、正常動作するようになりました。
>
> バカ正直に「DeleteするのはCloseした後じゃないのかなぁ」と思って
> いじってみたのですが、

これだとあらゆる場合にデータファイルが削除されてしまいます:-)

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

えぐぞせ です。

On <f7dffd10607041607y29426b7fw258e5e59f5726bef@...>
"TADA Tadashi" <sho@...> wrote:

> > # ちなみに、パッチあては手動で行いました
> > # この場合は行頭に"+"がある行を追加するだけですよね？
> 
> そうですね。追加した行の最初の「unless」を「if」に変えてみてもらえ
> ますか?

ばっちぐー（死語）です！
御指導ありがとうございます。

> これだとあらゆる場合にデータファイルが削除されてしまいます:-)

ホンマにキレイさっぱり！になってました…。



-- 
Takashi SATO <exocet@...>

ただただしです。

06/07/05 に Takashi SATO<exocet@...> さんは書きました:
> > そうですね。追加した行の最初の「unless」を「if」に変えてみてもらえ
> > ますか?
>
> ばっちぐー（死語）です！

よかったです。

この修正をTrunkにcommitしました。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

ただただしです。

Windows上での実行時、旧referer形式から新referer形式への自動
コンバートで下記と同様の問題が出ていたので、そちらも修正しました。

06/07/06 に TADA Tadashi<sho@...> さんは書きました:
> ただただしです。
>
> 06/07/05 に Takashi SATO<exocet@...> さんは書きました:
> > > そうですね。追加した行の最初の「unless」を「if」に変えてみてもらえ
> > > ますか?
> >
> > ばっちぐー（死語）です！
>
> よかったです。
>
> この修正をTrunkにcommitしました。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。