管理画面からインストールされる場合

管理画面からインストールされる場合の処理を見ると、プラグインのインストール手順は以下のような感じ。

1. プラグイン名のチェック(正しいファイル名かどうか・同名のプラグインがすでにインストールされているかどうか)
2. 新しいプラグインの順序(porder)の決定
3. PreAddPluginイベント
4. pluginテーブルに書き込み、プラグインIDをゲット
5. managerが持つインストール済みプラグインのキャッシュを削除
6. プラグインのインスタンスをmanagerから取得してみる(ファイルを読むのはここ・新しいプラグインのinit()が呼ばれている)
7. 要求されるNucleusのバージョンやパッチ・レベルの確認
8. 依存しているプラグインの確認
9. プラグインのinstall()メソッドを呼ぶ
10. PostAddPluginイベント
11. イベントデータの更新 (その後、一覧画面にリダイレクト)

Nucleusのバージョン確認や依存するプラグインの確認でインストール不可と判断された場合(ファイルの読み込みには成功している場合)、ADMINクラスのインスタンスのdeleteOnePlugin()が呼ばれ、その後同じくADMINクラスのインスタンスのerror()でエラー・メッセージを表示している。

プラグインから自身のインストールをキャンセルするには、install()メソッド中でそれを真似すればよさそう。ADMINクラスのインスタンスはグローバル変数の$adminなので、単純に

  function install() {
    global $admin;
    $admin->deleteOnePlugin($this->getID());
    $admin->error('You is a big fool man.'); // エラーメッセージ
  }

で多分大丈夫。

インストールスクリプトからインストールされる場合

ADMINクラスのインスタンスが使われないため、別の実装が必要。……というか、きちんと対応するのは無理くさい。pluginテーブルから情報を消すので精一杯。

COMMENT::getComment($commentId)を実行したときの返り値は、コメントが存在しない場合はfalse、存在する場合は連想配列になる。

イベントPrePluginOptionsEditについて、それが起こされる状況によって$dataの内容が違う。

イベントPrePluginOptionsEditがNucleusコアで起こるのは以下の2つの場合。

• プラグイン管理画面から遷移するプラグインオプションの編集画面を表示するとき(プラグインオプションのコンテキストはglobal)
• ブログやメンバー、アイテム等の管理・編集画面でプラグインオプションの編集フォームを表示する直前(コンテキストはglobal以外)

以下に違いのまとめ。

せっかくFx3からjavascript1.8が使えるようになったのに、Firebug(このメモを書いている現在、最新版は1.2.1)のコンソールでは有効でないので、使えるようにする。

"(プロファイルのディレクトリ)\extensions\firebug@software.joehewitt.com\content\firebug\libs.js"の260行目付近、addScriptという名前のメソッド内以下のコードを置き換える。

element.setAttribute("type", "text/javascript");

これを

element.setAttribute("type", "application/javascript;version=1.8");

何か危険かも。

JSDeferredについては、なんか上手いまとめ方が思いつかない。

83行目

ここまで全部コメント。

function Deferred () { return (this instanceof Deferred) ? this.init(this) : new Deferred() }

関数Deferredの定義。クラスDeferredのコンストラクタ。初期化処理はDeferred.prototype.initにある。

コメントにはDeferred();はnew Deferred();のショートハンドである、的なことが書いてある。Deferred();を実行すると、(this instanceof Deferred)の値はfalseになり、三項演算子の後のほうが実行され、結果的にnew Deferred();の結果が返ってくる。

JavaScript の new 演算子の意味: Days on the Moonを参考にして、無名関数・引数ありの場合で書き直すとこんな感じ?

function() {
  if (this instanceof arguments.callee) {
    return this.init.apply(this, arguments);
  } else {
    var Constructor = function() {};
    Constructor.prototype = arguments.callee.prototype;
    var instance = new Constructor();
    var result = arguments.callee.apply(instance, arguments);
    return (result instanceof Object) ? result : instance;
  }
}

なんかすっごい長くなった。

84行目〜125行目

Deferred.prototypeの定義。

85行目〜92行目

初期化処理を行うDeferred.prototype.initの定義。

	init : function () {
		this._next    = null;
		this.callback = {
			ok: function (x) { return x },
			ng: function (x) { throw  x }
		};
		return this;
	},

94, 95行目, 104行目〜108行目

	next  : function (fun) { return this._post("ok", fun) },
	error : function (fun) { return this._post("ng", fun) },

