Download & Extend
Universally Unique IDentifier » Issues

Alpha3 causes Notice: Undefined variable: replacements in uuid_tokens() (line 95 of ...sites\all\modules\uuid\uuid.tokens.)

Posted by David4514 on February 5, 2012 at 5:01am
Project:Universally Unique IDentifier
Version:7.x-1.0-alpha3
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (duplicate)

Issue Summary

After installing alpha3 release, any attempt to edit any content results in the following message.

  Notice: Undefined variable: replacements in uuid_tokens() (line 95 of ...sites\all\modules\uuid\uuid.tokens.inc).

In the newly updated uuid_tokens function in uuid.tokens.inc, if the $type is not one of the values in the switch block (node, user, file, term, or comment), an attempt is made to return $replacements. However, the $type is 'entity' which does not match any of the values in the switch block. The $replacements variable is undefined and the return fails.

Probably the $replacements variable should be defined as an empty array prior to the switch block. I do find it odd that no test is made for a $type of entity.

I see that this function has been replaced since alpha3. The newly written function available in the dev version as of 2/4/2012 also fails, but with different messages.

Notice: Undefined index: uuid in uuid_tokens() (line 38 of ...sites/all/modules/uuid/uuid.tokens.inc).
Notice: Undefined index: vuuid in uuid_tokens() (line 40 of .../sites/all/modules/uuid/uuid.tokens.inc).
Login or register to post comments

Comments

#1

Posted by kannary100 on February 5, 2012 at 9:55pm

Exact same problem with identical errors on the content edit page.

#2

Posted by jzornig on February 6, 2012 at 12:25am

subscribe

#3

Posted by cpelham on February 6, 2012 at 1:04am

I'm getting the same error.

#4

Posted by dixon_ on February 6, 2012 at 7:52am
Status:active» fixed

This is a duplicate of #1423542: Rewrite token integration to be more entity generic and fixed in 7.x-1.x-dev already.

#5

Posted by dixon_ on February 6, 2012 at 7:54am
Status:fixed» closed (duplicate)

#6

Posted by Torenware on February 28, 2012 at 1:20am

Fixed, but is there a released version that has the fix? Or do you recommend the dev build for this?

#7

Posted by heckad on July 8, 2012 at 2:39am

same problem any fix?

#8

Posted by trangunghoa on July 11, 2012 at 11:18am
Category:bug report» support request

I fixed this error by:
add line "$replacements = array();" in head function uuid_tokens

