I received an email from Oleg with the following bug report:

Hi,

We're using your Dart module, and I just spent a ton of time trying to figure out one particular hook bug.

hook_dart_tag_alter was only called for one of the ads on the page, not for all of them.

Here's the code change that fixed the problem. In dart.module these lines should be inside the loop just a couple of lines above: 

249 |     // Give other modules the opportunity to add infrmation to the tag.
250 |     drupal_alter('dart_tag', $tag);


thanks for your code, man!

Oleg
CommentFileSizeAuthor
#1 hook_alter.patch501 bytesbleen

Comments

bleen’s picture

Title: hook_ » hook_dart_tag_alter not running for all tags
Status: Active » Needs review
StatusFileSize
new501 bytes

Attached is a patch based on the suggestions of Oleg

bleen’s picture

Status: Needs review » Closed (fixed)

I just went ahead and committed this