nextメソッドとerrorメソッドの定義。細かいことは_postメソッドにある。

ちょっと飛ばして、その_postメソッドについて。

	_post : function (okng, fun) {
		this._next =  new Deferred();
		this._next.callback[okng] = fun;
		return this._next;
	},

これは見たまま。this._nextの値はDeferredクラスの新しいインスタンス、で、それが返ってくる。

新しいインスタンスが返ってくるので、以下のようにつなげられる。

var testDeferred = new Deferred();
testDeferred.
next(function() { /* ... */ }).
next(function() { /* ... */ });

95, 96行目, 110行目〜124行目

	call  : function (val) { return this._fire("ok", val) },
	fail  : function (err) { return this._fire("ng", err) },

	_fire : function (okng, value) {
		var self = this, next = "ok";
		try {
			value = self.callback[okng].call(self, value);
		} catch (e) {
			next  = "ng";
			value = e;
		}
		if (value instanceof Deferred) {
			value._next = self._next;
		} else {
			if (self._next) self._next._fire(next, value);
		}
		return this;
	}

作ったチェーンを開始するメソッドcall、あるいはエラーを処理するメソッドfail、そしてその中身。普通に使う分にはあまり意識しなくていいっぽい。

_fireメソッドは内側でself.callback[okng]を実行している。okngは文字列'ok'または'ng'。実行したとき何かthrowされているとself._next.callback.ngが、そうでないならself._next.callback.okがself._next._fire経由で実行される。

初期化直後だとself.callback['ok']は第1引数をそのまま返す関数で、self.callback['ng']は第1引数をそのままthrowする関数になっている。なので、以下の2つは結果的に同じ動きになりそう。

var testDeferred = new Deferred();
testDeferred.next(function(){ alert('asdf'); });
testDeferred.call();

var testDeferred = new Deferred();
testDeferred._next = new Deferred();
testDeferred._next.next(function(){ alert('asdf'); });
testDeferred.call();

self.callback[okng]の実行でDeferredクラスのインスタンスが返されたりthrowされたりすると、self._next._fireは実行されず、そのインスタンスが返される。ただし、そのインスタンスは_nextとしてself._nextの値を持つ。_fireメソッドは、この返されたインスタンスの_fireメソッドを実行する機能を持たない。

var testDeferred = new Deferred();
testDeferred.next(function() {
  alert('asdf');
  var subDeferred = new Deferred();
  return subDeferred.next(function() { alert('fdsa'); });
}).next(function() {
  alert('asdffdsa');
});
testDeferred.call();

上のようにするとalert('asdf');は実行されるがalert('fdsa');とalert('asdffdsa');は実行されない。

なんというか、bbs2chreaderのスキンをうだうだしたほうが、結果的に便利なものができるかもしれない。

とりあえず試してみたことをメモ。

SQLを実行する/実行結果を取得する

"(o2on管理画面のURL)xml/sql"に対してPOSTメソッドでSQLを渡してやれば(変数名は"sql")、そのSQLを実行できる。その後"(o2on管理画面のURL)xml/sql"をGETすれば、そのSQLの実行結果が得られる。POSTに対するレスポンスが帰ってきてからでないと結果は期待できない。

例

ママチャリスレの情報を取得

var o2onAdminUrl = 'http://127.0.0.1:9999/';
var oThreadInfo = {
  domain: '2ch.net',
  bbsname: 'news4vip',
  datname: '1164641644.dat'
};
 
var sql = (function(oWhere) {
  var result = 'SELECT * FROM dat', aWhere = [];
  for (var column in oWhere) {
    aWhere.push(column + ' = "' + oWhere[column] + '"');
  }
  result += ' WHERE ' + aWhere.join(' AND ');
  return result;
})(oThreadInfo);
 
GM_xmlhttpRequest({
  method: 'POST',
  url: o2onAdminUrl + 'xml/sql',
  headers: {
    'Content-Type': 'application/x-www-form-urlencoded'
  },
  data: 'sql=' + encodeURIComponent(sql),
  onload: (function() {
    GM_xmlhttpRequest({
      method: 'GET',
      url: o2onAdminUrl + 'xml/sql',
      onload: (function(res) {
        GM_log(res.responseText);
      })
    });
  })
});

検索にスレッドを登録/登録解除/有効化/無効化

"(o2on管理画面のURL)xml/query"にPOST。使用する変数は以下のとおり。

