When creating a new facet, lets enable the default sorts equally to what d7 did for new facets, being (in given order):
Facet active desc
Facet count desc
Display value asc

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StryKaizer created an issue. See original summary.

StryKaizer’s picture

borisson_’s picture

Status: Active » Postponed

Let's postpone this on that other issue.

StryKaizer’s picture

Status: Postponed » Needs work

As #2656668: Facet's results sorting with multiple sorts. got committed, this issue is active again

StryKaizer’s picture

Status: Needs work » Needs review
FileSize
8.43 KB

Status: Needs review » Needs work

The last submitted patch, 5: enable_default_sorts-2712515-5.patch, failed testing.

borisson_’s picture

Status: Needs work » Needs review
FileSize
2.65 KB
9.48 KB

Tried to reproduce the test failure locally and I got a fail on php7 as well - tried to fix that. Also some other changes to clean up coding standards.

Hoping this'll be green, but not sure.

borisson_’s picture

FileSize
539 bytes
9.48 KB

  • StryKaizer committed 524cc45 on 8.x-1.x
    Issue #2712515 by borisson_, StryKaizer: Enable default sorts for new...
StryKaizer’s picture

Status: Needs review » Fixed

rtbc, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.