There is an option "0" to be chosen for setting the permission for users for any kind of ads. Please see the attachment for details. Thanks!

CommentFileSizeAuthor
#13 ad_flash-perm_0.patch419 bytesjohn franklin
#12 ad_flash-perm_0.patch634 bytesjohn franklin
#10 702170.patch563 bytesAlex Andrascu
#2 ad_permission.JPG52.47 KBycwjjjj

Comments

ycwjjjj’s picture

ycwjjjj commented
Title: one of permission for users is "0" » one of permissions for users is "0"
ycwjjjj’s picture

ycwjjjj commented
StatusFileSize
new ad_permission.JPG52.47 KB
ycwjjjj’s picture

ycwjjjj commented
Project: Advertisement » Ad Flash
Version: 6.x-2.2 » 6.x-2.4
Component: ad module » Code

The problem happens after the module ad_flash installed and enabled. Checked with ad_flash version 6.x-2.4. Thanks!

Alex Andrascu’s picture

Alex Andrascu commented
Assigned: Unassigned » Alex Andrascu
Priority: Normal » Critical

Thanks...i'll look into it ASAP.

Alex Andrascu’s picture

Alex Andrascu commented

Couldn't reproduce with 6.x-2.2 version on one of my production websites. I'll try with 6.x-2.4

Alex Andrascu’s picture

Alex Andrascu commented

can you please tell me what Advertisment version do you use ? I don't have the same permissions as in the picture above for the Ad 6.x-2.2 (latest stable)

Alex Andrascu’s picture

Alex Andrascu commented

Nevermind...i see you're reffering to admin/content/ad/configure permissions page. That is true...the big 0 is there.

Molot’s picture

Molot commented

Is this project maintained at all? What does that 0 mean?

Alex Andrascu’s picture

Alex Andrascu commented

This project is maintained indeed. It reads right on the front page of it. Search for Under Active Development. I'll have a look at that permission and get back to you. In the mean times patches or clues are welcome.

Alex Andrascu’s picture

Alex Andrascu commented
Version: 6.x-2.4 » 6.x-2.x-dev
Status: Active » Patch (to be ported)
StatusFileSize
new 702170.patch563 bytes

Attached is a patch trying to solve the above problem. Patched against rev 1.9 of ad_flash.module. Commited to DEV version.

john franklin’s picture

john franklin commented

That patch did not fix it for me. I still get the '0' permission with Flash Ad 6.x-2.7 + 702170.patch and Advertisement 6.x-dev.

john franklin’s picture

john franklin commented
StatusFileSize
new ad_flash-perm_0.patch634 bytes

The attached patch does fix it.

john franklin’s picture

john franklin commented
StatusFileSize
new ad_flash-perm_0.patch419 bytes

Oops. That includes the patch for #821270: warning: Parameter 2 to ad_flash_adapi() expected to be a reference (php 5.3 compatibility) as well. Rerolled with only the permission part.

Alex Andrascu’s picture

Alex Andrascu commented
Status: Patch (to be ported) » Needs review
Alex Andrascu’s picture

Alex Andrascu commented

Commited to dev.
With thanks to John Franklin :)