その他

他にも多数の情報を"(o2on管理画面のURL)xml/(なんとかかんとか)"で取得できる。

Be-2ちゃんねる　人柱募集中。。。Part5の924 (現在はdat落ち) にあった、1つのBe番号から識別番号を算出し、さらに識別番号から存在しうるBe番号を列挙するperlのスクリプトは以下。perlはわからないけど、インデントはしてみた。

#! /usr/bin/perl
 
my $be = shift
  or die "usage: $0 BE_NUMBER\n";
my $x = int($be/10) % 10;
my $y = $be % 10;
die "$0: illegal be-number: $be\n" if $x < 2 || $x > 9 || $y < 2 || $y > 9;
my $k = $x * $y * 3;
my $n = int($be/100) + $x - $y - 5;
die "$0: illegal be-number: $be\n" if $n % $k != 0;
my $id = $n / $k;
my @list = ();
for my $y (2..9) {
  for my $x (2..9) {
    my $be = ($id * 3 * $x * $y - $x + $y + 5) * 100 + $x*10 + $y;
    push(@list, $be);
  }
}
for my $be (sort {$a<=>$b} @list) {
  print "http://be.2ch.net/test/p.php?i=$be\n";
}

bbs2chreaderで使いたいので、javascriptで識別番号を算出するところを書き出してみた。

function(origBe) {
  var be, x, y, k, n;
  if (typeof origBe == 'undefined') {
    throw new Error('missing be-number');
  }
  be = parseInt(origBe, 10);
  if (!be) {
    throw new Error('illegal be-number: ' + origBe.toString());
  }
  x = Math.floor(be / 10) % 10;
  y = be % 10;
  if (x < 2 || x > 9 || y < 2 || y > 9) {
    throw new Error('illegal be-number: ' + origBe.toString());
  }
  k = x * y * 3;
  n = Math.floor(be / 100) + x - y - 5;
  if (n % k) {
    throw new Error('illegal be-number: ' + origBe.toString());
  }
  return n / k;
}

で、手始めにクイック・メニューをMenuBarにしてみました。 → NP_YUIQuickMenuMenuBar。

あとNP_ResetAdminCSSを更新。書き直しただけで特に差異なし。同梱のCSSは新しくしましたが、Fx 2.0とIE7でしかチェックしてないです。IE6だとデザインが崩れるだろうという確信があります。

Yahoo! UI、面白いです。スキンでまで使う気にはまだなりませんが、管理画面ではいい感じ。

"Yahoo! UIを管理画面で使うとか"で書いたNP_YUIImporterには問題があって上手く動かないことが発覚。

例えば

containerコンポーネントでパネルを作るとき、ドラッグ・ドロップで移動できるようにする機能を持たせたい場合、containerコンポーネントより前にdragdropコンポーネントを読み込ませる必要があります。もちろん、移動が不要な場合はdragdropコンポーネントも不要です。

現バージョンのNP_YUIImporterを使って、

1. dragdropなしで、containerを使うプラグイン
2. dragdropも、containerも使うプラグイン

の順でNP_YUIImporterのimportメソッドが呼ばれると、dragdropコンポーネントの読み込みはcontainerコンポーネントの読み込みより後になってしまうので、パネルの移動は不可能になってしまいます。

結局

で、これを解消するためには、すぐにscriptタグを出力せず、どのYUIコンポーネントを読み込むかがすべて出揃ってから、何らかの形で整えられた順序でscriptタグを出力するようにしないといけないわけで、どう考えてもNP_YUIImporterの仕様変更は必須。めんどくさいなあ。

読み込み順の制御方法・案

YUI Loader Utilityを使う

YUI側で用意されているYUI Loader Utility ( Yahoo! UI Library: YUI Loader Utility ) を使う。処理を丸投げできるのでNP_YUIImporter側では簡単、に見えてonSuccessイベントの処理とか面倒。

固定する

あらかじめ、なるべくトラブルがおきない読み込み順を調べて、ハードコーディングしてしまう。後はそれにそって各コンポーネントを読み込む。Loader程じゃないけどちょっと気持ち悪いかも。

配列で受け取ってソートする

各プラグインごとに、求められる読み込み順を配列として受け取って、なるべく矛盾しないように並べ替える。上手く行くかわからない。

Yahoo! UI ( The Yahoo! User Interface Library (YUI) ) の2.5.0が出てたので格闘中。手も足も出ません。

