We are currently upgrading our 6.x site to 7.x. Diff is causing some problems when enabled. The first issue is on any editable page, a warning occurs:

Warning: Illegal offset type in isset or empty in _node_revision_access() (line 1823 of .../modules/node/node.module).

The line in question is: "if (isset($map[$op]) && (!user_access($map[$op]) && !user_access('administer nodes'))) {"

The second is when a new pending revision is accessed, four warning messages occur:

Warning: Missing argument 1 for diff_node_revision_access() in diff_node_revision_access() (line 101 of .../sites/all/modules/diff/diff.module).

This line is: "function diff_node_revision_access($node, $op = 'view') {"

Each case is an issue with an array variable ($map and $node). This is happening with core 7.7. Any ideas? Thanks.

Comments

realityloop’s picture

realityloop commented
Status: Active » Postponed (maintainer needs more info)

Unless you can provide an environment to debug this on I don't know I'll be able to help much.

bcguitars’s picture

bcguitars commented

We got this fixed, and I thought I'd just note this post with the solution we came up with.

By modifying the weight value in the system table (set diff to -2, revisioning to -1), this fixed the issue and the two modules are now playing nicely together.

bcguitars’s picture

bcguitars commented
Status: Postponed (maintainer needs more info) » Closed (fixed)