EC-CUBE / ec-cube

EC-CUBE is the most popular e-commerce solution in Japan
https://www.ec-cube.net
Other
744 stars 650 forks source link

Repository::getQueryBuilderBySearchData() の抽象化 #975

Closed nanasess closed 6 years ago

nanasess commented 9 years ago

検索条件を Chain of responsibility or Strategy でつなげられるようにできないか。 Strategy にすれば、プラグインから検索項目を増やしたい場合などにも有用と思われる。

やたらと長く、単純な条件分岐が続いているため scrutinizer のスコア低下の一因になっている模様。 すっきりと簡潔に記述できるようにしたい。

Yangsin commented 9 years ago

スコアはともかく、プラグインでの拡張の難易度が下げれるのであれば、積極的に採用したいと思います。

chihiro-adachi commented 6 years ago

4.0.0である程度のリファクタリングができている状態のため、クローズします