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.

CommentFileSizeAuthor
#3 1384652-undefined_var_notice_fix.patch521 bytessnufkin

Comments

rszrama’s picture

rszrama commented
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.

ntigh52’s picture

ntigh52 commented

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/

snufkin’s picture

snufkin commented
Status: Active » Needs review
StatusFileSize
new 1384652-undefined_var_notice_fix.patch521 bytes

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

snufkin’s picture

snufkin commented
Title: bug with sission limit » Undefined replacements variable throws notice
ntigh52’s picture

ntigh52 commented
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.

snufkin’s picture

snufkin commented
Status: Fixed » Reviewed & tested by the community
deekayen’s picture

deekayen commented
Status: Reviewed & tested by the community » Fixed

committed

Status: Fixed » Closed (fixed)

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