fix slider so there are no longer duplicate products

[jameshilliard]

This fixes a bug where you get 2 slider entries per product.

[jameshilliard]

It would seem this fix is causing a crash when used with postgres

[JDutil]

Yea you can't use uniq with postgres instead you need to use a DISTINCT clause like:
https://github.com/spree/spree/blob/master/core/app/helpers/spree/taxons_helper.rb#L7

[swcool]

This duplicate display happens in slider, featured and latest. Instead of fixing in views for two files: _slider.html.erb and index.html.erb, I think a better way is to fix in controller within one place. Here is the file to fix

• source
• Fix content

Spree::HomeController.class_eval do

  def index
    uniq_method = "DISTINCT (spree_products.id), spree_products.*, spree_products_taxons.position"

    slider = Spree::Taxon.where(:name => 'Slider').first
    @slider_products = slider.products.active if slider
    @slider_products = @slider_products.select(uniq_method)

    featured = Spree::Taxon.where(:name => 'Featured').first
    @featured_products = featured.products.active if featured
    @featured_products = @featured_products.select(uniq_method)

    latest = Spree::Taxon.where(:name => 'Latest').first
    @latest_products = latest.products.active if latest
    @latest_products = @latest_products.select(uniq_method)

  end

end

If people agree above solution, I think @jameshilliard or myself can submit a new pull request.