FIX:Strict Standards: Only variables should be passed by reference in
ITEMACTIONS.php on line 820 の修正

「メンバーの編集(editmembersettings)」画面でスキン変数<%pluginextras%>がパースされていなかった不具合を修正

MERGE: リビジョン1992。テンプレート「SHOWLIST_LISTPLUG_TABLE_NAVILIST」の追加

<%commentnavlist%>、<%itemnavlist%>のナビゲーション部分はハードコーディングされていたが、これをテンプレートで編集可能とした。

Revision 1992:
ADD: template named as SHOWLIST_LISTPLUG_TABLE_NAVILIST
This template is for navigation list for items and comments.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1902

MERGE: リビジョン1991。Adminクラスにおける<%skinfile%>の引数を修正

BaseActions::parse_skinfile()はディレクトリ区切り文字を付加しないことを考慮した修正。

Revision 1991:
FIX: the wrong path for <%skinfile%> in Admin class
BaseActions::parse_skinfile() don't add path separator. So <%skinfile%>
needs it.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1901

MERGE: リビジョン1894〜1990

Revision 1894:
CHANGE: make setlocale() do in include_translation()
The aim of this patch is preventing from E_NOTICE/E_STRICT/E_WARNING
generated by time/date-related function. If the PHP process has no
locale settings, these functions generate these messages.
What is that PHP process has locale settings:
setlocale() is executed in the script in advance of executing
time/date-related functions.
To confirm this, this patch executes setlocale() in
include_translation() in globalfunctions.php, refering to
i18n::get_current_locale() or i18n::get_forced_locale().
But there are some problems:
1. Some translation files executes setlocale() when they are included.
2. Some templates includes LOCALE template and Template:read() execute
setlocale() with it.
3. <%archivedate%> can given locale argument and execute setlocale()
with it.
4. The arguments of setlocale() depends on each implement of C library.
This causes apparent difference between Windows and the others.
To solve these:
1. Remove for unification of setlocale()
2. Remove for unification of setlocale()
3. Deprecate for unification of setlocale()
4. Add _LOCALE_IN_WINDOWS constant value in each translation files and
include_translation() include it when the server is  running under
windows operating system.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1894

Revision 1895:
supplemental for previous commit
I did fail to patch for ja_Jpan_JP.EUC-JP.php at 1894.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1895

Revision 1896:
FIX: reference issue
Making arguments be passed by reference > Function arguments @ php.net
http://php.net/manual/en/functions.arguments.php
According to PHP document, the value with reference:
A. can be written in arguments of functions or class methods
B. can't be given to arguments of  calling functions or class methods
In this grammer,
A. function hoge(&$data) is OK
B. hoge(&$data) isn't OK
And if we want to give array as argument for calling function or class
methods, we should put array to a variable once.
This commit modify some codes related to these points
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1896

Revision 1897:
ADD: give an argument for template name to <%commentnavlist%> tag
Add missing argument for AdminActions::parse_commentnavlist().
And modify the comment of AdminActions::parse_itemnavlist()
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1897

Revision 1898:
ADD: E_STRICT for $CONF['debug'] mode
Previously E_ALL is set for $CONF['debug'] mode but E_STRICT should also
be applied.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1898

Revision 1899:
FIX: Strict Standards Errors
Some codes generates Strict Standards Errors. This commit fix these.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1899

Revision 1890:
FIX: enable update scripts to run previous version
We changed upgrades scripts with some functions added in 4.0 but it's
wrong. This commit
fix it and add some codes for 4.0.
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1890

前回のコミットの補遺

コミット「7ea26abd7e41c2644f42ce936f6557d58755cb3a」は、Manager::notify()で参照が付与された変数を扱う際に、PHPの文法に従っていなかったというバグの修正が目的だった。
すなわち、以下のPHPマニュアルを参照して

参照渡しで引数を作成する > 関数の引数 @ jp2.php.net
http://jp2.php.net/manual/ja/functions.arguments.php#functions.arguments.by-reference

参照を付与した変数に関して以下のことが言える。
A. 関数・メソッド定義の引数に記述することはできる
B. 関数・メソッド呼び出しの引数には記述できない

修正漏れしていたいくつかのスクリプトを、このルールに従い、このコミットで修正する。

