Would suggest making a quick change in the function content_lock_node($nid, $uid)
It currently does not display a message if you have a user without administer permissions. We want users to be able to see the lock is there that have editing rights but that do not have full admin rights to the whole system. Hence this code:

 if (user_access('administer checked out documents')) {
$message = content_lock_lock_owner($lock);
    $url = "admin/content/node/content_lock/release/$nid";
      }
   
      if (isset($url)) {
        $message .= '<br />'. t('Click <a href="!release-url">here</a> to check back in now.', array('!release-url' => url($url, array('query' => 'destination='. $_GET['q']))));
	}

Should be changed to this

$message = content_lock_lock_owner($lock);
    $url = "admin/content/node/content_lock/release/$nid";
    
    if (user_access('administer checked out documents')) {
      if (isset($url)) {
        $message .= '<br />'. t('Click <a href="!release-url">here</a> to check back in now.', array('!release-url' => url($url, array('query' => 'destination='. $_GET['q']))));
      }
	}

Also I am working on some of the wording of the Text as it is a bit confusing at times. Will post more bout that. I am doing the fixes here manually.

Comments

omaster’s picture

Project: Checkout (Content locking) » Content locking (anti-concurrent editing)
Version: master » 6.x-2.1
omaster’s picture

Project: Content locking (anti-concurrent editing) » Checkout (Content locking)
Version: 6.x-2.1 » master
omaster’s picture

Project: Checkout (Content locking) » Content locking (anti-concurrent editing)
Version: master » 6.x-2.1
eugenmayer’s picture

Category: feature » bug

Thats a bug, not a feature :) Thank you for the contribution!

eugenmayer’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.