When using hook_url_outbound_alter() to rewrite links to include the tokenauth token, tokenauth_get_token() is called. This function just hits the database. Stash the value as a static and the extra database hits can be avoided.

CommentFileSizeAuthor
#1 1038584_tokenauth_reset.patch1.12 KBhefox
tokenauth.static.patch881 bytesGrayside

Comments

hefox’s picture

hefox commented
StatusFileSize
new 1038584_tokenauth_reset.patch1.12 KB

Adding a reset and resetting it after updating the token.

Tested via:


dsm('Should say original token: ' . tokenauth_get_token(1));

tokenauth_reset_user(1, 'blah');
dsm('Should say blah: ' . tokenauth_get_token(1));

$result = array(
  'token' => 'blah2',
  'uid' => 1,
);
drupal_write_record('tokenauth_tokens', $result, 'uid');

dsm('Should say old token: ' . tokenauth_get_token(1));
dsm('Should say blah2: ' . tokenauth_get_token(1, TRUE));
Grayside’s picture

Grayside commented
Status: Needs review » Fixed

http://drupal.org/cvs?commit=493384

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.