FIX:$manager->notify()の第二引数に変数を渡すように修正。

スクリプトに含まれるすべての$manager->notify()の第二引数を変数に変更した
プラグインメソッドは、引数を参照で受け取るようにする必要がある。

Revert "FIX:プラグインに渡す引数が参照で無いためプラグインオブジェクトのイベントメソッドが呼ばれない問題を修正。"

実際は上記コミット後に以下の修正が入っていたので改行コードをLFに変更した。
 FIX:MANAGER::instance()をstaticに
 いくつかのstandardsエラーの修正

いくつかのstandardsエラーの修正

FIX:MANAGER::instance()をstaticに

FIX:プラグインに渡す引数が参照で無いためプラグインオブジェクトのイベントメソッドが呼ばれない問題を修正。

PreItemイベントでNP_AccessControlが正常に動作しない事例を確認。
call_user_func()の第二引数を'&$data'とするとメソッドが呼び出されることを確認。

FIX:アップグレードスクリプトがNucleus3.6上で動作するように修正

全アップグレードスクリプトのデータベースハンドラの変更を元に戻した。
（完全にRevertとも言えないのでFIXとしています）

ADD:バージョン4.0へのアップグレードスクリプトを作成

MERGE: リビジョン1873〜1893。skinnable-masterのマージ

skinnable-masterを上流にマージした。

加えて、コード規則に従い、改行記号はUnixライクなLFに統一した。

FIX: blogsettingsのマークアップ修正

FIX: skinieoverview/templateeditのマークアップ修正

FIX: 管理画面用スキンのマークアップ修正

table要素のthead/tbody子要素の追加

CHANGE: <%locale%>を再整備

XML要素の属性に用いるための<%locale%>を、language-regionを出力するように変更。
管理画面用スキンがja-JPを含んでいたため置換した。

REMOVE: contemporary_jpに関するファイルやコードを除外

日本語用パッケージ由来のファイルは上流に含まないほうがよい

FIX: Admin::$ticketless_actionsにbrowseownitemsを追加

チケット不要なアクションだったため

FIX: AdminActions::$normal_skin_typesの変更

liginとmembereditを除外
ほか、normalに関する前回変更の修正

CHANGE: ノーマルスキンタイプに関する修正

標準のスキンタイプをノーマルと定義し、クラスメンバー・メソッドをnormalという名前に統一した。

FIX: ラベル「includeプリフィックス」が表示されないバグ（skinoverview/adminskinoverview）

showlist.phpにおいて定数の参照が間違えていたバグの修正。

FIX: NP_Mediaの翻訳メッセージが適用されないバグ、ファイルサイズが表示されないバグ

ラベル名が異なるためメッセージが適用されなかった。
MediaObject::refine()をしなければファイルサイズや画像の縦横比が取得出来なかった。

CHANGE: メディア管理プラグインがない場合、ダイアログを表示した際にアラートを表示

NP_MediumあるいはNP_MediaTocuなどのプラグインがインストールされていない場合、アイテム追加・編集画面でダイアログを開くと「Media
Plugin Not Fount」のアラートが出るよう修正。
合わせて、JavaScriptのメソッド「setMediaPopupURL」を削除。

ADD: NP_Mediumに日本語訳ファイルを追加

FIX: 管理画面用スキンのマークアップの修正

table要素に関して

Merge branch 'skinnable-master' of git.sourceforge.jp:/gitroot/nucleus-jp/nucleus-next into skinnable-master

FIX: 管理画面用スキンのマークアップの修正

div要素とp要素、table要素に関してXHTML1.0に従い修正。

FIX: 管理画面用スキンのマークアップの修正

div要素とp要素、table要素に関してXHTML1.0に従い修正。

CHANGE: <%systemsettings(modrewrite)%>の処理を変更

これまでの実装はphpinfo(INFO_MODULES)の出力から文字列「mod_rewrite」を探すことで検出していたが、ボトルネックが大きく、ページ出力時間のボトルネックとなっていた。そこで、apache_get_modules()が定義されている場合はこちらの出力を利用することで、ボトルネックの短縮を図るよう修正した。

NP_Mediumのインデントの修正

CHANGE: Adminクラスの整理

変更:
[アクセス修飾子をprivateからpublicに]
 Admin::$action
 Admin::$passvar
 Admin::$aOptions
