rhaco-ja project in fixdap

generatorが吐くTableモデルにおけるextraのchoicesに関して

tokushima — 2008-11-29T02:02:02+09:00

generatorが吐くTableモデルにおけるextraのchoicesに関して

yabeken — 2008-11-28T16:40:06+09:00

Title | generatorが吐くTableモデルにおけるextraのchoicesに関して
Status | 完了
Priority| Middle
Handler | tokushima
Deadline| 2008-11-29 15:56 (+0900)

project.xmlでextra内にchoices書くと、
生成されるHogeTableクラスのcaptionFugaメソッドが
Hoge::extraFuga()を呼び出すべきところを、Hoge::columnFuga()になっている。
このため、Fatal Errorで止まる。

setup/resources/templates/library/model/table/TableObjectColumn.inc Line 58

- return TableObjectUtil::caption($this,{$table.method}::column{$column.method}());
+ return TableObjectUtil::caption($this,{$table.method}::{$functype}{$column.method}());

で行けました。

[要望] confirmedCreate, confirmedUpdateでbackした時の処理

yabeken — 2008-11-11T14:17:43+09:00

[相談]SMTP->mail()のRCPT TO:生成の修正案

kanonji — 2008-09-20T01:20:14+09:00

[相談]SMTP->mail()のRCPT TO:生成の修正案

tokushima — 2008-09-19T22:32:04+09:00

[相談]SMTP->mail()のRCPT TO:生成の修正案

kanonji — 2008-09-19T15:25:57+09:00

Title | [相談]SMTP->mail()のRCPT TO:生成の修正案
Status | 完了
Priority| Middle
Handler | kanonji
Deadline| 2008-09-19 01:00 (+0900)

一度コミットし、完了にしたタスクですが、その後のLingrでの意見などを参考にすると、下手にMail->_mailstr()を変えるより、こうしたほうがいいかなと考え直しました。
そしてあわよくばMail->_mailstr()は削除か、@deprecatedに・・・
どうでしょうか？

