In the latest commit, i added proper namespacing to the 2 Realms the module is exposing (I added a new 'Graph' realm so you do not to code to make this module useful, in combination with the same facet, shown as a list)

The adding let me to realize, i needed to add the module name as a namespace, basically rendering the Data Combo realms that all 19 of you are running obsolete.

If you find your Data Combo graph realms empty, either edit your custom features manually, or reconfigure the Data Combo Realm.

The name change is from 'graphs_data_combo' to 'facetapi_graphs_graphs_data_combo'

Comments

batje’s picture

batje commented
Assigned: Unassigned » batje
Status: Needs review » Needs work

the new realm does not show any widget.

batje’s picture

batje commented
Status: Needs work » Needs review

Graphs can now only be added to realms that support graphs. The key is the 'element type'. The realm needs to be implemented like this:

$realms['facetapi_graphs_graphs'] = array(
      'label' => t('Graphs'),
      'weight' => 90,
      'sortable' => FALSE,
      'default widget' => 'facetapi_graphs',
      'element type' => 'graphs',
      'settings callback' => '', // not yet implemented sample is: facetapi_block_realm_settings (though that function is not implemented anywhere yet)
      'description' => t(
          'You may use the <em>Graphs</em> realm to display facets as graphs.'
      ),
  );

The widget sets this dependency like this:

    'facetapi_graphs' => array(
      'handler' => array(
        'label' => t('Graphs'),
        'class' => 'FacetAPIGraphsWidgetGraphs',
        'requirements' => array(
          'facetapi_requirement_realm_property' => array('element type' => 'graphs')
        ),
        'query types' => array(
          'term',
          'date',
        ),
      ),
    ),

A default realm (the above one) called Graphs is included.

batje’s picture

batje commented
Status: Needs review » Fixed

this seems to work in several of our installs

Status: Fixed » Closed (fixed)

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