hook_node_type_insert has no sample function body

I am open to update this & provide the patch

Hi @Sree,

Thanks for taking this on. Are you still working on this issue? If not, we'll unassign it in a day or two so that someone else can give it a try. (Feel free to assign it back to yourself if you'd still like to work on it, as well.) Thanks!

Hi there,

I came across this issue, and thought i'd supply a basic batch. I added a simple function body to illustrate the hook.
I hope it's sufficient like this :)

Patch looks good, it's a simple example and it does not really need any extra comments since the output message explain it.

I would say it's ok.

+++ b/core/modules/node/node.api.phpundefined
@@ -984,6 +984,9 @@ function hook_ranking() {
+    drupal_set_message('Changed the content type from %old-type to %type.', array('%old-type' => $info->old_type, '%type' => $info->type));

Shouldn't this be in t()?

-22 days to next Drupal core point release.

I need to pay more attention :-p

It is a bit confusing that the machine name is $type but I'm okay with this example.

Committed to 7.x and 8.x. Thanks!

It seems like the patch included on both 7.x and 8.x was the one on comment 8(not the one on comment 12).

Was it a mistake? If so, please re-close again.

This was on purpose. Sorry for not being more clear.

So what was the reasoning for using #8 instead of #12?