nodeownership.module defines the "administer nodeownership" permission with the following description: "Allows a user to view, approve, and decline node ownership claims".

However, this permission is not properly used in the nodeownership_menu function.

Specifically, the following section should use "administer nodeownersihp" instead of the "administer site configuration" permission in its access arguments definition:

  $items['admin/config/nodeownership/nodeclaim/%/%'] = array(
    'page callback' => '_node_claim_process',
    'page arguments' => array(4, 5),
    'title' => 'Ownership claim confirmation',
    'access arguments' => array('administer site configuration'),
    'file' => 'nodeownership.pages.inc',
    'file path' => drupal_get_path('module', 'nodeownership') . '/includes',
  );

Another problem I have not been able to track down is that giving a user "administer nodeownership" permission does not add a menu link for them to reach the Claims (admin/content/claims) page. User 1 has a link called "Claims" under the Content section of the admin menu.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sja1 created an issue. See original summary.

rajeshwari10’s picture

Assigned: Unassigned » rajeshwari10
rajeshwari10’s picture

Status: Active » Needs review
FileSize
579 bytes

Hi sja1,

I have changed the access argument to administer node ownership as per your instructions.
For User to reach the Claim page.
1. Give Administer node ownership claims permission to User.
2. Give Access the content overview page permission to User.
After giving the above permissions User will be able to see the Claims (admin/content/claims) page under the Content section of the admin menu.

Sonal.Sangale’s picture

Assigned: rajeshwari10 » Sonal.Sangale
Status: Needs review » Reviewed & tested by the community

Patch works fine!

rajeshwari10’s picture