[処理の変更]
 Admin::action()

整理と名前変更:
[チケット不要なアクションのリストの整理]
 Admin::$actions_needless_to_checkを
  Admin::$ticketless_actionsに
[管理画面用スキンアクションのリストの整理]
 Admin::$edit_actionsを
  Admin::$adminskin_actionsに

廃止:
[メンバーのアクセス修飾子をpublicにしたため]
 Admin::getAdminaOption()
 Admin::getAdminAction()
 Admin::getAdminpassvar()
 Admin::getAdminextrahead()
[Admin::action()の処理変更により不要となったため]
 Admin::existSkinContents()
[Skinクラスのメソッドで代替可能なため]
 Admin::getAdminskinNameFromID()
 Admin::getAdminskinIDFromName()
[Admin::pagehead()にハードコーティングしたため]
 Admin::$xml_version_info
 Admin::$formal_public_identifier
 Admin::$system_identifier
 Admin::$xhtml_namespace
[どこからも参照されてないため]
 Admin::specialActionsAllow()

FIX: Adminクラスにおいて自身のメンバーへのアクセスにselfを使っていないものを修正

REMOVE: Adminクラスの不要なメソッドを削除

以下のメソッドは以前のimportAdminページアクション用の実装であり、現在は不要。
Admin::action_doAdminskinimport()
Admin::action_importAdmin()

CHANGE:　Admin::skinieimport()とAdmin::skiniedoimport()からAdminActionsに変数を渡す方法

変数をグローバル空間で渡すと、プラグインのバッチ処理などで意図しない変更が加えられる可能性がある。$_REQUESTではなく、Admin::$contentsに変更した。

REMOVE:: Admin::addToAdminTemplate()の削除

Admin::addToTemplate()と全く同じ処理をしていたため。

FIX: Admin::action_akiniedoimport()のパースエラーの修正

self::skiniedoimport()のtypoが原因。

FIX: Adminクラス、showlist.phpのいくつかの関数でグローバル変数にアクセスできないバグの修正

コミット「eb080ad31a3105dac5cb1ba827ca2475df51b205」のエンバグ。

CHANGE: Adminクラスのスキン関係メソッドの整理・統合

管理画面/通常画面用ページアクションにおいて同じ処理をしている箇所がある。
コードのメンテナンスを簡便にするため、それらの処理を独立した以下のメソッドに集約した。

Admin::skinclone()
Admin::skindeleteconfirm()
Admin::skineditgeneral()Admin::skiniedoimport()
Admin::skinieexport()
Admin::skinieimport()
Admin::skinremovetypeconfirm()

FIX: ページアクション「createitem」のパースエラーの修正

ADD: 翻訳メッセージ「_ADMIN_SYSTEMOVERVIEW_VERSION_LATEST」を追加

前回のコミット 8a78b6e09d3c7521205a5fb6b4ace3d5bd0a0dfd
で新規に追加された翻訳メッセージを、他の翻訳メッセージファイルにも追加。

CHANGE:スキン変数<%newestcompare%>をpageheadからsystemoverviewへ移動
FIX:バージョンチェックの結果最新版だったときに最新版である旨表示するようにした

CHANGE: 時刻・日付表示のための翻訳メッセージを整理

翻訳メッセージの追加及び削除。それに関連する修正。

以下を新設:
_LABEL_YEAR_UNIT
_LABEL_MONTH_UNIT
_LABEL_DAY_UNIT
_LABEL_HOUR_UNIT
_LABEL_MINUTE_UNIT

以下を廃止:
_ITEM_ADDEDITTEMPLATE_YEAR
_ITEM_ADDEDITTEMPLATE_MONTH
_ITEM_ADDEDITTEMPLATE_DAY
_ITEM_ADDEDITTEMPLATE_HOUR
_ITEM_ADDEDITTEMPLATE_MINUTE
_ARCHIVETYPE_MONTH
_ARCHIVETYPE_DAY

CHANGE: ギリシャ文字を明示

Merge branch 'skinnable-master' of sakamocchi@git.sourceforge.jp:/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

MERGE: リビジョン1869。翻訳ファイルの拡充。

翻訳ファイルを拡充。不足メッセージをen_Latn_USからコピーし、重複定義を調整した。

