In hook_node_insert drupal_goto with alias name not working

Question

In hook_node_insert() i have used drupal_goto(), to go to the newly created node with alias name of it, but id doesn't work.

I have already created url alias with patterns for that content type.

$alias = drupal_get_path_alias('node/'.$node->nid);
drupal_goto($alias);

If i give drupal_goto() with nid it works.

drupal_goto('node/'.$node->nid);

What is the issue with respect to alias name not working or how to get alias of newly created node and get it redirected to view page of it with alias name as url.

Any ideas or suggestions? Thanks in advance.

[Updated code]

This is the code which i am trying

function hook_node_insert($node)
{
    if($node->type == 'product')
    {

        $_SESSION['product_node_nid'] = $node->nid;
        $banner = $_SESSION['banner'];
        $theme = $_SESSION['theme'];
        $params = array();
        $params['node'] = $node;
        $site_email = variable_get('site_mail', '');                
        if(($banner || $theme) || ($banner && $theme)) {
            drupal_goto('payment-approval');
        }
        else {
            //$alias = drupal_get_path_alias('node/'.$node->nid);
            module_load_include('inc', 'pathauto'); // include the inc file for Pathauto module
            pathauto_create_alias('node', 'insert', 'node/'.$node->nid, array('node' => $node), $node->type, $language->language); // Language is optional
            $alias = drupal_get_path_alias('node/'.$node->nid);
            //$alias = drupal_get_path_alias('node/'.$node->nid); print_r($alias); exit();
            drupal_goto($alias);
        }
    }
}

Answer 1

Third variable for module_load_include() function is missing in your code.

You should use code module_load_include('inc', 'pathauto', 'pathauto'); instead of module_load_include('inc', 'pathauto');

There is another method for your problem that you can use hook_form_alter instead of hook_node_insert hook in your module.

hook_node_insert will execute if you will insert node using any other method like importing node using feeds module.

Update Code:

function custom_feature_node_insert($node) {

  $options = array('language' => pathauto_entity_language('node', $node));

  //$alias = drupal_get_path_alias('node/'.$node->nid);
  module_load_include('inc', 'pathauto', 'pathauto'); // include the inc file for Pathauto module

  pathauto_create_alias('node', 'insert', 'node/'.$node->nid, array('node' => $node), $node->type, $options['language']); // Language is optional
  $alias = drupal_get_path_alias('node/'.$node->nid);

  drupal_goto($alias);

}

Answer 2

I checked the drupal_goto and found out that you must provide the path for parameter. You can't use alias for drupal_goto. So i think you can solve your problem like that:

1. While you create node. You will also create the alias for that node. The reference for create the alias for node here. In your code just normally redirect like that

   drupal_goto('node/'.$node->nid);
   

2. After that you will install Global redirect module. This module will have responsibility to redirect your node/nid to your node's alias

[Updated]

Can you try to generate the alias by using the Pathauto. So can you put these code in hook_insert()

function yourmodule_node_insert($node){
  module_load_include('inc', 'pathauto');
  pathauto_cleanstring($node->title);
  $node->path = array('pathauto' => TRUE );
  drupal_goto('node/'.$node->nid);
}

Answer 3

drupal_get_path_alias() expects a full internal Drupal path, like this:

drupal_get_path_alias('node/23');

However, you're only giving it an NID, which is being evaluated like this:

drupal_get_path_alias(23);