エクスポート機能のデフォルト値の不具合  【解決済み】

質問全般・改善要望
tadaochi
メンバー
メンバー
記事: 4
登録日時: 2013年7月08日(月) 16:09

エクスポート機能のデフォルト値の不具合

投稿記事by tadaochi » 2013年7月08日(月) 16:30

サイト全体を静的HTMLとしてエクスポートする機能で、不具合らしきものを見つけました。
MODxのバージョンは MODx Evolution 1.0.10J-r1 です。

エクスポートをする際、パラメータとして「文字列を置換(置換前)」と「文字列を置換(置換後)」を
入れますが、せっかく入力しても、この値が次回のエクスポート時には無くなってしまいます。
調べたところ、「文字列を置換(置換前)」と「文字列を置換(置換後)」のデフォルト値を取ってくるところで、
間違ったオプション名を指定しているようでした。

下記のように修正したところ、上記問題が解決出来ました。
manager\actions\export_site.static.php 79行目あたり

変更前:
$repl_before = $modx->getOption('repl_before',$modx->config['site_url']);
$repl_after = $modx->getOption('repl_after',$modx->config['site_url']);

変更後:
$repl_before = $modx->getOption('export_repl_before',$modx->config['site_url']);
$repl_after = $modx->getOption('export_repl_after',$modx->config['site_url']);

ご確認をお願い致します。


※すみません。追記です。

「無視するID(カンマ区切りで複数指定可)」も同様で、デフォルト値の取得が間違っているようでした。
export_site_static.php 72行目あたりを修正して直しました。

修正前:
$ignore_ids = $modx->getOption('ignore_ids');

修正後:
$ignore_ids = $modx->getOption('export_ignore_ids');
アバター
yama
管理人
記事: 3154
登録日時: 2009年7月29日(水) 02:50

Re: エクスポート機能のデフォルト値の不具合  【解決済み】

投稿記事by yama » 2013年7月08日(月) 18:15

ありがとうございます。次リリースで対応します。(今週の金曜日にリリース予定です)
アバター
yama
管理人
記事: 3154
登録日時: 2009年7月29日(水) 02:50

Re: エクスポート機能のデフォルト値の不具合

投稿記事by yama » 2013年7月09日(火) 09:35

もしよければ開発リポジトリにコミットしてみてください。

https://github.com/
上記にアクセスしてアカウントを作成し、

https://github.com/modxcms-jp/evolution ... static.php
上記ページのコード領域右上の「Edit」ボタンをクリックし、改善したコードを貼り付けます。
(「Preview」ボタンをクリックすると差分を目視で確認できて便利です)
「Commit Changes」ボタンをクリックすると、プルリクエストとしてコミット内容が送信されます。あとはこちらで内容を確認してから受理操作を行ないます。
tadaochi
メンバー
メンバー
記事: 4
登録日時: 2013年7月08日(月) 16:09

Re: エクスポート機能のデフォルト値の不具合

投稿記事by tadaochi » 2013年7月15日(月) 00:15

yama様

githubの件、今になって気づきました。すみません。
リリース版への反映ありがとうございました。

※githubについては、あまりよく理解できていないのですが、今後のために勉強しておきます。f(^^;;;