2 related bugs

Hi there,

While looking at the code, I've spotted 2 minor issues that I'd consider bugs:

1. In simple_page_title.db.inc (lines 94-96):

     if (!empty($path_alias))
     $result_object = db_fetch_object($result);
     return $result_object;
   

   Only setting $result_object if $path_alias is not empty doesn't seem correct. Maybe it should be: if (!empty($result))




2. The reason that code isn't causing trouble currently is because $path_alias is always being passed in, because of this code from simple_page_title.module (lines 98-100):

     $path = drupal_get_path_alias($_GET['q']);
     //check for custom title based on original url and alias
     $page_title_object = simple_page_title_get_page_title_object_for_path($_GET['q'], $path);
   

   drupal_get_path_alias returns the value passed to it if no alias exists. So anytime that's the case, simple_page_title_get_page_title_object_for_path is being called with 2 identical parameters. It works, but I doubt that's what was intended.