副産物

で、とりあえずYUIのライブラリを重複しないように読み込むためのプラグイン、NP_YUIImporter。

NP_YUIImporter v0.0.0このバージョンは最新のものではありません。 "NP_YUIImporter"を参照してください。

使い方はこんな感じ。

  function getPluginDep() {
    return array('<a class="plug_nkautolinkbyitemtitle" title="Read item : NP_YUIImporter" href="http://niku.suku.name/item/602">NP_YUIImporter</a>');
  }
 
  function event_AdminPrePageHead($data) {
    $manager =& MANAGER::instance();
    $objYUIImporter =& $manager->getPlugin('<a class="plug_nkautolinkbyitemtitle" title="Read item : NP_YUIImporter" href="http://niku.suku.name/item/602">NP_YUIImporter</a>');
    $data['extrahead'] .= $objYUIImporter->import('assets/skins/sam/skin.css');
 
    $data['extrahead'] .= $objYUIImporter->import('yahoo/yahoo-min.js');
    $data['extrahead'] .= $objYUIImporter->import('dom/dom-min.js');
  }

importメソッドはライブラリを読み込むためのXHTMLタグを返します。拡張子が.cssならlink要素、そうでないならscript要素。ただしタグが返るのはライブラリごとに最初の1回のみで、2回目以降は空文字列が返されるという、至極単純な仕組みです。

YUIはYahoo!でホスティングもされてるので、このプラグインを通すことで同じサーバのライブラリを使うか、ホスティングされているものを使うか切り替えることができます。

オチ

Rich Text Editorでアイテムを編集するプラグインに取り掛かってみましたが、<%image%>などの処理でいい方法が浮かばず断念。あとRich Text EditorのCSSとNucleus管理画面のCSSが衝突してデザインが狂うのはどうにも。既存のモノに付け足して使うようなライブラリではない、ということなんでしょうかね。それとも何か上手い方法があるんでしょうか。

オプションの設定が気持ち悪くなってたので変な方法を使って修正してます。将来的な仕様変更には耐えられそうにないです。相性問題はそのまま。

» NP_PHPSpeedy。

というかこうなりました→ NP_Compressor v0.0.0 (0.3.1対応版) 。まだ叩き台なので専用ページは作りません。名前を変えて専用ページを作りました。→ NP_PHPSpeedy

compressorディレクトリ内に上のリンクから落としてきたclass.compressor.phpとlibsディレクトリを突っ込むことで動くのではないかと。compressorライブラリの仕様で、少なくともPHP4.3以上でないと動かないはず(もっと厳しい条件があるかも)。あと"もう一度メモ 出力バッファリングプラグインをどうするか"で書いたような出力バッファリングに関するプラグイン同士の相性問題があります。

オプションが複雑ですが、そこはカンでなんとか。

内部的な問題

v0.3では一箇所引っかかったところがあって、HTMLをgzip圧縮する場合、コンストラクタでオプションを設定する部分、pageに関する設定を最後にしないとcssとjavascriptの処理が行われません。

/**
 * これならうまくいく
 */
$compressor_a = new compressor('css,javascript,page');
 
/**
 * これはだめ
 */
$compressor_b = new compressor('page,css,javascript');

言語ファイルから読み込まれるのをUTFだと限定して、そこからプラグイン内で変換するとか。具体的に書くと、プラグイン用の言語ファイル(japanese-euc.php)に

<?php include_once('japanese-utf8.php'); ?>

とだけ書いておいて、そのあとmb_convert_encodingする。

素直にUTFから変換して作ることにします。

変数名、大文字小文字の区別

通常のNucleus変数については大文字小文字を区別しない("<%Blog(...)%>"と"<%blog(...)%>"はまったく同じ動作)。プラグインによるNucleus変数には区別がある(NP_Exampleというプラグインのスキン変数は"<%Example%>"であって"<%example%>"だと失敗する)。

パラメータ前後の空白

各パラメータはtrimされた後にプラグインに渡される。なので、"<%Example(test)%>"と"<%Example( test )%>"は同じ動作。

PHP4のサポートが終わった

PHP 4.4.8が出てPHP4は終了?

Nucleus側でいつまでPHP4の対応をするのかは気になりますが、マニュアルにPHP4.0.6以降が必要って書いてあるんだから、それより前のバージョンに対応するためのコードはないほうがすっきりするんじゃないかと思います。parse_textのeval()とか。

