More code review from @willtp87, @DiegoPino and @whikloj.

Islandora · Sep 14, 2015 · 79d2f65 · 79d2f65
1 parent efc12a5
commit 79d2f65
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/islandora_web_archive.install b/islandora_web_archive.install
@@ -31,12 +31,15 @@ function islandora_web_archive_uninstall() {
 
 /**
  * Print and log a message about possible lost physicalDescription data.
+ *
+ * @see https://jira.duraspace.org/browse/ISLANDORA-1422
  */
 function islandora_web_archive_update_7100(&$sandbox) {
-  drupal_set_message(filter_xss(t('The origin default form had incorrect XPath for the titleInfo block for relatedItem type="host" block. Existing mods:titleInfo fields will exist at the top level instead of nested in the relatedItem type="host" block. Please consult the !url for further information.',
+  $t = get_t();
+  watchdog('islandora_web_archive', 'The origin default form had incorrect XPath for the titleInfo block for relatedItem type="host" block. Existing mods:titleInfo fields will exist at the top level instead of nested in the relatedItem type="host" block.', array(), WATCHDOG_NOTICE, l(t('ISLANDORA-1422'), 'https://jira.duraspace.org/browse/ISLANDORA-1422', array('external' => TRUE)));
+  return $t('The origin default form had incorrect XPath for the titleInfo block for relatedItem type="host" block. Existing mods:titleInfo fields will exist at the top level instead of nested in the relatedItem type="host" block. Please consult the !url for further information.',
     array(
       '!url' => l(t('ticket'), 'https://jira.duraspace.org/browse/ISLANDORA-1422', array('absolute' => TRUE)),
     )
-  )));
-  watchdog('islandora_web_archive', 'The origin default form had incorrect XPath for the titleInfo block for relatedItem type="host" block. Existing mods:titleInfo fields will exist at the top level instead of nested in the relatedItem type="host" block.', array(), WATCHDOG_NOTICE, l(t('ISLANDORA-1422'), 'https://jira.duraspace.org/browse/ISLANDORA-1422', array('external' => TRUE)));
+  );
 }