Render alternative content if user is logged in instead of redirecting to user/uid

Question

I have a menu router with type MENU_CALLBACK and it's 'page callback' property points to a function that implements render API, as follow:

 /** Implements hook_menu **/
function custom_render_menu() {
  $items['register-profile'] = array(
    'page callback' => 'custom_render_register_profile',
    'type' => MENU_CALLBACK,
  );
  return $items;
}

/** Implements callback page for register-profile **/
function custom_render_register_profile() {
  $user_register_form = drupal_get_form('user_registration_form');
  $markup = user_is_logged_in() ? t("Alternative Content") : drupal_render($user_register_form);
  $output = array( '#markup' => $markup);
  return $output;
}

If the user is not logged in, the registration form displays correctly, but if the user is logged in, the page was redirected to user/uid router, showing user's profile. How can I display an alternative content instead of redirecting the user to their user page?

Answer 1

How about this:

/** Implements callback page for register-profile **/
function custom_render_register_profile() {
  $markup = user_is_logged_in() ? t("Alternative Content") : drupal_render(drupal_get_form('user_registration_form'));
  $output = array( '#markup' => $markup);
  return $output;
}

Answer 2

Scott's answer helped me on this. For some reason, if you had called drupal_get_form('user_register_form'), even if you assigned it to a variable that isn't being concatenated to '#markup', drupal will still go through the process of checking if the user is logged in or not.

Calling the drupal_get_form('user_register_form') right when assigning the '#markup' does the trick.

function custom_render_register_profile() {
  $output = array( 
    '#markup' => user_is_logged_in() ? t("Alternative Content") : drupal_render(drupal_get_form('user_registration_form')); 
  );
  return $output;
}

edited: The code above will work but will result in a strict warning: only variables should be passed by reference. Use elaborated if statement instead:

function custom_render_register_profile() {
  $output = array();
  if(user_is_logged_in()) {
    $output['#markup'] = t('Alt. Content');
  }else{
    $form_registration = drupal_get_form('user_register_form');
    $output['#markup'] = drupal_render($form_registration);
  }

 $output += array(
  //Add more attributes to your $output
 );
 return $output;
}

As to why the ternary version of the if statement wouldn't work, I'm not sure.

Answer 3

function custom_render_register_profile() {
global $user;
global $base_url;
$uid=$user->uid;
if($uid)
{
drupal_goto('url where you want to take login user..like $base_url./node/1');
}
else
{
  $user_register_form = drupal_get_form('user_registration_form');
  $markup =drupal_render($user_register_form);
  $output = array( '#markup' => $markup);
  return $output;
}
}