Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Pretty much a clone of this issue #1669918: Allow Panels to properly use Facet API blocks but for Current Search blocks
Currently, you cannot place a Facet API block above the actual search results in Panels.
Also, the blocks are listed under "Miscellaneous".This, in conjunction with #1669908: hook_ctools_content_subtype_alter() does not run early enough, fixes this.
Comment | File | Size | Author |
---|---|---|---|
#6 | facetapi-1710822-4.patch | 807 bytes | dawehner |
#4 | facetapi-1710822-4.patch | 789 bytes | dawehner |
#1 | facetapi_current_search-render-last.patch | 749 bytes | nbucknor |
Comments
Comment #1
nbucknor CreditAttribution: nbucknor commentedComment #3
alanom CreditAttribution: alanom commentedNot sure if this is within the intended scope of this patch, but it didn't solve the problem for me setting panels in ctools Page Manager. Steps taken:
Is the intention to fix this across the ctools/panels family, or just in setups made purely in the Panels module?
Comment #4
dawehnerIt worked for me without any problems once the patch got applied. Great idea to render it later!
Rerolled the patch so it should apply for the testbot.
Comment #5
cpliakas CreditAttribution: cpliakas commentedThis looks good. Marking as needs work, only because "category" should probably be
t('Current Search blocks')
unless there is a good argument for it to be names otherwise. Once the change is made (or my suggestion is proven to be a bad one), I will be happy to roll the patch into Facet API!Thanks for the contribution!
Chris
Comment #6
dawehnerWhen I first added the current search block in the panel I looked for facetapi (no special reason, just a first guess).
Everything is better then putting it into the miscell. category :)
Comment #7
cpliakas CreditAttribution: cpliakas commentedComment #8
cpliakas CreditAttribution: cpliakas commentedComment #9
jaydub CreditAttribution: jaydub commentedRan into this issue as well, patch works great!
Comment #10
paranojik CreditAttribution: paranojik commentedWorks for me also. Thanks guys!
Comment #11
gmclelland CreditAttribution: gmclelland commentedThe patch in #6 worked for me. Thank you!
Comment #12
cpliakas CreditAttribution: cpliakas commentedLooks good. Thanks for the patch and to all the testers! Committed to all branches of Facet API:
dawehner, congrats on becoming the 37th code contributor to Facet API!