OpenPNE 3

このテストを行っている際に、「ユーザー画面で大カテゴリのソート順を考慮して小カテゴリを表示していない」問題が見つかりました。
この件については #1859 で対応を行います。

以下のテストを行っています。

更新履歴 9b81fb9edc73633f112d7e5803d46fa6be0ca583 で適用されました。

メモ¶

• lft が 1 のときにそのノードがルートであるか？

根拠は Symfony の NestedSet でそう定義されているということで問題なし．（）

lib/model/doctrine/CommunityCategoryTable.class.php

 25   public function getAllRootsQuery($sort = true)
 26   {
 27     $q = $this->createQuery()->where('lft = 1');
 28     if ($sort)
 29     {
 30       $q->orderBy('sort_order');
 31     }
 32 
 33     return $q;
 34   }

lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Node/NestedSet.php

 866     /**
 867      * determines if node is root
 868      *
 869      * @return bool            
 870      */
 871     public function isRoot()
 872     {
 873         return ($this->getLeftValue() == 1);
 874     }

• 子供とする根拠がNestedSetと違う

Symfony の NestedSet と同様の実装をしたほうが良い．

lib/model/doctrine/CommunityCategoryTable.class.php

 41   public function getAllChildrenQuery($sort = true)
 42   {
 43     $q = $this->createQuery()->where('lft > 1');
 44     if ($sort)
 45     {
 46       $q->orderBy('sort_order');
 47     }
 48 
 49     return $q;
 50   }

lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/Doctrine/Node/NestedSet.php

  56     /**
  57      * test if node has children
  58      *
  59      * @return bool            
  60      */
  61     public function hasChildren()
  62     {
  63         return (($this->getRightValue() - $this->getLeftValue()) > 1);
  64     }

そのたまだ色々ありそう

ドラッグアンドドロップで移動できるようにはなったが，保存されていないように見える．
追記: 手元の環境でソースを変更していただけでした．問題無いです．

note-14 の内容は #1859 での指摘内容だったため，本チケットは問題無いとしてテスト待ちに変更します．