During The Prime Hacking Hours

After activating SimpleTagging plugin I faced some problem. To avoid that I disabled the Gengo’s feature `Gengo should append language codes to permalinks automatically.’.

Days later I noticed a WordPress’ strange behavior.

I have pages having translations and when I clicked a link to the another language’s page then I saw a `404 Error’ page.

That’s strange.

Went further investigation I found a bad smelling code in the `locale’ filter.

Index: gengo.php
===================================================================
--- gengo.php   (revision 80)
+++ gengo.php   (working copy)
@@ -1222,7 +1222,7 @@
  } else {
    $value = $parser->query_vars['pagename'];
    if (($position = strpos($parser->query_vars['pagename'], '/')) !== false) {
-     $value = substr($value, $position + 1);
+     $value = substr($value, 0, $position);
    }
    $where = "p.post_name = '$value'";
    $join = "INNER JOIN $wpdb->posts AS p ON p.ID = p2l.post_id";

One of the pair pages I faced 404 error have 3 depth hierarchy.

software/wordpress-plugins/l10n-helper.html    <English>
software_ja/wordpress-plugins/l10n-helper.html <Japanese>

Before modification the value of $value was set as:

wordpress-plugins/l10n-helper.html <English>
wordpress-plugins/l10n-helper.html <Japanese>

$wpdb->posts.post_name only have a post name for 1 depth of hierarchy but the code got of 2 depth. And the code does not seem to care of WordPress feature that it allows to have same post name among pages while the top of ancestors of those page’s are not same.

I fixed the code to obtain the first part of uri.

SimpleTaggingプラグインとの相性の悪さから、Gengo設定の「Gengo should append language codes to permalinks automatically.」をOFFにしているが、この設定におけるGengoの不具合発覚。

l10 helperプラグインは英日２言語を用意しているが、一方のページから他方の翻訳版リンクをクリックすると404エラーが発生。おかしい。

調べてみると、localeフィルタ関数内でページ名からロケールを判定する部分に甘さがあった。

Index: gengo.php
===================================================================
--- gengo.php   (revision 80)
+++ gengo.php   (working copy)
@@ -1222,7 +1222,7 @@
  } else {
    $value = $parser->query_vars['pagename'];
    if (($position = strpos($parser->query_vars['pagename'], '/')) !== false) {
-     $value = substr($value, $position + 1);
+     $value = substr($value, 0, $position);
    }
    $where = "p.post_name = '$value'";
    $join = "INNER JOIN $wpdb->posts AS p ON p.ID = p2l.post_id";

当該ページは英日ともに多階層構造になっていて、各相対リンクはこう：

software/wordpress-plugins/l10n-helper.html    <英>
software_ja/wordpress-plugins/l10n-helper.html <日>

上記変更前のコードで $value に代入される値はこう：

wordpress-plugins/l10n-helper.html <英>
wordpress-plugins/l10n-helper.html <日>

$wpdb->posts.post_nameには１階層分しか登録されてないのに修正前は2階層取得してしまっている。さらに最初の階層(software/software_ja)さえ異なっていれば以降の階層では同一名が使えるようになっているWordPressの仕様も考慮されていない。

ということで、最初の階層部分のみを取り出すようにコードを修正した。

…あ、これはキャッシュと関係なかったか。

どうもGengoはキャッシュと相性が悪い、というかキャッシュを考慮してない様子。

WordPress自体、キャッシュ機能がデフォルトで無効になっている影響なのか、不具合や考慮もれがあちらこちらにありそう。

wp-hackers MLにもポストしたが、ちょこちょことパッチを作っている次第。

そのひとつに、サイドバーのページリストが、あるロケールでは表示されるけどそれ以外のロケールでは表示されない、という不具合がある。

wp_list_pages() にかわるgengo_list_pages()を作って解決を目指す。

<?php
if (!function_exists('gengo_list_pages')) {
function gengo_list_pages($args = '') {
    if (is_array($args)) {
        $r = &$args;
    } else {
        parse_str($args, $r);
    }
    $r = array_merge(array('language' => the_language(true)), $r);
    return wp_list_pages($r);
}
}
?>

/wp-includes/post.php にも不具合があるので、パッチ。

Index: post.php
===================================================================
--- post.php    (revision 3)
+++ post.php    (working copy)
@@ -1070,9 +1070,14 @@
        extract($r, EXTR_SKIP);

        $key = md5( serialize( $r ) );
-       if ( $cache = wp_cache_get( 'get_pages', 'page' ) )
-               if ( isset( $cache[ $key ] ) )
-                       return apply_filters('get_pages', $cache[ $key ], $r );
+       if ( $cache = wp_cache_get( 'get_pages', 'page' ) ) {
+               if ( isset( $cache[ $key ] ) ) {
+                       $pages = apply_filters('get_pages', $cache[ $key ], $r );
+            if ( $pages && ( $child_of || $hierarchical ) )
+                $pages = & get_page_children($child_of, $pages);
+            return $pages;
+        }
+    }

        $inclusions = '';
        if ( !empty($include) ) {

「はてな記法」の実装であるText::Hatenaをxamppに組み込む手順。

• xamppのインストール
• xamppと共に公開されているXAMPP Windows版 アドオン、perl-5.xをインストール
• 環境変数 PATH に、インストールしたPerlのディレクトリを追加
  1. 「コントロールパネル」を開く
  2. 「システム」を選択
  3. 「詳細設定」タブを開く
  4. 「環境変数(N)」を選択
  5. ユーザ環境変数またはシステム環境変数にすでにPATHがあれば選択して「編集(E)」を、なければ「新規作成(N)」
  6. 「変数名(N)」にPATH、変数値(V)にPerlへのパスを追加。「編集（E)」の場合、既存の値の後ろにセミコロン (;) を挟んで追加。
     (例) C:\WINDOWS;c:\xampp\perl\bin
• コマンドプロンプト(cmd.exe)を開く
• cpanを開く。

C:\WINDOWS\SYSTEM32> cpan

• Text::Hatenaのインストール１回目。何か聞かれたらすべて YES で。

cpan> install Text::Hatena

• 順調であれば、「mk_classdataが見つからない」という意味のエラーでインストールが中断する。
  Class::Data::Inheritableをインストール。

cpan> install Class::Data::Inheritable

• Text::Hatenaのインストール2回目。うまくいけばそれで完了。

cpan> install Text::Hatena

Gengoという多言語プラグインを導入。導入マニュアルみたいなんが見つからないので結構時間がかかった。整備したらんとあかんなぁ。