Blacklight Plugin

Add Blacklight.config[:facet][:templates] to handle local overrides to facet views

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Affects Version/s: None
  • Fix Version/s: 2.6
  • Component/s: None
  • Description:
    Hide
    This is a follow-on to #CODEBASE-220, which adds a new facet configuration parameter for controlling the facet partial rendered on a per-facet basis. This might be more contentious (maybe local implementations should override the name helper method and call it good..), but I thought I should offer it anyway.

    The patch would add a configuration parameter to config/initializers/blacklight_config.rb like:
    config[:facet] = {
      :field_names => [ ...],
      :labels => { ... },
      :limits => { ... },
      :templates => {
        "lc_callnum_display" => "catalog/_facet_partials/callnumbers"
      }
    }

    The patch is in http://github.com/cbeer/blacklight/commit/c120d82a4b3e29d2d549d612550ef5c0894e9d4e and attached.

    Show
    This is a follow-on to #CODEBASE-220, which adds a new facet configuration parameter for controlling the facet partial rendered on a per-facet basis. This might be more contentious (maybe local implementations should override the name helper method and call it good..), but I thought I should offer it anyway. The patch would add a configuration parameter to config/initializers/blacklight_config.rb like: config[:facet] = {   :field_names => [ ...],   :labels => { ... },   :limits => { ... },   :templates => {     "lc_callnum_display" => "catalog/_facet_partials/callnumbers"   } } The patch is in http://github.com/cbeer/blacklight/commit/c120d82a4b3e29d2d549d612550ef5c0894e9d4e and attached.
  1. bl-facet-config.patch
    (0.9 kB)
    Chris Beer
    04/May/10 11:29 AM

Activity

Hide
Chris Beer added a comment - 27/Jul/10 1:11 PM
I believe jrochkind's CODEBASE-220 fix makes this irrelevant.
Show
Chris Beer added a comment - 27/Jul/10 1:11 PM I believe jrochkind's CODEBASE-220 fix makes this irrelevant.

People

Dates

  • Created:
    04/May/10 11:29 AM
    Updated:
    27/Jul/10 1:11 PM
    Resolved:
    27/Jul/10 1:11 PM