ネタ元

NP_Container v0.4 - Nucleusだとか

Nucleus(JP)フォーラム :: トピックを表示 - NP_Container

この記事内での用語については"NP_Containerについて私的メモ"を参照。

v0.3からの変更点

• 定義に使う記法が緩くなった
  • パーツ定義時にコメントが書けるようになった
• <%parsedinclude()%>、NP_IncludeSpecialに対応
  • PreSkinParseイベントでコンテナ定義を行う前に、これらの変数による読み込みを先に実行してしまう(読み込み対象の内容を当該部分に展開する)ことで実現している様子。
• NP_ContainerToTemplateのマージ
• 複数箇所で同一コンテナを定義した場合の不具合解消

こんなカンジでしょうか

オマケ

ちょっと前に作ったプラグイン、NP_ContainerToForm。

→ NP_ContainerToForm v0.0.0。

formsディレクトリにある.templateファイルの代わりにコンテナを使用してフォーム類を表示するプラグイン。いろいろ未テスト。

使用例はこんな。

<%Container(begin,ContainerToForm)%>
<!--PART name="nucleusbutton" Nucleusボタン -->
<img id="nucleusbutton" src="<%formdata(imgurl)"  width="<%formdata(imgwidth)%>" height="<%formdata(imgheight)%>" />
<!--/PART (nucleusbutton)-->
<%Container(end)%>
<%ContainerToForm(nucleusbutton)%>

変数の引数

最初の引数は表示するフォーム類のタイプ。以下のいずれか。末尾の"form"は省略可能。

• additemform
• commentform
• loginform
• membermailform
• nucleusbutton
• searchform

二番目の引数は使用するコンテナ。使用されるパーツ名はformsディレクトリ内の対応する.templateファイルの名前と同一(例 : "commentform-loggedin")。省略時は"ContainerToForm"と名づけられたコンテナを探す。

三番目以降の引数はもともとの変数で使用される引数と同様。最初の引数に"nucleusbutton"を指定した場合、画像URLや縦横のサイズを指定できる。

_CHARSETを定義しているのが言語ファイル内で、その言語ファイルのファイル名に文字セット情報が含まれているという不思議。$CONF['Charset']を作ってしまうのが一番いいと思うけど、英語力がなくて説明できず。

Nucleus標準そのままでは、hAtomを使ってカテゴリーを表現することができません。

ネタ元

Stolen Moments: rel-tag を利用する場合のリンク先について

rel-Tagを使う場合、リンクのURLで何のタグかを示します。仕様書では以下のように決まってます。

タグはURIの中に分かりやすい形で埋め込まれています、そのためURIからタグを機械的に抽出することが比較的容易です。具体的に説明すると、タグはURIパスの最後にある単語（一番後ろの"/"より後の文字列）となります。

なので、例えば"Nucleus"というタグを使いたい場合、リンクのURLは"http://example.com/tags/Nucleus"とか、"http://example.com/category/Nucleus"とかにする必要が。

Nucleus標準ではFancyURLにした場合でもURLに登場するのは数字(カテゴリーID)なのでこのような指定はできません。FancyURLにしなかった場合、カテゴリーに関する情報はクエリ部分に収められることになるので、この場合も無理。

FancyURLの場合はGenerateURLイベントが使えるのでまだ希望はありますが、そうでない場合は正直絶望的かも(カテゴリーの代わりにブログを使うとかならまだしも)。

easyScrapper.user.js。

unsafeWindow使ったりとか、下手すればプレーンテキストでパスワードを保存したりするので、たたき台以外の用途にはオススメできません。あとE4X使ってるのでFirefoxでしか動かない。

使い方は適当にインストール後、Greasemonkeyのメニューから"easyScrapper - Setting"を選択。APIのURL、ブログのID、ユーザー名を入力。他に任意設定の項目で認証に使うパスワード、ショートカットキー、アクセスキーの設定ができる。パスワードを入力しなかった場合投稿時点で毎回入力が必要。一度設定を行っている場合、その設定への変更を反映させるにはリロードが必要。

設定が行われれば"easyScrapper - Post"のメニューができるので、ページの好きな範囲を選択してそのコマンドを使えばブログに情報が送信される。

metaWeblog.newPostを使用。拡張はmt_allow_comments(1固定)のみ。publish設定は公開固定。カテゴリーの選択もできない。

