Undefined replacements variable throws notice [#1384652]

hi,
what do you think is the Reason,
when I finish to order some products, and session_limit module enable I get:

•Notice: Undefined variable: replacements in session_limit_tokens() (line - 71 C:\wamp\www\mysite\sites\all\modules\session_limit\session_limit.tokens.inc).

What do you think is the reason?
thanks a lot.

Comment	File	Size	Author
#3	1384652-undefined_var_notice_fix.patch	521 bytes	snufkin
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rszrama CreditAttribution: rszrama commented 26 December 2011 at 04:15

Project:	Commerce Core	» Session Limit
Version:	7.x-1.1	» 7.x-2.0-beta1
Component:	Checkout	» Code

No clue, never used the module. Obviously there's something odd in the token definition there, and perhaps the Rule to send an order notification e-mail just turned it up.

Comment #2

ntigh52 CreditAttribution: ntigh52 commented 26 December 2011 at 11:16

Thanks rszrama,
Its also when I change user role - the same problem.
example:
from •Anonymous user to •Authenticated user.
or from •Authenticated user to some role else...
any Solutions to someone?
Thanks/

Comment #3

snufkin CreditAttribution: snufkin commented 26 December 2011 at 15:19

Status:

Active

» Needs review

File	Size
1384652-undefined_var_notice_fix.patch	521 bytes

The issue is that the module did not define the $replacements variable in its hook_tokens implementation. Patch fixes this notice.

Comment #4

snufkin CreditAttribution: snufkin commented 26 December 2011 at 15:21

Title:

bug with sission limit

» Undefined replacements variable throws notice

Comment #5

ntigh52 CreditAttribution: ntigh52 commented 30 December 2011 at 12:50

Status:

Needs review

» Fixed

snufkin,
I dont know how to install a patch,
But I opened your patch and added the line ( 1 line ) that you define the $replacements variable in its hook_tokens implementation. ( on the place that I saw in your patch ).
Its work!!!
Well done.
Thanks a lot.