custom hello user block

Hi Draven, thanks for taking the time to share your code.

Your code can be a little cleaner. First of all, you only have to call global $user only once within each function.
Second, the way you insert the image url into the img tag looks a little messy.
Third, I used if ($user->uid) at the beginning to make sure the rest of the code is only run if there is a logged-in user.
Fourth, I wrapped 'Go to your account' in the t() function. It's good practice to do this with all strings that are shown to a user. See http://api.drupal.org/api/function/t/6 for more info.
Last but not least, instead of using <br /> for your lay-out, use paragraphs and css for styling. This will also improve the validity of your HTML.

Personally, I like to define all needed values first and then create the output. So, my version would be:

<?php
  global $user;
  if ($user->uid == 0) { // uid == 0 means user is anonymous
    return;
  }
  $name = $user->name;
  if ($user->picture) {
    $picture = $user->picture;
  }
  else { 
    $picture = 'misc/default_profile_image.jpg';
  }
  $link = l(t('Go to your account'), 'user/'.$user->uid;)
?>

<div id="welcome-user-block">
  <h3>Welcome back <i><?php print $name; ?></i>!</h3>
  <p>
    <img src="/<?php print $picture; ?>" alt="user picture" />
  </p>
  <p>
    <?php print $link; ?>
  </p>
</div>

I hope that helps.

As you all can see above, it's very easy to expand on this code and add all kinds of data. I think it should be mentioned that there is a downside to this. When you read this article: http://2bits.com/articles/free-your-content-php-moving-php-code-out-bloc..., you will learn why it is in fact not ideal to store php code in the database. The alternative is to write a hello user module, which would look something like this (untested code):

<?php
/* implementation of hook_block() */
function hello_user_block($op = 'list', $delta = 0, $edit = array()) {
  switch ($op) {
    case 'list':
      $blocks[0] = array(
        'hello user'       => t('Hello user block'),
      );
      return $blocks;
    case 'view': default:
      if($delta == 0) {
        $block['subject'] = t('Title of block #1');
        $block['content'] = hello_user_block_contents();
      }
      return $block;
  }
}

/* implementation of hook_theme() */
function hello_user_theme() {
  return array(
    'hello_user_block' => array(
      'arguments' => array('vars'),
    ),
  );
}

/* collect data for the block */
function hello_user_block_contents() {
  global $user;
  if($user->uid == 0) {
    return;
  }
  $vars['name'] = $user->name;
  if ($user->picture) {
    $vars['picture'] = $user->picture;
  }
  else {
    $vars['picture'] = 'misc/default_profile_image.jpg';
  }
  $vars['uid'] = $user->uid;
  return theme('hello_user_block', $vars);
}

/* Generate the block's html output in a theme function, to allow theme overrides */
function theme_hello_user_block($vars) {
  $output  = '<div id="welcome-user-block">';
  $output .= '<h3>Welcome back <i>'.$vars['name'].'</i>!</h3>';
  $output .= '<p><img src="'.$vars['picture'].'" alt="user picture" /></p>';
  $output .= '<p>'.l(t('Go to your account'), 'user/'.$vars['uid']).'</p>';
  $output .= '</div>';
  return $output;
}
?>

This code should be put in a file called hello_user.module, in a folder called hello_user. You should also put a simple hello_user.info file in the same folder, see http://drupal.org/node/231036. When uploaded to your modules folder, it will show up on the modules mapge. When the module is enabled, the block will show up on the blocks page.

I got this code to work just fine. It prints the picture, my profile, and logout just like I need it to, however, when I click on logout out it hangs the site and you can't get back in. The screen goes blank except for the header and the only way to get back in is to completely remove the code.

Any ideas what is going on???

Thanks

Here is what I used:

 <?php
  global $user;
  if ($user->uid == 0) { // uid == 0 means user is anonymous
    return;
  }
  $name = $user->name;
  if ($user->picture) {
    $picture = $user->picture;
  }
else {
    $picture = 'misc/default_profile_image.jpg';
  }
  $link = l(t('My Profile'), 'user/'.$user->uid);
 $logout = l(t('Logout'), 'logout');
?>



<div id="loginpic">
  Hey, <?php print $name; ?><img src="/<?php print $picture; ?>" alt="user picture" height="50" align="left" /><br>
  <?php print $link; ?><br>
 <?php print $logout; ?>  
  
  
</div>

Hi people,

I'm a new user of Drupal 7 and I'm interested in this module with the name and the image of the user once connected.

I managed to create the module, with in the file hello_user.module :

/**
* Implements hook_block_info().
*/
function hello_user_block_info() {
  $blocks['hello']['info'] = t('Hello User');
  $blocks['hello']['cache'] = DRUPAL_NO_CACHE;
  $blocks['hello']['properties']['administrative'] = TRUE;
  return $blocks;
}

It seems good in the available blocks (Admin > Structure > Blocks). But how return all the visible elements with the appropriate syntax to Drupal 7 ? It does not work with the higher available code...

Thanks to you. Bye