以下メモ。

エラーのときはalert()を使って警告画面を出してます。で、成功したときにも"成功した！"って反応を返すべきなのですが、その方法がいまいち思いつきません。何のエラーも起きてないのにalert()ってのも変。Firefoxはステータスバーの変更がデフォルトで不許可になってたりするし。ただそれだけのためにDHTMLをいじるのも面倒……。

Nucleusは標準で4種類のXML-RPCインターフェイスをサポートしていて、それらは以下のとおり。

• "blogger."で始まるBlogger API
• "metaWeblog."で始まるmetaWeblog API
• "mt."で始まるMovable Type API
• "nucleus."で始まるNucleus API (deprecatedでマニュアルにも書いてない)

で、(すでに非推奨であるNucleus APIは放っておくとして)アイテム追加機能を提供するBlogger APIとmetaWeblog APIについて、アイテムのコメントを許可するか否か(closedでないのか、あるいはclosedなのか)決定する条件が微妙に食い違ってる気が。

Blogger APIはコメントを許可する/許可しないという設定項目を持たない。Nucleusではこれを使ってアイテムを投稿した場合にはコメントは許可されることになる(実際にはブログの設定に依存)。

metaWeblog APIではMovable Typeで実装された拡張"mt_allow_comments"を使ってこの設定ができる。Nucleusではこの値を省略したとき(というか拡張を使用しなかったとき)にアイテムがclosedになってしまう。

APIの仕様的に間違ってるわけではなさそう(未確認)だし、バグというわけでもないんですが、要望の形でFlySprayにでも投げておこうかと思う次第。

変更案

手元の3.3 日本語EUC版のパッチ(未検証)。"mt_allow_comments"が未指定の場合に、closedが指定されなかったとしてアイテムを作成するようにしたつもりのもの。

56c56,61
< 		$comments = (int) _getStructVal($struct, 'mt_allow_comments') ? 0 : 1;
---
> 		$comments = $struct->structmem('mt_allow_comments');
> 		if ($comments) {
> 			$closed = (int) _getStructVal($struct, 'mt_allow_comments') ? 0 : 1;
> 		} else {
> 			$closed = 0;
> 		}
61c66
< 		$res = _addItem($blogid, $username, $password, $title, $content, $more, $publish, $comments, $category);
---
> 		$res = _addItem($blogid, $username, $password, $title, $content, $more, $publish, $closed, $category);

追記

mt_allow_comments = 2 ?

#5366 (XMLRPC interface misinterprets mt_allow_comments value) - WordPress Trac - Trac。

Boolean型じゃなかったのか。

この場合の変更案

上記の参照先に依れば、mt_allow_commentsの値とコメントの表示・投稿の可否についてはこんな感じらしい(?)。

Nucleusでは0と2の状態の差がアイテムに設定できないので、0と2を同様にclosed設定を行う指定だと解釈すべきだと思う。

以下のやつは"mt_allow_commentsが指定されていて、なおかつその値が1以外ならclosed"なパターン、のつもり。上の例に加えてeditPostの変更を加えている。

56c56,61
< 		$comments = (int) _getStructVal($struct, 'mt_allow_comments') ? 0 : 1;
---
> 		$comments = $struct->structmem('mt_allow_comments');
> 		if ($comments) {
> 			$closed = (intval(_getStructVal($struct, 'mt_allow_comments')) == 1) ? 0 : 1;
> 		} else {
> 			$closed = 0;
> 		}
61c66
< 		$res = _addItem($blogid, $username, $password, $title, $content, $more, $publish, $comments, $category);
---
> 		$res = _addItem($blogid, $username, $password, $title, $content, $more, $publish, $closed, $category);
190c195
< 			$comments = (int) _getStructVal($struct, 'mt_allow_comments') ? 0 : 1;
---
> 			$closed = (intval(_getStructVal($struct, 'mt_allow_comments')) == 1) ? 0 : 1;
192c197
< 			$comments = $old['closed'];
---
> 			$closed = $old['closed'];
195c200
< 		$res = _edititem($itemid, $username, $password, $catid, $title, $content, $more, $wasdraft, $publish, $comments);
---
> 		$res = _edititem($itemid, $username, $password, $catid, $title, $content, $more, $wasdraft, $publish, $closed);

海外フォーラム参照

Nucleus Support :: View topic - Posting via XML-RPC disables comments。

簡単な直し方。closedにしない人に。