Update logic if anonymous comments' e-mail is not available

Sorry, I should clarify I am testing with MonsterId against anonymous user comments. I can see that registered users get unique gravatars, or a gravatar associated with their email address.

Ah, yes you are right! But I took a look at the code and see that from the $variables array you can extract 'hostname' which is the commenter's IP address and I think this could be used as an alternate fallback hash seed for anonymous users who do not provide an email address.

If you could tell me how to get 'hostname' from that array I could implement it myself. So far I have tried

$variables['account']->hostname;

But this is always empty, which is odd because $variables['account']->mail; supposedly carries the email address string from the same array.

My php is really rusty so I might be wrong, but when I put a print_r($variables); in gravatar.module gravatar_preprocess_user_picture() I get the array spitting out and including 'hostname' with the correct value. I am probably wrong but I just assumed if I can echo out that full array from the gravatar.module it must have access to the full array.

OK, it's probably no big deal to most but this is how I got round this, it might help others who want individual gravatars for anons who don't want to leave any email address:

Find:

      // Load mail/homepage variable from an anonymous comment.

      if (empty($account->mail) && !empty($variables['account']->mail)) {
        $account->mail = $variables['account']->mail;
      } 
      if (empty($account->homepage) && !empty($variables['account']->homepage)) {
        $account->homepage = $variables['account']->homepage;
      }

And replace it with

      // Load mail/homepage variable from an anonymous comment.

      if (empty($account->mail) && !empty($variables['account']->mail)) {
        $account->mail = $variables['account']->mail;
      } elseif (empty($account->mail)){
      	 $account->mail = $variables['account']->name;
      }
      if (empty($account->homepage) && !empty($variables['account']->homepage)) {
        $account->homepage = $variables['account']->homepage;
      }

It hashes on the name they provide which for me is as good a fall back as I think I am going to get.