Bulk Update messaging lies

Hi Jen! I realize that this is 3 years old but I happened to be in the pathauto issue queue while I update this module at work. I was having an issue with "Bulk Generate" not working so I was trying "Update URL alias" to see if that worked. I was getting the same thing you were, "Updated x number of nodes" but nothing worked. With this patch, I am able to see that the url aliases updated.

I also tested this with having the "Do nothing..." option checked and it returned "Skipped x number of nodes because "Do nothing." is checked" So it seems like this works well. Would be neat to get this into a release.

Thanks. :)

I just wanted to reference this issue here as well. #2770217: Bulk generate does no work in Drupal 7.50. Seems like the "Bulk Generate" feature would benefit from some more descriptive text in it's returned message as well. Just a thought.

+++ b/pathauto.module
@@ -590,13 +590,55 @@ function pathauto_node_update_alias(stdClass $node, $op, array $options = array(
+    if ($updated > 0) {
+      drupal_set_message(format_plural($updated, 'Updated URL alias for 1 node.', 'Updated URL aliases for @count nodes.'));
+    }
+    if ($not_automatic > 0) {
+      drupal_set_message(format_plural($not_automatic, 'Skipped 1 node because its alias was not automaticly generated.', 'Skipped @count nodes because their aliases were not automaticly generated.'));
+    }
+    if ($patternless > 0) {
+      drupal_set_message(format_plural($patternless, 'Skipped 1 node because it did not have an alias pattern.', 'Skipped @count nodes because they did not have an alias pattern.'));
+    }
+    if ($action_do_nothing > 0) {
+      drupal_set_message(format_plural($action_do_nothing, 'Skipped 1 node because update action is set to <em>Do nothing</em>.', 'Skipped @count nodes because update action is set to <em>Do nothing</em>.'));
+    }
   }

I think the message text needs to wrapped in t()

I'll update the patch later tonight, if that's cool? Looks like a nice learning opportunity.

Added the t() around the drupal_set_message message text. This works, but is this what you meant?

Added t() around all of the messages instead of the entire section. Still works this way when testing, but lets see if it fails again on here.