PEAK XOOPS - Re: piCal 0.84 in englishin japanese

Re: piCal 0.84

  • As this forum is only for commentation, you cannot open a new topic
  • Guests cannot post into this forum
Target Downloads
Subject
Summary
Previous post - Next post | Parent - Children.1 | Posted on 2005/11/15 17:50
okuhiki  伍長   Posts: 47
\modules\piCal0.84\class\piCal.php Line 157
もしかしたら、
header( $reload_str4header ) ;
でしょうか。
Line 156 で $reload_str4header がセットされただけで未使用だったので。

\modules\piCal0.84\include\read_configs.php Line 90
リファレンスを受け取るようですが、Noticeが表示されます。
私の力量では詳細までは判りませんが、XoopsMemberHandler(member.php)のgetGroupsByUser関数はリファレンスを返す作りになっていない為にNoticeが表示されるのかな。
XoopsMembershipHandler extends XoopsObjectHandler(group.php)のgetGroupsByUser関数はリファレンスを返してますね。

\modules\piCal0.84\class\piCal_xoops.php Line 1010
\modules\piCal0.84\admin\admin.php Line 122
リファレンスの受け取り。
XoopsGroupPerm extends XoopsObjectのgetItemIds関数はリファレンスを返してくれない。
\modules\piCal0.84\admin\mygrouppermform.php Line 160 では値をそのまま受け取っていますね。

\modules\piCal0.84\class\piCal_xoops.php に記述された、以下の関数は「配列を返す」とコメントに書かれていますが、使用時にはリファレンスを受け取ろうとしてNoticeが出るように感じます。
get_blockarray_coming_event
get_blockarray_new_event
get_blockarray_date_event

リファレンスの受け取りに関しては、私の能力を越えた部分ですから詳細不明につき、見当違いな書き込みなら無視して下さい。

-----
他所の投稿へのコメントも合わさせてここに書かせて頂きます。
Quote:
GIJOE:
留守中にいろいろとレスをつけていただき、ありがとうございました。かなり多くのスレッドが解決していてとても助かりました。
意味が判っての投稿なら良いのでしょうが、中途半端な理解力での投稿ばかりで、ご迷惑をおかけしていないかと心配しています。

新刊本「PHP サイバーテロの技法 攻撃と防御の実際」の発売日が待ち遠しいです。(^o^)ワクワク
-----

今回はさらに中途半端ですが、よろしくお願いします。
Votes:3 Average:3.33

normal Re: piCal 0.84

msg# 1.1
Previous post - Next post | Parent - Children.1 | Posted on 2005/11/16 11:02
GIJOE  先任軍曹   Posts: 4110
okuhikiさん、こんにちは。またまた、丁寧なご指摘ありがとうございます。

Quote:
\modules\piCal0.84\class\piCal.php Line 157
もしかしたら、

header( $reload_str4header ) ;

でしょうか。
Line 156 で $reload_str4header がセットされただけで未使用だったので。
typoのご指摘ありがとうございます。0.84a で修正します。

Quote:
\modules\piCal0.84\include\read_configs.php Line 90
リファレンスを受け取るようですが、Noticeが表示されます。
私の力量では詳細までは判りませんが、XoopsMemberHandler(member.php)のgetGroupsByUser関数はリファレンスを返す作りになっていない為にNoticeが表示されるのかな。
XoopsMembershipHandler extends XoopsObjectHandler(group.php)のgetGroupsByUser関数はリファレンスを返してますね。
とりあえずスカラー型および配列型は「参照を返す」こと自体がPHPの特性に合っていません。というわけで、ハンドラなどの、明示的なオブジェクト以外については、値で受け取るようにします。

このあたり、XOOPSの初期設計の問題でもありますね。module.textsanitizer.php なんかは、やたらと参照渡しを利用していますが、PHPの特性(リファレンスカウント)を知っていたらこんなコードにはなりません。これらもすべて値渡しに統一すべきでしょう。

Quote:
\modules\piCal0.84\class\piCal_xoops.php に記述された、以下の関数は「配列を返す」とコメントに書かれていますが、使用時にはリファレンスを受け取ろうとしてNoticeが出るように感じます。
get_blockarray_coming_event
get_blockarray_new_event
get_blockarray_date_event
これも私のミスですね。
PHPでは、配列であろうと、参照ではなく値で受け取るべきです。

0.84aの変更で、piCalが出すNoticeが無くなれば良いのですが。
Votes:1 Average:10.00

normal Re: piCal 0.84

msg# 1.1.1
Previous post - Next post | Parent - No child | Posted on 2005/11/22 15:23
okuhiki  伍長   Posts: 47
いつもながら早い対応をありがとうございます。

すぐにダウンロードさせて頂いておりましたが、レスを付けていませんでした。失礼しました。
レスに書かれている意味の深い部分は未だに理解不能な進歩しないユーザですが今後もよろしくお願いします。
肝心の「Notice」ですが、出ていません。ありがとうございます。
Votes:5 Average:10.00

  Advanced search


Login
Username or e-mail:

Password:

Remember Me

Lost Password?

Register now!