OpenPNE 3

Overview (概要)¶

SnsTerm の 複雑な使用例 に記載されている下記のようなコードが正しく動作せず、語形変化が適用されていない「friend」のまま出力される。

<?php echo __('%friend% are removed.', array('%friend%' => $op_term['friend']->pluralize()->fronting())); ?>

この問題は #1759 における 9b2f55d8 の修正以降から発生している。

これにより、opDiaryPlugin の日記作成画面を英語で表示した際に「My Friends」と表示されるべき箇所が「my friend」と表示される状態になっている。

Causes (原因)¶

前提として、SnsTerm は以下のような性質を持っている。

1. SnsTerm インスタンスに設定した語形変化は SnsTerm::__toString() が呼ばれると初期状態に戻る

   • $term = Doctrine_Core::getTable('SnsTerm')->get('my_friend');
     $term->titleize();
     
     var_dump((string)$term); // "My Friend" 
     var_dump((string)$term); // "my friend" 
     

2. SnsTerm インスタンスは SnsTermTable クラス内でキャッシュされる

   • $term = Doctrine_Core::getTable('SnsTerm')->get('my_friend');
     $term->titleize();
     
     // $term と同一のインスタンスが返る
     $term2 = Doctrine_Core::getTable('SnsTerm')->get('my_friend');
     var_dump((string)$term2); // "My Friend" 
     

それを踏まえて、opI18N::__() メソッドの動作を追うと以下のようになる。

  // 以下のように呼ばれる。期待する出力は「Friends are removed.」
  echo __('%friend% are removed.', array('%friend%' => $op_term['friend']->pluralize()->fronting()));

....

  // lib/i18n/opI18N.class.php
  public function __($string, $args = array(), $catalogue = 'messages')
  {
    if (empty($this->parsed[$string])) // 初回の呼び出しでは true になる
    {
      $this->parsed[$string] = array();

      $matches = array();
      preg_match_all('/%([a-zA-Z_]+)%/', $string, $matches); // 「%friend%」がヒットする

      array_shift($matches);

      foreach ($matches as $match)
      {
        foreach ($match as $v)
        {
          if ($this->terms[$v])
          {
            $term = $this->terms[$v]; // 性質 2 より、語形変化が適用されたままのインスタンスが返る
            if ($this->titleize) // デフォルトで false
            {
              $term = $term->titleize();
            }

            $this->parsed[$string]['%'.$v.'%'] = (string)$term; // ここで返るのは「Friends」
          }
        }
      }
    }

    $parsedString = $this->parsed[$string];
    if (is_array($args))
    {
      foreach ($args as $k => $v)
      {
        if ($v instanceof SnsTerm)
        {
          $args[$k] = (string)$v; // 性質 1 より、ここで返るのは「friend」
        }
      }
      $parsedString = array_merge($parsedString, $args); // 「Friends」が「friend」に上書きされる
    }

    // 「friend are removed.」が出力される (不具合)
    return parent::__($string, $parsedString, $catalogue);
  }

Way to fix (修正内容)¶

#1759 の修正を一旦取り消し、上記のような SnsTerm の不具合が生じないように #1759 の修正を再度行う。