Merge remote-tracking branch 'upstream/7.x' into 7.x

islandora_basic_collection/includes/delete_collection.inc

@@ -113,7 +113,7 @@ function islandora_collection_deletion_form_submit($form, &$form_state) {
  * @param string $pid
  */
 function delete_root_collection($object) {
-
+  $pid = $object->id;
   try {
     $object->repository->purgeObject($object->id);
   } catch (RepositoryException $e) {

islandora_basic_collection/includes/manage_policies.inc

@@ -27,8 +27,9 @@ function islandora_manage_policies_form($form, &$form_state, $collection_object)
         $item = $repository->getObject($content_model);
         $cm_name = $item->Label;
         $cm_options[$content_model] = $cm_name;
+      } catch (RepositoryException $e) {
+        
       }
-      catch(RepositoryException $e) {}
     }
   }
 
@@ -71,36 +72,43 @@ function islandora_manage_policies_form($form, &$form_state, $collection_object)
     '#title' => t("Manage collection policy for @collection_title", array('@collection_title' => $collection_object->label)),
   );
 
-  $form ['manage_collection_policy']['add']['content_model_to_add'] = array(
-    '#title' => "Choose content model",
-    '#type' => 'select',
-    '#options' => $new_options,
-    '#description' => t("Choose content model to add to this collection policy."),
-  );
+  if (count($new_content_models) > 0) {
+    $form ['manage_collection_policy']['add']['content_model_to_add'] = array(
+      '#title' => "Choose content model",
+      '#type' => 'select',
+      '#options' => $new_options,
+      '#description' => t("Choose content model to add to this collection policy."),
+    );
 
-  $form ['manage_collection_policy']['add']['new_cp_namespace'] = array(
-    '#title' => "Choose namespace",
-    '#type' => 'textfield',
-    '#size' => 15,
-    '#default_value' => $namespace,
-    '#description' => t("Choose namespace for objects in this collection associated with this content model"),
-  );
+    $form['manage_collection_policy']['add']['new_cp_namespace'] = array(
+      '#title' => "Choose namespace",
+      '#type' => 'textfield',
+      '#size' => 15,
+      '#default_value' => $namespace,
+      '#description' => t("Choose namespace for objects in this collection associated with this content model"),
+    );
 
-  $form['parent_collection'] = array(
-    '#type' => 'hidden',
-    '#value' => $collection_object->id,
-  );
+    $form['parent_collection'] = array(
+      '#type' => 'hidden',
+      '#value' => $collection_object->id,
+    );
 
-  $form['collection_pid'] = array(
-    '#type' => 'hidden',
-    '#value' => $collection_object->id,
-  );
+    $form['collection_pid'] = array(
+      '#type' => 'hidden',
+      '#value' => $collection_object->id,
+    );
 
-  $form['manage_collection_policy']['add']['submit'] = array(
-    '#type' => 'submit',
-    '#value' => t('Add content model to collection policy'),
-    '#id' => 'add_cm'
-  );
+    $form['manage_collection_policy']['add']['submit'] = array(
+      '#type' => 'submit',
+      '#value' => t('Add content model to collection policy'),
+      '#id' => 'add_cm'
+    );
+  }
+  else {
+    $form['manage_collection_policy']['no_content_models'] = array(
+      '#markup' => t('No content models available to add!'),
+    );
+  }
 
   if (count($current_models_in_policy) > 0) {
     $form['manage_collection_policy']['remove'] = array(