Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I don't understand the reason for replacing "adsense" with "" in the following line of code in block--adsense-managed.tpl.php...
<div id="jcv <?php print str_replace('adsense', '', $block_html_id); ?>" class="<?php print str_replace('adsense', '', $classes); ?>"<?php print $attributes; ?>>
It is causing the ids and classes to end up like this...
<div id="jcv block--managed-0" class="block block--managed">
Also, a bit deeper into the DIV hierarchy, there's a DIV that ends up with the following classes...
<div class="adsense adsense_managed_" style="display:inline-block;width:728px;height:90px;">
I assume there's either supposed to be a number after "adsense_managed_", or the trailing "_" should not be there.
Comments
Comment #1
jcnventura CreditAttribution: jcnventura commentedIt's all on purpose, to circumvent some adblocker rules, that search for block-adsense-managed.
The 'jcv' in the ID is something I need to remove, as it was left-over debug info.
Comment #2
jcnventura CreditAttribution: jcnventura commentedComment #3
ExTexan CreditAttribution: ExTexan commentedWell, couldn't you come up with a better result than leaving it as "block--managed" and "block--managed-0"? That just makes it look like a bug.
And what about "adsense_managed_"... is something supposed to follow? ...like the managed ad number, perhaps ("0" in this case)?
Comment #4
jcnventura CreditAttribution: jcnventura commentedExTexan: no I couldn't. Feel free to suggest something else that:
a) doesn't collide with any other Drupal module, and
b) will not immediately be added to the AdBlock lists.
Nothing is supposed to follow "adsense_managed_". It was just a way to avoid being blacklisted. It's already blacklisted as well, so I'll probably remove it altogether. One possibility would be to generate a random class, and have that class be known inside Drupal, but I don't think it's worth the effort.
Comment #5
jcnventura CreditAttribution: jcnventura commentedSee also http://forums.lanik.us/viewtopic.php?f=64&t=12453&p=46661&hilit=drupal#p...