Revision 1869: ADD: new translation file from language repository

From:
http://nucleuscms.org/languages.php

Added:
el_Latn_GR.UTF-8.php
el_Latn_GR.WINDOWS-1253.php
hr_Latn_HR.UTF-8.php
hr_Latn_HR.WINDOWS-1250.php
id_Latn_ID.ISO-8859-1.php
id_Latn_ID.UTF-8.php
ku_Arab_TR.UTF-8.php
pl_Latn_PL.UTF-8.php
ro_Latn_RO.ISO-8859-1.php
ro_Latn_RO.UTF-8.php
sr_Cyrl_RS.UTF-8.php
ur_Arab_PK.UTF-8.php
vi_Latn_VN.UTF-8.php
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1869

MERGE: リビジョン1869。翻訳ファイルの拡充。

翻訳ファイルを拡充。不足メッセージをen_Latn_USからコピーし、重複定義を調整した。

Revision 1869: ADD: new translation file from language repository

From:
http://nucleuscms.org/languages.php

Added:
el_Latn_GR.UTF-8.php
el_Latn_GR.WINDOWS-1253.php
hr_Latn_HR.UTF-8.php
hr_Latn_HR.WINDOWS-1250.php
id_Latn_ID.ISO-8859-1.php
id_Latn_ID.UTF-8.php
ku_Arab_TR.UTF-8.php
pl_Latn_PL.UTF-8.php
ro_Latn_RO.ISO-8859-1.php
ro_Latn_RO.UTF-8.php
sr_Cyrl_RS.UTF-8.php
ur_Arab_PK.UTF-8.php
vi_Latn_VN.UTF-8.php
http://nucleuscms.svn.sourceforge.net/viewvc/nucleuscms?view=revision&revision=1869

<%itemtime%>タグをパースする度にイベント「PreAddItemForm」が発生するバグの修正

イベント「PreAddItemForm」はAdmin::action_createitem()ですでに発生しているため、AdminActions::parse_itemtime()で発生する必要がない。

以下で報告されたバグの修正も含む
http://sourceforge.jp/projects/nucleus-jp/lists/archive/developers/2012-June/001059.html

Revert "FIX:ブックマークレットで、スキン変数<%itemtime%>が正しくパースされない不具合を修正"

This reverts commit 802236e993f1c975ecdb8eda62d0e3acb68aa5ab.

FIX:ブックマークレット画面内の改行追加とスタイルを修正

FIX:ブックマークレットで、スキン変数<%itemtime%>が正しくパースされない不具合を修正

FIX:変数名の誤記を修正

CHANGE: インストラーのヘッダーにロケール名が常に表示されるよう変更

ロケール名は長いものもある。
ページレイアウトを考慮して配置の再検討が必要。

FIX: bg_Cyrl_BG.UTF-8.phpのtypoの修正

define()をddefine()と記述していた。

FIX: ブックマークレット用スキンのアイテムの編集で自動ドラフト保存が効かないバグ

pagefootスキンで出力していたJavaScriptのコードが、itemeditなのにcreateitem用のものが挿入されていた。pagefootスキンからcreateitem/itemeditスキンにコードを移動した。

FIX: en_Latn_US翻訳メッセージファイルのtypoの修正。

FIX: バッチリスト、ナビリストでテンプレートが適用されないバグ

コミット「b82168398ee0160603b015a4930998f6e656c098」においてshowlist.phpに追加したlistplug_batchlist()とlistplug_navlist()だが、引数にテンプレート名を持たないため、その後の処理にテンプレートが適用されない不具合があった。

CHANGE: globalfunctions.phpのdoError()から制御を戻すよう修正

doError()はexitせず制御を呼び出し元に戻すように関連する箇所を含めて修正。

FIX: グローバル設定において、管理画面用/ブックマークレット用スキン選択肢に「サイトの設定を使う」が含まれるバグ

コミット「a9efdd5a0b245be17da4d64c9b4e0384c9411eb0」の修正。グローバル設定においては必ず管理画面用/ブックマークレット用スキンが選択されている必要がある。

FIX: Adminクラスの各メソッドが制御を戻すよう修正

エラーページ表示の際、処理が戻っていなかったために続きの処理が行われ、ページが重複して表示されていたバグの修正。今後の機能拡張に備える目的で、exitを使わずに処理を呼び出しファイルに戻すようにした。

