Notice: Undefined variable: pid in simple_access_node_delete() line 279

db_delete($table)
      ->condition('pid', $pid)
      ->execute();

Should be nid?

db_delete($table)
      ->condition('nid', $node->nid)
      ->execute();
CommentFileSizeAuthor
#5 simple_access.patch564 bytescooperq

Comments

Wolfflow’s picture

Wolfflow commented

same error

sylv3st3r’s picture

sylv3st3r commented

Just change the code. I have a working D7 modules, well with some bugs. Can't contact this module author...

Wolfflow’s picture

Wolfflow commented

@sylv3st3r thx

Jiri Volf’s picture

Jiri Volf commented

Thanks, works for me too.

cooperq’s picture

cooperq commented
Status: Active » Patch (to be ported)
StatusFileSize
new simple_access.patch564 bytes

Here is a patch so the developer can integrate.

andresp’s picture

andresp commented

this also works for me

damien_vancouver’s picture

damien_vancouver commented
Version: 7.x-2.0-alpha1 » 7.x-2.0-beta1
Status: Patch (to be ported) » Closed (fixed)

The above fix worked for me too.

Then I noticed that it's actually in the 7.x-2.x-dev branch, as of commit 3b31876.

Therefore I'm marking this issue as fixed. Using 7.x-2.x-dev instead of 7.x-2.0-beta1 will remove the problem.