<=1.6.0
Index: SMTP.php
===================================================================
--- SMTP.php (revision 876)
+++ SMTP.php (working copy)
@@ -113,8 +113,8 @@
if(Variable::istype("Mail",$mail)){
if($this->cmd(sprintf("MAIL FROM: <%s>",$mail->from))){
$fail = false;
- foreach ($mail->_mailstr() as $rcptTo){
- $fail = $this->cmd($rcptTo) ? $fail : true;
+ foreach (array_keys($mail->to) as $to){
+ $fail = $this->cmd(sprintf("RCPT TO: <%s>",$to)) ? $fail : true;
}
if(!$fail){
if($this->cmd("DATA")){

arbo
Index: Gmail.php
===================================================================
--- Gmail.php (revision 64)
+++ Gmail.php (working copy)
@@ -72,7 +72,9 @@
$this->_talk($fp,base64_encode($this->password));

$this->_talk($fp,sprintf("MAIL FROM: <%s>",$this->login));
- $this->_talk($fp,sprintf("RCPT TO: <%s>",$this->_mailstr($this->to)));
+ foreach(array_keys($this->to) as $to){
+ $this->_talk($fp,sprintf("RCPT TO: <%s>",$to));
+ }
$this->_talk($fp,"DATA");
$this->_talk($fp,$this->manuscript().".");
$rtn = $this->_talk($fp,"QUIT");

[相談]network.mail.Mailのmessageの改行コードを他と同様にコンストラスタでセットしたeolに管理させたい

kanonji — 2008-09-19T14:48:01+09:00

Title | [相談]network.mail.Mailのmessageの改行コードを他と同様にコンストラスタでセットしたeolに管理させたい
Status | 未着手
Priority| Middle
Handler | Undecided
Deadline| 2008-09-20 11:05 (+0900)

※2箇所で利用してるプロパティなので、自分の中で整理できてないです。
　まずは相談という事で、お願いします。

network.mail.Mailのmessageの改行コードを、他と同様にコンストラスタでセットしたeolに管理させたいです。

Mailが内部で生成するメールソースで、Mail->message()でセットするbody部だけが、唯一改行コードが他の部分と扱いが違っています。
Mailではコンストラスタでのフラグで、CRLFかLFかを選択できますが、Mail->message()の場合は引数で渡したままの改行コードが、最後まで維持されます。

これは最終的にMail->send()内でcallしてる組み込みのmail()の仕様の為だと思いますが、network.mail.SMTPからMailインスタンスを利用する場合に、トラブルになる場合があります。
つまり、Mail->message()の改行コードに対して、Mail->send()はLF、SMTP->mail()はCRLFを、求めています。

また、これとは別に文字コードが絡む不安要素がいくつかあります。
・組み込みmail()のbody部を受け取る引数は、マニュアルではLFを必要としますが、実際のところCRLFでも問題ないようです。
※yabekenさんがcoreserverで試した限り。

・組み込みmail()の第4引数(additional headers)は、原則CRLFですが、マニュアルによると場合によってはLFにする必要があるとの事。
　LinuxのMTAの一部が、無条件にLFをCRLRに置換し、CRCRLRになってしまうようです。
※webで検索する限り、postfixとsendmail。修正済みなのか、最新バージョンでもこの動作をするのか、分かりません。

色々と分からない点はありますが、組み込みmail()の第3引数がCRLFでも動作するらしいので、改行コードの変換をしてみました。
組み込みmail()の第3引数は、マニュアルに従うなら、Mail->send()内で改めてCRLFをLF化すればいいんじゃないかと思ったりしています。

Index: Mail.php
===================================================================
--- Mail.php (revision 876)
+++ Mail.php (working copy)
@@ -372,6 +372,35 @@
* @param string $value
*/
function _lw($value=""){
+ /***
+ * $r = "\r";
+ * $n = "\n";
+ * $f = new File(Rhaco::constant('CONTEXT_PATH').'/__settings__.php');
+ * $b = base64_encode($f->read());
+ * $bCR = trim(chunk_split($b,76,$r));
+ * $bLF = trim(chunk_split($b,76,$n));
+ * $bCRLF = trim(chunk_split($b,76,"{$r}{$n}"));
+ * $ar = array(new Mail(), new Mail('','',true));
+ * foreach($ar as $mail){
+ * $e = $mail->eol;
+ * eq($e,$mail->_lw());
+ * eq("example{$e}",$mail->_lw('example'));
+ * eq("This{$e}is{$e}a{$e}example{$e}",$mail->_lw("This{$n}is{$n}a{$n}example"));
+ * eq("This{$e}is{$e}a{$e}example{$e}",$mail->_lw("This{$r}{$n}is{$r}a{$n}example"));
+ * eq("{$e}{$e}{$e}{$e}",$mail->_lw("{$n}{$r}{$r}{$n}"));
+ * $be = trim(chunk_split($b,76,$mail->eol));
+ * eq("$be{$e}",$mail->_lw($be));
+ * eq("$be{$e}",$mail->_lw($bCR));
+ * eq("$be{$e}",$mail->_lw($bLF));
+ * eq("$be{$e}",$mail->_lw($bCRLF));
+ * }
+ */
+ if(strpos($value,"\n") !== false || strpos($value, "\r") !== false){
+ $ptnCRLF = '/\r\n/';
+ $ptnLF = '/(?+ $ptnCR = '/\r(?!\n)/';
+ $value = $this->eol === "\r\n" ? preg_replace(array($ptnLF, $ptnCR), $this->eol, $value) : preg_replace(array($ptnCRLF, $ptnCR), $this->eol, $value);
+ }
return $value.$this->eol;
}

[相談]SMTP->mail()のRCPT TO:生成の修正案

kanonji — 2008-09-19T06:33:47+09:00

Title | [相談]SMTP->mail()のRCPT TO:生成の修正案
Status | 完了
Priority| Middle
Handler | kanonji
Deadline| 2008-09-19 01:00 (+0900)

Committed.
http://code.google.com/p/rhaco/source/detail?r=876 1.6.0
http://code.google.com/p/rhaco/source/detail?r=877 1.6.1

若干の訂正を加え、下記内容でコミットしました。

Index: Mail.php
===================================================================
--- Mail.php (revision 864)
+++ Mail.php (working copy)
@@ -179,13 +179,25 @@
}

/**
- * アドレス保持配列からメールアドレスのみを取得する
+ * アドレス保持配列をRCPT TO: foo@example.comの形式の配列で返す
*
- * @param array $list メールアドレス配列
- * @return string
+ * @return array
*/
- function _mailstr($list){
- return implode(",",array_keys($list));
+ function _mailstr(){
+ /***
+ * $mail = new Mail;
+ * $mail->to('foo@example.com');
+ * $ar = $mail->_mailstr();
+ * eq('RCPT TO: ',$ar[0]);
+ * $mail->to('bar@example.com');
+ * $ar = $mail->_mailstr();
+ * eq('RCPT TO: ',$ar[1]);
+ */
+ $array=array();
+ foreach(array_keys($this->to) as $to){
+ $array[] = sprintf("RCPT TO: <%s>",$to);
+ }
+ return $array;
}

/**
Index: SMTP.php
===================================================================
--- SMTP.php (revision 864)
+++ SMTP.php (working copy)
@@ -112,7 +112,11 @@
function mail($mail){
if(Variable::istype("Mail",$mail)){
if($this->cmd(sprintf("MAIL FROM: <%s>",$mail->from))){
- if($this->cmd(sprintf("RCPT TO: <%s>",$mail->_mailstr($mail->to)))){
+ $fail = false;
+ foreach ($mail->_mailstr() as $rcptTo){
+ $fail = $this->cmd($rcptTo) ? $fail : true;
+ }
+ if(!$fail){
if($this->cmd("DATA")){
if($this->cmd($mail->manuscript().".")){//from to subject x-mailer
return $this->getText();

[相談]SMTP->mail()のRCPT TO:生成の修正案

yabeken — 2008-09-18T18:07:16+09:00

Title | [相談]SMTP->mail()のRCPT TO:生成の修正案
Status | 完了
Priority| Middle
Handler | kanonji
Deadline| 2008-09-19 01:00 (+0900)

コピペ

LFで送信してるのは、PHPの内部コードがToだけLFでsendmailに投げてるからだったかな。
そのせいで、BodyとかをCRLFで送るとCRCRLFにMTAが変えちゃってーとかだった気が・・・。
RFCで規定されているからSMTPはCRLFで送らないと怒られますねー

[相談]SMTP->mail()のRCPT TO:生成の修正案

kanonji — 2008-09-18T16:53:27+09:00

Title | [相談]SMTP->mail()のRCPT TO:生成の修正案
Status | 完了
Priority| Middle
Handler | kanonji
Deadline| 2008-09-19 01:00 (+0900)

network.mail.SMTPのmail()はnetwork.mail.Mailのインスタンスを引数に貰って、それをSMTPで送信してくれます。
Mail->toに2個以上の宛先がセットされている場合、現状ではメールの仕様に合わない形式で、RCPT TO:を作っているようです。

処理はMail->_mailstr()をcallしているので、こちらを変えてみました。
_mailstr()はSMTP->mail()からのみcallされているメソッドだった為、_mailstr()を消してmailRcptTo()を作りました。
STMP->mail()の方も、多少変えてます。

下のdiffは、自分のやりたい事をやる為の修正なので、_mailstr()を消すとかも含め、まずは相談したいです。
実は_mailstr()は使う場面もあるから残す必要があるとかあるかもですし。

Index: Mail.php
===================================================================
--- Mail.php (revision 864)
+++ Mail.php (working copy)
@@ -179,13 +179,25 @@
}

/**
- * アドレス保持配列からメールアドレスのみを取得する
+ * アドレス保持配列をRCPT TO: foo@example.comの形式の配列で返す
*
- * @param array $list メールアドレス配列
- * @return string
+ * @return array
*/
- function _mailstr($list){
- return implode(",",array_keys($list));
+ function mailRcptTo(){
+ /***
+ * $mail = new Mail;
+ * $mail->to('foo@example.com');
+ * $ar = $mail->mailRcptTo();
+ * eq('RCPT TO: ',$ar[0]);
+ * $mail->to('bar@example.com');
+ * $ar = $mail->mailRcptTo();
+ * eq('RCPT TO: ',$ar[1]);
+ */
+ $array=array();
+ foreach(array_keys($this->to) as $to){
+ array_push($array, sprintf("RCPT TO: <%s>",$to));
+ }
+ return $array;
}

/**
Index: SMTP.php
===================================================================
--- SMTP.php (revision 864)
+++ SMTP.php (working copy)
@@ -112,7 +112,11 @@
function mail($mail){
if(Variable::istype("Mail",$mail)){
if($this->cmd(sprintf("MAIL FROM: <%s>",$mail->from))){
- if($this->cmd(sprintf("RCPT TO: <%s>",$mail->_mailstr($mail->to)))){
+ $fail = false;
+ foreach ($mail->mailRcptTo() as $rcptTo){
+ $fail = $this->cmd($rcptTo) ? $fail : true;L::debug($fail);
+ }
+ if(!$fail){
if($this->cmd("DATA")){
if($this->cmd($mail->manuscript().".")){//from to subject x-mailer
return $this->getText();

[バグ]HtmlParserでiframeの処理が無限ループする

tokushima — 2008-09-13T02:00:01+09:00

[バグ]HtmlParserでiframeの処理が無限ループする

tokushima — 2008-09-13T01:52:51+09:00

[バグ]HtmlParserでiframeの処理が無限ループする

redriver — 2008-09-12T18:44:24+09:00

Title | [バグ]HtmlParserでiframeの処理が無限ループする
Status | 完了
Priority| Middle
Handler | tokushima
Deadline| 2008-09-13 18:35 (+0900)

iframeにhrefが無いと無限ループになりましたよ

こんな感じにしたら直りました

while(true){
$list = $tag->getIn("iframe");
if(sizeof($list) <= 0) break;
$href = 0;
foreach($list as $obj){
if($this->_isReference($obj)){
$url = Url::parseAbsolute(dirname(Url::parseAbsolute($this->path,$this->filename)),$obj->getParameter("href"));
$src = str_replace($obj->getPlain(),$this->read($url),$src);
$href++;
}
}
$tag->set($src,"body");
if($href==0) break;
}

[要望] confirmedCreate, confirmedUpdateでbackした時の処理

shigepon — 2008-09-09T22:18:54+09:00

SimpleTag::toHash で 2147483647 を越える数字をもつ要素も intval されてしまう

tokushima — 2008-09-09T21:50:08+09:00

SimpleTag::toHash で 2147483647 を越える数字をもつ要素も intval されてしまう

riaf — 2008-09-09T19:47:14+09:00

Title | SimpleTag::toHash で 2147483647 を越える数字をもつ要素も intval されてしまう
Status | 完了
Priority| Middle
Handler | Undecided
Deadline| 2008-09-01 22:53 (+0900)

でも、ASINとかはそもそも数値ではないので、int で無ければならない！訳ではないと思います。むしろこの場合は文字列が正しいかと。
なので、toHashでは勝手なcastはしない方が良いんじゃないかなーとか思ったりもします。

[バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す

kanonji — 2008-09-09T19:45:40+09:00

Title | [バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す
Status | 完了
Priority| Middle
Handler | Undecided
Deadline| 2008-09-10 05:44 (+0900)

Committed.
http://code.google.com/p/rhaco/source/detail?r=871 1.6.1
http://code.google.com/p/rhaco/source/detail?r=870 1.6.0

[バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す

kanonji — 2008-09-09T19:06:48+09:00

Title | [バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す
Status | 完了
Priority| Middle
Handler | Undecided
Deadline| 2008-09-10 05:44 (+0900)

- if(!empty($this->chartype)) $result .= "chartype=".$this->chartype.",";
+ if(!empty($this->chartype)) $result .= "chartype=".str_replace(",","\\,",$this->chartype).",";

で大丈夫みたいです。

"column=modelnumber,variable=modelnumber,type=string,chartype=/[A-Z]{2\,3}-[A-Z0-9]{3\,5}/,"という辞書が生成されました。

[バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す

tokushima — 2008-09-09T18:32:45+09:00

Title | [バグ?]project.xmlのchartypeに含むカンマが、ArrayUtil::dict()渡されColumnクラスのコンストラクタの引数を壊す
Status | 完了
Priority| Middle
Handler | Undecided
Deadline| 2008-09-10 05:44 (+0900)

ColumnModel::getConstractArg
で
if(!empty($this->chartype)) $result .= "chartype=".$this->chartype.",";
のところを

if(!empty($this->chartype)) $result .= "chartype=".str_replace(",","\\,",$this->chartype).",";

でどうですか？

SimpleTag::toHash で 2147483647 を越える数字をもつ要素も intval されてしまう

tokushima — 2008-09-09T18:27:36+09:00