Nucleus CMSの新リリースを準備するためのリポジトリ。現在はNucleus CMS 4.0にマージするためのコードをコミットしている。
Revision | da44c01b57689e2a5f80267d52724684ef9be35e (tree) |
---|---|
Time | 2012-06-29 01:07:07 |
Author | sakamocchi <o-takashi@saka...> |
Commiter | sakamocchi |
前回のコミットの補遺
コミット「7ea26abd7e41c2644f42ce936f6557d58755cb3a」は、Manager::notify()で参照が付与された変数を扱う際に、PHPの文法に従っていなかったというバグの修正が目的だった。
すなわち、以下のPHPマニュアルを参照して
参照渡しで引数を作成する > 関数の引数 @ jp2.php.net
http://jp2.php.net/manual/ja/functions.arguments.php#functions.arguments.by-reference
参照を付与した変数に関して以下のことが言える。
A. 関数・メソッド定義の引数に記述することはできる
B. 関数・メソッド呼び出しの引数には記述できない
修正漏れしていたいくつかのスクリプトを、このルールに従い、このコミットで修正する。
@@ -2357,14 +2357,12 @@ class Admin | ||
2357 | 2357 | if ($blog->getDefaultCategory() == $catid) { |
2358 | 2358 | return _ERROR_MOVEDEFCATEGORY; |
2359 | 2359 | } |
2360 | - $manager->notify( | |
2361 | - 'PreMoveCategory', | |
2362 | - array( | |
2363 | - 'catid' => &$catid, | |
2364 | - 'sourceblog' => &$blog, | |
2365 | - 'destblog' => &$destblog | |
2366 | - ) | |
2360 | + $data = array( | |
2361 | + 'catid' => &$catid, | |
2362 | + 'sourceblog' => &$blog, | |
2363 | + 'destblog' => &$destblog | |
2367 | 2364 | ); |
2365 | + $manager->notify('PreMoveCategory', $data); | |
2368 | 2366 | // update comments table (cblog) |
2369 | 2367 | $query = 'SELECT ' |
2370 | 2368 | . ' inumber ' |
@@ -2400,14 +2398,13 @@ class Admin | ||
2400 | 2398 | . 'WHERE ' |
2401 | 2399 | . ' catid = %d'; |
2402 | 2400 | sql_query(sprintf($query, $destblogid, $catid)); |
2403 | - $manager->notify( | |
2404 | - 'PostMoveCategory', | |
2405 | - array( | |
2406 | - 'catid' => &$catid, | |
2407 | - 'sourceblog' => &$blog, | |
2408 | - 'destblog' => $destblog | |
2409 | - ) | |
2401 | + | |
2402 | + $data = array( | |
2403 | + 'catid' => &$catid, | |
2404 | + 'sourceblog' => &$blog, | |
2405 | + 'destblog' => $destblog | |
2410 | 2406 | ); |
2407 | + $manager->notify('PostMoveCategory', $data); | |
2411 | 2408 | return; |
2412 | 2409 | } |
2413 | 2410 |
@@ -200,7 +200,8 @@ class MediumActions extends BaseActions | ||
200 | 200 | public function parse_pluginextras() |
201 | 201 | { |
202 | 202 | global $manager; |
203 | - $manager->notify("MediaUploadFormExtras", array()); | |
203 | + $data = array(); | |
204 | + $manager->notify("MediaUploadFormExtras", $data); | |
204 | 205 | return; |
205 | 206 | } |
206 | 207 |
@@ -75,7 +75,8 @@ | ||
75 | 75 | $trackbacks[] = _getArrayVal($tblist, $i); |
76 | 76 | } |
77 | 77 | |
78 | - $manager->notify('SendTrackback', array ('tb_id' => $itemid, 'urls' => & $trackbacks)); | |
78 | + $data = array('tb_id' => $itemid, 'urls' => & $trackbacks); | |
79 | + $manager->notify('SendTrackback', $data); | |
79 | 80 | } |
80 | 81 | |
81 | 82 | return $res; |
@@ -209,7 +210,8 @@ | ||
209 | 210 | $trackbacks[] = _getArrayVal($tblist, $i); |
210 | 211 | } |
211 | 212 | |
212 | - $manager->notify('SendTrackback', array ('tb_id' => $itemid, 'urls' => & $trackbacks)); | |
213 | + $data = array('tb_id' => $itemid, 'urls' => & $trackbacks); | |
214 | + $manager->notify('SendTrackback', $data); | |
213 | 215 | } |
214 | 216 | |
215 | 217 | return $res; |
@@ -532,4 +534,4 @@ | ||
532 | 534 | |
533 | 535 | ) |
534 | 536 | ); |
535 | -?> | |
\ No newline at end of file | ||
537 | +?> |
@@ -188,8 +188,9 @@ | ||
188 | 188 | |
189 | 189 | $trackbacks = array (); |
190 | 190 | $tbstruct = array (); |
191 | - | |
192 | - $manager->notify('RetrieveTrackback', array ('tb_id' => $itemid, 'trackbacks' => & $trackbacks)); | |
191 | + | |
192 | + $data = array('tb_id' => $itemid, 'trackbacks' => & $trackbacks); | |
193 | + $manager->notify('RetrieveTrackback', $data); | |
193 | 194 | |
194 | 195 | while (list(,$v) = each ($trackbacks)) { |
195 | 196 | $tbstruct[] = new xmlrpcval( |
@@ -423,4 +424,4 @@ | ||
423 | 424 | |
424 | 425 | |
425 | 426 | |
426 | -?> | |
\ No newline at end of file | ||
427 | +?> |