URL tokens need 'path' data, not 'url' data

token_token_info() defines the url token type as:

  // URL tokens.
  $info['types']['url'] = array(
    'name' => t('URL'),
    'description' => t('Tokens related to URLs.'),
    'needs-data' => 'url',
  );

This is wrong, because token_tokens() is actually looking for a 'path' key in the data, not a 'url' key:

  // URL tokens.
  if ($type == 'url' && !empty($data['path'])) {
    $path = $data['path'];

Patch changes the 'needs-data' value to be 'path' instead of 'url'. URL tests still run green after the patch.

Apparently the 'needs-data' key has no function except as documentation (and the documentation is wrong in this case...). This is unexpected - I would have thought that by specifying 'needs-data' there would be a type check built into the token API so that token_generate() and token_replace() would fail if the required data types were not present in the arguments. Instead, they just return silently without doing anything.