Merge branch 'skinnable-master' of ssh://shizuki@git.sourceforge.jp/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

FIX:グローバル設定に準じている管理エリア・ブックマークレットのスキン設定を-サイトの設定を使う -と表示するように修正

CHANGE: インストラーのロケール選択画面に翻訳メッセージを追加

これまでは英語によるメッセージがハードコーディングされていた。
メッセージの内容を精査し、翻訳ファイルにメッセージを追加。

ユーザーエージェントの送信するAccept-Languageヘッダーフィールド情報に応じてロケールを切り替える処理は、RFC2616を参照して再確認する必要があると思われる。

CHANGE: ロケール名に翻訳メッセージが適用されるよう修正

ロケール名の一覧はAdmin::parse_localeselectoptions()と/install/index.phpにおいて表示される。
これまではlanguage tagsを援用した名前で表示されていたが
翻訳メッセージが適用されるよう修正した。

FIX:ブックマークレット用のメンバー毎のスキンが半円されていなかったのを修正
FIX:ブックマークレットのスキンエラーの時にヘッダ情報を送信するように修正

FIX:nucleus_menberテーブルのカラム追加に伴うインストールスクリプト修正

実質はインストールスクリプトの修正は必要無かった。
install.sqlのINSERT文の修正とスキンバックアップファイルから一部改行コードの除去を行った。

ADD: ロケールの翻訳メッセージを追加

en_Latn_US/ja_Jpan_JPにのみ、現在利用可能なロケール名に対する翻訳メッセージを追加した。翻訳メッセージが固まり次第、他の翻訳メッセージファイルにも追加したい。

なお、数値実体参照あるいは文字実体参照を用いてUSC-2に含まれるキャラクターで一律に記述する方法もあるが、ユーザーのコンピューターにそのキャラクターに対応するフォントファイルが含まれていなければ読むことができなくなる（この場合、いちキャラクターがひとつの四角として表示される）ため、それぞれの翻訳メッセージファイルで持つのが適切であると判断した。

FIX:NP_SecurityEnforcerのpreg_matchのデリミタを変更して「#」のエスケープを削除

FIX:MANAGER::$membersをMANGERクラスのconstruct時に初期化するように修正

FIX:メンバーに設定されている管理エリア用スキンが有効でない場合にデフォルトの管理エリア用スキンを使用するように修正

FIX:メンバーごとの管理エリア用スキンが反映されていなかったので修正
FIX:メンバーが設定しているスキンを削除できないように修正

FIX:NP_SecurityEnforcerの正規表現パターンの修正
「\」と「#」が正しくエスケープされていなかったので修正

FIX:MANAGER::getMember()がコールされたとき、MANAGER::membersがnullの時があるので修正

Merge branch 'skinnable-master' of ssh://shizuki@git.sourceforge.jp/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

ADD:メンバー毎の管理画面およびブックマークレット用スキン選択ボックスを追加
「メンバーの編集」画面にメンバー毎の管理画面およびブックマークレット用スキン選択用のスキン変数<%defadminskinselect%><%defbookmarkletselect%>を追加
上２つの表示用に日本語と英語の翻訳ファイルに翻訳文を追加

Merge branch 'skinnable-master' of sakamocchi@git.sourceforge.jp:/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

CHANGE: pagehead/pagefootスキンの内容が取得できない場合の処理を変更

BaseActionsクラスを継承したクラスであれば、$this-parser->parse()により任意の内容をパースすることができる。これを利用して、parse_pagehead()/parse_pagefoot()のコンテンツが取得できない場合の処理を変更した。

Merge branch 'skinnable-master' of ssh://shizuki@git.sourceforge.jp/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

ADD:デフォルトの管理画面およびブックマークレット用スキン選択ボックスを追加
「グローバル設定」画面にデフォルトの管理画面およびブックマークレット用スキン選択用のスキン変数<%defadminskinselect%><%defbookmarkletselect%>を追加
上２つの表示用に日本語と英語の翻訳ファイルに翻訳文を追加

FIX: SQL文のINSERT構文にVALUE()リストを記述していたバグの修正

VALUES()リストが正しい

FIX: media.phpをインクルードすることによって生じるエラーを修正

