Broken the Data Combo Realm! [#2028529]

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

Comment #1

batje CreditAttribution: batje commented 27 June 2013 at 15:18

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

the new realm does not show any widget.

Comment #2

batje CreditAttribution: batje commented 30 June 2013 at 12:10

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.