作業の合理化、簡便化

36 views
Skip to first unread message

tenpura

unread,
Sep 5, 2011, 2:31:58 AM9/5/11
to WordPress 日本語版作成チーム
定期的に Naoさんよりご意見をいただきますので、第一弾目として以下を提案したいと思います。

1. /wp-admin/includes/schema.php の直接編集で行っていることを、install.php drop-ins で行
う。
2. /wp-admin/includes/dashboard.php hl=en → hl=ja の修正をやめる。理由はどちらの場合も表示が
同じであるように見えるため。

上記に問題がないようなら、リリース毎に直接修正するファイルが二つ減ります。

1用の drop-ins はこちらです。
http://db.tt/98TXUMM

英語版 WordPress の /wp-content/install.php に配置し新規インストールを行うことで動作確認できます。
install.php は新規インストールとアップグレードの際に読み込まれ、新規インストールまたは、マルチサイトの新規サイト追加の時のみ日本語
版用にデフォルト値の変更を行います。(schema.php の編集と同等)

以上、お時間がありましたら動作の確認をお願いいたします。

--
Kuraishi (tenpura)

Mako N

unread,
Sep 5, 2011, 5:38:44 AM9/5/11
to wp-j...@googlegroups.com
From: tenpura <210...@gmail.com>
Subject: [wp-ja-pkg:1669] 作業の合理化、簡便化
Date: Sun, 4 Sep 2011 23:31:58 -0700 (PDT)

> 1. /wp-admin/includes/schema.php の直接編集で行っていることを、install.php drop-ins で行う。

なるほど、こんな仕組みがあるんですね。
動作確認はしていませんが、中身を見てみました。

この仕組みは、たとえばカスタマイズした WP をたくさんインストールすると
か、そんな使い方がされるんでしょうか。
そういうことを考えると、ファイル名は単純な install.php ではなくて、
WordPress 日本語版作成チーム提供、ということがわかるようなものにしたほ
うがいいように思います。

> 2. /wp-admin/includes/dashboard.php hl=en → hl=ja の修正をやめる。理由はどちらの場合も表示が
> 同じであるように見えるため。

確かに、結果は同じように見えますねえ。
あるいは、言語設定を拾って自動で埋め込むようにこの部分を変更してもらう、
でしょうか。でもあとでユーザーが変更可能な箇所でもあるし、やりすぎですかねえ。

> 上記に問題がないようなら、リリース毎に直接修正するファイルが二つ減ります。

ついでに。

直接翻訳のファイルをもうひとつ減らせる、と
http://core.trac.wordpress.org/ticket/17815
を提出したのですが、3.2直前で時期が悪かったのか、放置されています。
これもどなたか見てもらって、後押ししてもらえると助かります。

tenpura

unread,
Sep 5, 2011, 8:26:15 AM9/5/11
to wp-j...@googlegroups.com
Mako さん、ありがとうございます。

> この仕組みは、たとえばカスタマイズした WP をたくさんインストールすると
> か、そんな使い方がされるんでしょうか。

そうですね。初期オプションを変更してインストールするやり方のサンプルと言えると思います。

> そういうことを考えると、ファイル名は単純な install.php ではなくて、
> WordPress 日本語版作成チーム提供、ということがわかるようなものにしたほ
> うがいいように思います。

残念ながら install.php という名前の場合しかこの挙動にはならないのです。
このファイルについてもう少し詳しく書くと

- シングルインストールの場合はインストール後にこのファイルを削除してしまって構わない。
- マルチサイトの場合は追加サイトにこのファイルにある初期値が必要ないのなら、このファイルを削除してしまっても構わない。
- そもそもこのファイルにある初期値が必要ないのならインストール前にこのファイルを削除してしまっても構わない。
- アップグレード時に既存の /wp-content/install.php が新しい install.php で上書きされることはない。つまり我々が提供する install.php が有効となるのは新規インストールの場合のみであり、既存の install.php を使った運用を妨げることはない。

といった感じになります。

> 直接翻訳のファイルをもうひとつ減らせる、と
> http://core.trac.wordpress.org/ticket/17815
> を提出したのですが、3.2直前で時期が悪かったのか、放置されています。
> これもどなたか見てもらって、後押ししてもらえると助かります。

