i am trying to set up SimpleAds module so that one of my users can view nodes created by advertisements that are added. I have checked all the necessary permissions in the Home » Administration » People screen but it is still giving the user the "Access Denied" error

any feedback on a fix would be appreciated.

CommentFileSizeAuthor
#7 permission-issue-1795174-2.patch578 bytesyvesvanlaer

Comments

binaryweb’s picture

binaryweb commented
Assigned: binaryweb » Unassigned
Alexpi’s picture

Alexpi commented

Change this line 152 in simpleads.module:

if (isset($node->type) && $node->type == 'simpleads' && $op == 'view' && !user_access('administer nodes', $account)) {

to this:

if (isset($node->type) && $node->type == 'simpleads' && $op == 'view' && !user_access('access ad node', $account)) {
minnur’s picture

minnur commented
Assigned: Unassigned » minnur
Status: Active » Needs review
liliplanet’s picture

liliplanet commented
Status: Needs review » Reviewed & tested by the community

perfect, thank you! users can now see their ad ..

minnur’s picture

minnur commented

added to dev.

nor4a’s picture

nor4a commented
Version: 7.x-1.8 » 7.x-2.x-dev

The same issue with 2x.
Now the problem is in the includes/node.inc file - in the "simpleads_node_access" function. Incorrect permission naming.

Change the line:

if (isset($node->type) && $node->type == 'simpleads' && $op == 'view' && !user_access('access ad node', $account)) {

To the line:
if (isset ($node->type) && $node->type == 'simpleads' && $op == 'view' && !user_access('access advertisement node', $account)) {

yvesvanlaer’s picture

yvesvanlaer commented
Issue summary: View changes
StatusFileSize
new permission-issue-1795174-2.patch578 bytes

I created a patch for this. I think It is about time that this will be committed :).

minnur’s picture

minnur commented
Status: Reviewed & tested by the community » Closed (fixed)

Fixed in DEV.

Status: Closed (fixed) » Needs review

poorva queued 7: permission-issue-1795174-2.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 7: permission-issue-1795174-2.patch, failed testing.

minnur’s picture

minnur commented
Status: Needs work » Closed (won't fix)