Add test for Chapters listed in local list on Shareboard when they don't have a sharedcontent field

Nice cleanup.

+++ b/sharedcontent.moduleundefined
@@ -1669,3 +1669,27 @@ function sharedcontent_flag($op, $flag, $content_id, $account, $fcid) {
+ * Get is linkable.

"Returns TRUE if the index is linkable"?

+++ b/sharedcontent.moduleundefined
@@ -1669,3 +1669,27 @@ function sharedcontent_flag($op, $flag, $content_id, $account, $fcid) {
+ * Callback for the calculated is_linkable attribute of sharedcontent_index.

attribute => property

+++ b/sharedcontent.moduleundefined
@@ -1669,3 +1669,27 @@ function sharedcontent_flag($op, $flag, $content_id, $account, $fcid) {
+ * @param $index
+ *  The sharedcontent_index record to calculate the is linkable attribute.
+ * @return bool
+ *   TRUE if the referenced entity can link Shared Content, FALSE otherwise.

empty line between @param and @return.

+++ b/sharedcontent_ui/sharedcontent_ui.testundefined
@@ -0,0 +1,146 @@
+    module_load_include('inc', 'search_api', 'search_api.admin');
+    $form_state = array();
+    $form_state['values']['name'] = $form_state['values']['machine_name'] = 'db';
+    $form_state['values']['class'] = 'search_api_db_service';
+    $form_state['values']['options'] = array('form' => array('min_chars' => 1));
+    drupal_form_submit('search_api_admin_add_server', $form_state);

Seems like directly creating the entity would be easier?

  $form_state['server']  = $server = entity_create('search_api_server', $values);
  $server->configurationFormSubmit($form['options']['form'], $options, $form_state);
  $server->save();

+++ b/sharedcontent_ui/sharedcontent_ui.testundefined
@@ -0,0 +1,146 @@
+    // And one content type is linkable.
+    module_load_include('inc', 'sharedcontent_client', 'includes/sharedcontent_client.admin');
+    $form_state = array();
+    $form_state['linkables']['node']['bundles'][$type_a->type] = $type_a->type;
+    $form_state['values']['node']['bundles'][$type_a->type] = $type_a->type;
+    drupal_form_submit('sharedcontent_client_configure_linkables', $form_state);

pattern-apply-machine! :)

sharedcontent_client_attach_shared_content_field($field_name, $entity_type, $bundle);?

+++ b/sharedcontent_ui/sharedcontent_ui.testundefined
@@ -0,0 +1,146 @@
+    // And five non kinkabke content.

kinkabke? Is that something to eat?

+++ b/sharedcontent_client/sharedcontent_client.drush.incundefined
@@ -85,17 +85,16 @@ function sharedcontent_client_drush_command() {
-  if ($errors = form_get_errors()) {
-    drush_set_error('ERROR', implode(' | ', $errors));

I guess this one here is the reason using the form api makes sense. This allows you to e.g. prevent an attempt to create two servers with the same name.