Panopoly users missing default 'body_classes' [#2193399]

Panopoly users is used within Open Atrium and oa_panopoly_users tries to override (features override) page_manager_handlers.pm_existing_pages_user_login_panel_context.conf|body_classes_to_add and because it doesn't exist in panopoly_users, panopoly_users tries to add it to its page_manager_handlers. It seems if the defaults of: 'body_classes_to_add' => '', and 'body_classes_to_remove' => '', are added to panopoly_users then the problem is resolved, does this seem right? I will upload the patch just in case.

Comment	File	Size	Author
#1	panopoly_users-body_classes-2193399-1.patch	882 bytes	scottalan

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

scottalan CreditAttribution: scottalan commented 11 February 2014 at 07:09

File	Size
panopoly_users-body_classes-2193399-1.patch	882 bytes

Comment #2

scottalan CreditAttribution: scottalan commented 11 February 2014 at 07:22

Ok, I may have just learned something about `defaultconfig`. When I patched this feature with the patch above, the feature is no longer showing overridden. This is related to https://drupal.org/node/2193381.

This is what drush fd panopoly_users was showing:

Component type: info
  features[panelizer_defaults][] = user:user:default:featured
  features[variable][] = page_manager_user_view_disabled
> features[variable][] = panelizer_defaults_user_user
  features[variable][] = panelizer_user:user_allowed_layouts_default
  features[variable][] = panelizer_user:user_allowed_types_default


Component type: page_manager_handlers
        'panels_breadcrumbs_paths' => '',
        'panels_breadcrumbs_home' => 1,
>       'body_classes_to_add' => 'oa-login',
        'display' => array(
          'args' => array(),
          'title_pane' => 0,
        ),
<       'body_classes_to_add' => 'oa-login',
      ),
    ),

I didn't realize that even though a drush fd [FEATURE] would show it overridden, that drush fd does not.