Dear Maintainer,

I have a 5.7 install with the newst Ad, ad_embeded, text_ad, installed. And I was trying to add the ad manually with the "
"Replace ad bracket tags" or "Replace ad comment tags". This will cause the
ad_embed module to replace embedded [[add]] or

tags respectively."

But no result, i have been to "Administer >> Content
Management >> Ads >> Settings >> Embedded ads" to mark the tags as active.

Could it be that ad is not working with tinyMCE? Shoul we do a ad filter like insert view?

Dirk

Comments

jeremy’s picture

jeremy commented
Priority: Critical » Normal

If you disable tinyMCE, does the problem go away?

designwork’s picture

designwork commented

Hi Jeremy,

even if I disable tinyMCE the Problem is the same. I can randomly insert ads in contenttyps but no chance with the tags. I tryed [[ad]] or

even this

. The [[ad]] tag is displayed as normal text the

tag shows nothing (full HTML is enabled).

So I will have a look at the ad_embed module.

Dirk

designwork’s picture

designwork commented

Hi Jeremy,

I found the Problem. Its in the nodeapi. You are only replacing tags in nodes if the author has the permisssion´and comparing it with the node-uid. But in My case this does not work. Because some content creators do not have the permission to place ads in content,. And when I later wanna put some ads in the content, I´m not allowed to because you go this if (user_access('embed ads in content', user_load(array('uid' => $node->uid)))) {

So I changed it to simply if (user_access('embed ads in content' )) { This works perfect. Its line 81 in the ad_embed_module.

Thanks for the module... If you need some help just tell me.

Dirk

jeremy’s picture

jeremy commented
Title: Ad tags not working with tinyMCE » Problems with ad_embed permissions
Category: support » bug

Updating the title of the issue to reflect the problem. I've been meaning to revisit that module for a while now, but have not yet found time. When I do, I will take a closer look at this permission issues. Thanks for the detailed bug report.

jeremy’s picture

jeremy commented
Status: Active » Closed (works as designed)

Your solution doesn't really fix anything -- it only seems to work for you, as you have the necessary permissions. But someone else viewing your content that doesn't have the necessary permissions wouldn't actually be able to see the embedded ads. By design.