Closed (fixed)
Project:
Content locking (anti-concurrent editing)
Version:
6.x-2.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Sep 2010 at 11:59 UTC
Updated:
13 Oct 2010 at 14:50 UTC
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
Comment #1
omaster commentedComment #2
omaster commentedComment #3
omaster commentedComment #4
eugenmayer commentedThats a bug, not a feature :) Thank you for the contribution!
Comment #5
eugenmayer commented