Re: transMsg() (P2-php-dev 34) - p2

> お疲れさまです。
> 
> こちらでも僕が書いてたコードと1.5.23の差分を参考に
> フィードバックさせてもらいました。
> （[P2-php-dev 32]を読む前に作業してました）
> 
> showthreadkは次で直します。ありがとうございました。
> どうせなら「略」のときも
> $msg = preg_replace_callback($this->str_to_link_regex, array($this, 
> 'link_callback'), $msg);
> しようと思ってるんですが、どうでしょう？
> ・・・と書いたところで、URLは途中で切れる可能性が高いよな、と思い直す。
> ソースにコメント書いとこう。(笑）

そうそう、URLが途中で切れると困るので、
簡易的な表示にしてました。

でも、<br>と同じく、urlの途中で切れている場合も、
あらかじめカットしておけばOKかなと思ったものの、
やっぱり、 >>123 も ID:XXXX も途中で切れる可能性ある...。

うーん、略があるときは、簡易プレビュー的な扱いなので、
今のままでもいいかな。

新しい方のコードを見て…
$msg = preg_replace('/ *<[^>]*$/i', '', $msg);
$msg = preg_replace('/( *<br> *)+$/i', '', $msg);

'/ *<[^>]*$/i' の方が汎用に対応できるのでいいなー。
下の '/( *<br> *)+$/i' は、別になくても良さげかもですね。

--
aki

> 
> // +------------------------------+
> // | rsk <moons****@s32*****> |
> // +------------------------------+
> 
> On 2005/05/08, at 6:05, aki wrote:
> 
> > rskさん、ありがとうございました!
> > おかげでマージ＆リリース完了しました。
> > リンク変換部分のコードがずいぶんスッキリしました。
> >
> > ところで、p2ex-050504-0219 showthreadk.class.php の321行目、
> > $msg = preg_replace_callback('/(&gt;|＞){1,2}([1-9][0-9\\-,]+)/', 
> > array($this, 'quote_res_callback'), $msg);
> > は、
> > $msg = 
> > preg_replace_callback('/((?:&gt;|＞){1,2})([1-9](?:[0-9\\-,])*)+/', 
> > array($this, 'quote_res_callback'), $msg);
> > かなと、思ったのですがどうでしょう？
> >
> > --
> > aki
> >
> > ----------------------- Original Message -----------------------
> > From:    rsk <moons****@s32*****>
> > To:      p2-ph****@lists*****
> > Date:    Tue, 3 May 2005 19:26:33 +0900
> > Subject: [P2-php-dev 29] Re: transMsg()
> >
> >> rskです。
> >>
> >> 新しいURL/引用/IDリンク方式もだいぶバグがとれてきたので
> >> 前のtransmsg2.zipに現時点での拡張パック最新版相当の改良・修正を
> >> 施したものを添付します。
> >>
> >> // +------------------------------+
> >> // | rsk <moons****@s32*****> |
> >> // +------------------------------+
> >>
> >
> >
> >
> > _______________________________________________
> > P2-php-dev mailing list
> > P2-ph****@lists*****
> > http://lists.sourceforge.jp/mailman/listinfo/p2-php-dev
> >
> 
> _______________________________________________
> P2-php-dev mailing list
> P2-ph****@lists*****
> http://lists.sourceforge.jp/mailman/listinfo/p2-php-dev

p2

[P2-php-dev 34] Re: transMsg()