"
function uuid_tokens($type, $tokens, array $data = array(), array $options = array()) {
+ $replacements = array();
switch ($type) {
"

#9

Posted by bagpiper on July 15, 2012 at 9:44pm

Thank you, that did the trick Trangunghoa. Hope we get the beta release for UUID soon :-)

#10

Posted by flashwebcenter on July 18, 2012 at 2:14am

Thank you trangunghoa for your help.

#11

Posted by yannickoo on July 31, 2012 at 1:46pm

Saw this in the dev release so it's fixed.

#12

Posted by csander on August 3, 2012 at 6:06pm

Awesome, thanks for the fix!

#13

Posted by Citlalli on October 25, 2012 at 6:26pm

where add the line it??

#14

Posted by jmonroe on November 6, 2012 at 2:05pm

Thanks...this fixed my error too.

#15

Posted by jmonroe on November 6, 2012 at 2:08pm

@Citlalli - You need to add it under line 95 of /sites/all/modules/uuid/uuid.tokens.inc

So line 95-97 looks as below

function uuid_tokens($type, $tokens, array $data = array(), array $options = array()) {
+ $replacments = array();
switch ($type) {

That will fix it for you.

#16

Posted by Citlalli on November 10, 2012 at 1:32am

i put it, it isnt working :S, I am using local server, may be is for that isnt?

#17

Posted by luistope on November 14, 2012 at 4:16pm

It works for me!
@Citlalli be careful with the copy paste. + $replacments = array(); is + $replacements An 'e' misses in jmonroe solution. ;)

#18

Posted by kyke01 on November 23, 2012 at 9:31am

It's works for me too but without the + sign ,
just add $replacements = array(); to the function

#19

Posted by Myoos on November 25, 2012 at 9:42pm

trangunghoa : thank u soo much this was help full =)

#20

Posted by Seb36 on November 30, 2012 at 2:49am

thanks trangunghoa
kyke01 Lines beginning with "+" are added (without the "+", of course).
http://drupal.org/node/534548

#21

Posted by gregmarsh on December 8, 2012 at 11:40pm

Just wanted to say thank you for this ... very simple and quick fix.

All the best.

#22

Posted by timaholt on December 17, 2012 at 10:31pm

For anyone using drush make and the alpha3 release, here's a patch for this issue.

AttachmentSizeStatusTest resultOperations
uuid.token_.inc-undefined-replacements.patch433 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch uuid.token_.inc-undefined-replacements.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#23

Posted by roborracle on December 27, 2012 at 4:06pm

#22 patch worked for me

#24

Posted by shrop on December 29, 2012 at 12:57am

Patch #22 worked for me also. Thanks!

#25

Posted by kyleoliveira on January 4, 2013 at 7:21pm

#15 seems to have fixed the issue for me. I would definitely suggest adding it in the next release.

#26

Posted by Exploratus on February 2, 2013 at 11:11am

22 Worked for me.

Thanks. Should commit this no?

#27

Posted by Offlein on February 6, 2013 at 4:54pm
Status:closed (duplicate)» reviewed & tested by the community

This probably won't get fixed because the status was closed.

Please put #22 in.

#28

Posted by System Message on February 6, 2013 at 4:57pm
Status:reviewed & tested by the community» needs work

The last submitted patch, uuid.token_.inc-undefined-replacements.patch, failed testing.

#29

Posted by skwashd on February 12, 2013 at 1:48pm
Status:needs work» closed (duplicate)

This issue has previously been closed as a duplicate of #1423542: Rewrite token integration to be more entity generic. I am closing it again.

#30

Posted by windmaomao on February 25, 2013 at 6:46pm

so what is the issue to not commit this simple change to the repository ? Can someone explain why this line is not considered a good commit ? thanks. I'm puzzled, since everytime i do a drush dl uuid, i have to come back add this fix.

#31

Posted by Offlein on February 25, 2013 at 8:02pm

I'm sorry, I hadn't followed this properly, but I suspect, windmaomao, that you're not using Drush will only download the -alpha3 version, not the -dev version, and hence you're not dl'ing the latest code?

If you ARE, try submitting a new ticket.

#32

Posted by timaholt on March 4, 2013 at 2:00pm

Not sure why the patch failed testing as it patches correctly for me manually and using drush make. Any ideas?

#33

Posted by chrisjlee on March 6, 2013 at 5:10pm
Status:closed (duplicate)» needs review

#22: uuid.token_.inc-undefined-replacements.patch queued for re-testing.

#34

Posted by System Message on March 6, 2013 at 5:13pm
Status:needs review» needs work

The last submitted patch, uuid.token_.inc-undefined-replacements.patch, failed testing.

#35

Posted by chrisjlee on March 6, 2013 at 9:54pm
Status:needs work» fixed

I just updated to the dev release and it seems to have disappeared.

#36

Posted by ascii122 on March 19, 2013 at 8:02pm
Category:support request» bug report

The latest 7.x-1.0-alpha3 version has a typo: File: uuid.tokens.inc Line 48

$replacments = array();
should be
$replacements = array();

#37

Posted by pfrenssen on March 21, 2013 at 3:18pm
Status:fixed» closed (duplicate)

This has not been fixed here but in #1423542: Rewrite token integration to be more entity generic, marking this as a duplicate of that issue.

#38

Posted by dgamache on March 22, 2013 at 10:55am

$replacements = array();
Patch is not working for me.
drupal core 7.21
Open Enterprise Profile
Universally Unique ID 7.x-1.0

#39

Posted by kwfinken on March 26, 2013 at 6:37pm
Status:closed (duplicate)» needs review

#22: uuid.token_.inc-undefined-replacements.patch queued for re-testing.

#40

Posted by System Message on March 26, 2013 at 6:40pm
Status:needs review» needs work

The last submitted patch, uuid.token_.inc-undefined-replacements.patch, failed testing.

#41

Posted by pfrenssen on March 26, 2013 at 9:30pm
Status:needs work» closed (duplicate)

No need to test the patch or reopen this issue. This has long been fixed in the development version. Don't use the 7.x-1.0-alpha3, it is over a year old. Use 7.x-1.x-dev.

Marking this as a duplicate of #1423542: Rewrite token integration to be more entity generic, this is the issue in which this problem was fixed.

nobody click here