コミット「054ee431fa1b2f9be60c138216dcbb092f52bad9」の修正漏れ

FIX:ページをスクロールしている状態での動作を修正。

Merge branch 'skinnable-master' of ssh://shizuki@git.sourceforge.jp/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

FIX:デフォルトの管理画面用、ブックマークレット用テンプレートの名前が日本語だったので修正

FIX: オブジェクト名の誤りを修正

see.
http://sourceforge.jp/projects/nucleus-jp/lists/archive/developers/2012-May/001034.html

不要ファイル削除時の修正漏れの修正

REMOVE: 不要ファイルの削除とそれに伴う修正

以下の不要ファイルを削除した。
/createaccount.html
/nucleus/javascript/*
/nucleus/styles/*
/nucleus/forgotpassword.html
/nucleus/media.php

それに伴いいくつかのファイルのスタイルシートへのパスを変更した。

Merge branch 'skinnable-master' of sakamocchi@git.sourceforge.jp:/gitroot/nucleus-jp/nucleus-next.git into skinnable-master

ADD: プラグイン「NP_Medium」

メディアファイルを管理するための仕組みであるmedia.phpの代替となるプラグインを追加。
これによりmedia.phpとそれが参照するスタイルシート、JavaScriptをすべてコア機能から取り去ることができる。

プラグインはJavaScriptのmediumオブジェクトにすべてのコードを入れている。プラグイン呼び出しは、デフォルト管理画面用スキンのedit.jsのaddMedia()で行う。addMedia()はmediumオブジェクトが定義されていたらmedium.addMedia()を、そうでなければ新たにウィンドウを開き、nucleusMediaPopupURLをロードする。この条件分岐により下位互換性を維持する。

FIX:スキン変数<%categorysetting%>に統合されていたスキン変数<%category%>の処理を削除

FIX: 通常スキン表示の際、Manager::get()がないという警告で実行が止まるバグ

globalfunctions.phpにおいて、$manager->getSkin()と書くべきところを、$manager->get()と書いていたことによる。

FIX: スキンのコンストラクト時にAdminActionsではなくAdminを渡していたバグの修正

FIX: Skin::update()においてイベント発生条件が不適切だったバグの修正

4.0から新たに以下のイベントが追加されていた。
PreAAddSkinPart/PostAddSkinPart
PreUpdateSkinPart/PostUpdateSkinPart
PreDeleteSkinPart/PostDeleteSkinPart
（管理画面用スキンの場合はSkinをAdminSkinに読み替え）

元々は$contentとTRUEを関係演算子「==」で評価した結果も用いて分岐していたが、これでは常にFALSEとなり、イベントPreUpdateSkinPart/PostUpdateSkinPartが絶対に発生しない。

そのため、empty()関数により評価するように書き換えた。

FIX: Template::update()のINSERT文にINTOを追加

ANSI SQL-92に準拠するために修正。これまで問題なく動作していたのはMySQLの拡張機能と思われる。

FIX: AdminActions::parse_categories()により不要な問い合わせが行われていたのを修正

<%categories%>はページアクション「createitem」「itemedit」でのみ使用可能なことを踏まえてコードを整理した。

FIX: <%editcomment%>タグがパースされる度にPrepareCommentForEditイベントが発生するバグ

AdminActions::parse_editcomment()がPrepareCommentForEditイベントを発生させていたが、これだと<%editcomment%>タグがパースされる度にイベントが発生する。<%editcomment%>タグはcommenteditアクションページに複数含まれるため、イベントが重複して発生してしまう。

そのため、イベントはAdmin::action_commentedit()で発生するようにし、AdminActions::parse_editcomment()にはAdmin::$contentsで配列を渡すように変更した。

FIX: AdminActions::parse_commentnavlist()がSQL文による重複した問い合わせをしていたバグ

listplug_navlist()に$resourceではなく$queryを渡していた。

CHANGE: new SkinではなくManager:getSkin()を用いるよう変更

前回のコミットで追加した、Managerクラスのスキンオブジェクトキャッシュ機構を利用するように修正。

ADD: スキンをキャッシュするためにManager:getSkin()を追加

AdminActionsクラスでスキンオブジェクトを再利用する必要が生じたため。
加えてarray_key_exists()を用いてコードを整理した。