The Facepile plugin uses the "max_rows" attribute , where the attribute called is "numrows" as seen in Facepile documentation: https://developers.facebook.com/docs/reference/plugins/facepile/

This can be fixed by updating by applying the following updates to /fb_social/plugins/fb_plugin/facepile.inc:

From:

function facepile_defaults() {
  return array(
      'href' => '', 
      'size' => 'small', 
      'width' => 'button_count', 
      'numrows' => 1, 
      'width' => 200, 
      'colorscheme' => 'light' 
  );
}

To:

function facepile_defaults() {
  return array(
      'href' => '', 
      'size' => 'small', 
      'width' => 'button_count', 
      'max_row' => 1, 
      'width' => 200, 
      'colorscheme' => 'light' 
  );
}

and also


From:

  $form['numrows'] = array(
      '#type' => 'textfield', 
      '#title' => t('Num rows'), 
      '#description' => t('The maximum number of rows of profile pictures to show') 
  );

To:

  $form['max_rows'] = array(
      '#type' => 'textfield', 
      '#title' => t('Num rows'), 
      '#description' => t('The maximum number of rows of profile pictures to show') 
  );
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wjackson’s picture

The issue persisted against a fresh install of Drupal with the latest release of the module, so I went ahead and created a patch.

To apply this patch download it into your modules folder, and test with the following command from the CLI:

Test

$ patch -p0 --dry-run --verbose < fb_social-facepile-max-rows-1663734-1.patch

If you see "Hunk #1 succeeded at 1." then you can use the next command to apply the patch.

Apply

$ patch -p0 < fb_social-facepile-max-rows-1663734-1.patch

Note: When this patch is applied, the number of rows specified in your Facepile presets will default back to 1. Before seeing the changes you will need to reconfigure your row setting to display multiple rows.

wjackson’s picture

Assigned: wjackson » Unassigned