Awaiting Review という場所ができてからコミッターの方達が怠慢になってしまいましたね。
この件に関しては Nikolay に直接連絡するのが一番早いと思います。

--
Kuraishi (tenpura)

tenpura

unread,
Sep 5, 2011, 11:26:06 PM9/5/11
to wp-j...@googlegroups.com
> 1. /wp-admin/includes/schema.php の直接編集で行っていることを、install.php drop-ins で行
う。

自分で言い出してあれなのですが、長期的に見た場合に万が一でもコンパチビリティの問題が出ると厄介なので、こちらの案は取り下げたいと思います。やはり schema.php 方式が無難なようです。

ということで

> 2. /wp-admin/includes/dashboard.php hl=en → hl=ja の修正をやめる。理由はどちらの場合も表示が
同じであるように見えるため。

については引き続きご意見をお待ちしております。特に急ぎません。

--
Kuraishi (tenpura)

Naoko McCracken

unread,
Sep 7, 2011, 9:08:21 AM9/7/11
to wp-j...@googlegroups.com
お返事遅くなりすみません。
tenpura さん、Mako さん、どうもありがとうございます!

>> 2. /wp-admin/includes/dashboard.php hl=en → hl=ja の修正をやめる。理由はどちらの場合も表示が
> 同じであるように見えるため。

ふむ、チェックしてみたら最新版ではちょっとコードが変わっていましたね。hl= の記述はなくなった模様。
http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/dashboard.php#L64
http://core.trac.wordpress.org/browser/branches/3.2/wp-admin/includes/dashboard.php#L64

関連チケット:
http://core.trac.wordpress.org/ticket/17607

リファラを取ってきている Google のブログ検索 API もこれから変更になるのかもしれないので様子見が必要ですね。

Mako さんのチケットはニコライにアサインしてみました。
http://core.trac.wordpress.org/ticket/17815#comment:1

--
Nao

tenpura

unread,
Sep 7, 2011, 8:53:30 PM9/7/11
to wp-j...@googlegroups.com
Naoさん、ありがとうございます。

> ふむ、チェックしてみたら最新版ではちょっとコードが変わっていましたね。hl= の記述はなくなった模様。
> http://core.trac.wordpress.org/browser/trunk/wp-admin/includes/dashboard.php#L64
> http://core.trac.wordpress.org/browser/branches/3.2/wp-admin/includes/dashboard.php#L64

よくみたら hl=en は2年も前に廃止されていました。^^; 我々はずっと亡霊と戦っていたようです。

> Removed hard-coded home language of English from link URL and feed URL. Results are pulled from all languages, and home language set by Google when visit occurs.

http://core.trac.wordpress.org/changeset/12026
http://core.trac.wordpress.org/ticket/10685

> Mako さんのチケットはニコライにアサインしてみました。
> http://core.trac.wordpress.org/ticket/17815#comment:1

Milestone 3.3 になりましたね。setup-config.php も i18n 化されるみたいなので、一気に3つ直翻訳ファイルが減りそうです!

--
Kuraishi (tenpura)

Mako N

unread,
Sep 19, 2011, 8:27:32 PM9/19/11
to wp-j...@googlegroups.com
From: tenpura <210...@gmail.com>
Subject: Re: [wp-ja-pkg:1674] 作業の合理化、簡便化
Date: Thu, 08 Sep 2011 09:53:30 +0900

>> Mako さんのチケットはニコライにアサインしてみました。
>> http://core.trac.wordpress.org/ticket/17815#comment:1
>
> Milestone 3.3 になりましたね。

fixed になりました。とりあえずこれで wp-include/functions.php は直接翻
訳対象からはずせます。

ありがとうございました。

Odyssey

unread,
Sep 20, 2011, 11:38:52 AM9/20/11
to wp-j...@googlegroups.com
fixed になりました。とりあえずこれで wp-include/functions.php は直接翻
訳対象からはずせます。

チケットを読む限り
3.3 からはノータッチ、3.2.x までは引き続きメンテ、という理解でよいでしょうか?

(ともかく…excellent!!)

2011/9/20 Mako N <ma...@pasero.net>

Reply all
Reply to author
Forward
0 new messages