From d7b567447616c560e2bb1e1d4fad9193042b1dd2 Mon Sep 17 00:00:00 2001
From: Hypolite Petovan <hypolite@mrpetovan.com>
Date: Sun, 26 Jul 2020 18:35:02 -0400
Subject: [PATCH] Move mod/lockview to Module\PermissionTooltip

- Add explicit type parameter to lockview() in main.js
---
 doc/Addons.md                                 |   8 +-
 doc/de/Addons.md                              |   8 +-
 src/Module/PermissionTooltip.php              | 122 ++++++++++++++++++
 static/routes.config.php                      |   2 +
 view/js/main.js                               |  27 ++--
 view/templates/photo_view.tpl                 |   2 +-
 view/templates/search_item.tpl                |   2 +-
 view/templates/wall_thread.tpl                |   2 +-
 view/theme/frio/templates/photo_view.tpl      |   2 +-
 view/theme/frio/templates/search_item.tpl     |   4 +-
 view/theme/frio/templates/wall_thread.tpl     |   4 +-
 view/theme/quattro/templates/photo_view.tpl   |   2 +-
 view/theme/quattro/templates/search_item.tpl  |   2 +-
 view/theme/quattro/templates/wall_thread.tpl  |   2 +-
 view/theme/smoothly/templates/search_item.tpl |   2 +-
 view/theme/smoothly/templates/wall_thread.tpl |   2 +-
 view/theme/vier/templates/photo_item.tpl      |   2 +-
 view/theme/vier/templates/photo_view.tpl      |   2 +-
 view/theme/vier/templates/search_item.tpl     |   4 +-
 view/theme/vier/templates/wall_thread.tpl     |   2 +-
 20 files changed, 158 insertions(+), 45 deletions(-)
 create mode 100644 src/Module/PermissionTooltip.php

diff --git a/doc/Addons.md b/doc/Addons.md
index 54363cb1d7..c1861c7913 100644
--- a/doc/Addons.md
+++ b/doc/Addons.md
@@ -604,10 +604,6 @@ Here is a complete list of all hook callbacks with file locations (as of 24-Sep-
 
     Hook::callAll('post_local_end', $arr);
 
-### mod/lockview.php
-
-    Hook::callAll('lockview_content', $item);
-
 ### mod/uexport.php
 
     Hook::callAll('uexport_options', $options);
@@ -679,6 +675,10 @@ Here is a complete list of all hook callbacks with file locations (as of 24-Sep-
     Hook::callAll('register_account', $uid);
     Hook::callAll('remove_user', $user);
 
+### src/Module/PermissionTooltip.php
+
+    Hook::callAll('lockview_content', $item);
+
 ### src/Content/ContactBlock.php
 
     Hook::callAll('contact_block_end', $arr);
diff --git a/doc/de/Addons.md b/doc/de/Addons.md
index 745010ff4e..2ff7495497 100644
--- a/doc/de/Addons.md
+++ b/doc/de/Addons.md
@@ -312,10 +312,6 @@ Eine komplette Liste aller Hook-Callbacks mit den zugehörigen Dateien (am 01-Ap
 
     Hook::callAll('post_local_end', $arr);
 
-### mod/lockview.php
-
-    Hook::callAll('lockview_content', $item);
-
 ### mod/uexport.php
 
     Hook::callAll('uexport_options', $options);
@@ -422,6 +418,10 @@ Eine komplette Liste aller Hook-Callbacks mit den zugehörigen Dateien (am 01-Ap
 
     Hook::callAll('storage_instance', $data);
 
+### src/Module/PermissionTooltip.php
+
+    Hook::callAll('lockview_content', $item);
+
 ### src/Worker/Directory.php
 
     Hook::callAll('globaldir_update', $arr);
diff --git a/src/Module/PermissionTooltip.php b/src/Module/PermissionTooltip.php
new file mode 100644
index 0000000000..59478b326c
--- /dev/null
+++ b/src/Module/PermissionTooltip.php
@@ -0,0 +1,122 @@
+<?php
+
+namespace Friendica\Module;
+
+use Friendica\Core\Hook;
+use Friendica\Database\DBA;
+use Friendica\DI;
+use Friendica\Model\Item;
+use Friendica\Model\Group;
+use Friendica\Network\HTTPException;
+
+/**
+ * Outputs the permission tooltip HTML content for the provided item, photo or event id.
+ */
+class PermissionTooltip extends \Friendica\BaseModule
+{
+	public static function rawContent(array $parameters = [])
+	{
+		parent::rawContent($parameters); // TODO: Change the autogenerated stub
+
+		$type = $parameters['type'];
+		$referenceId = $parameters['id'];
+
+		$expectedTypes = ['item', 'photo', 'event'];
+		if (!in_array($type, $expectedTypes)) {
+			throw new HTTPException\BadRequestException(DI::l10n()->t('Wrong type "%s", expected one of: %s', $type, implode(', ', $expectedTypes)));
+		}
+
+		$condition = ['id' => $referenceId];
+		if ($type == 'item') {
+			$fields = ['uid', 'psid', 'private'];
+			$model = Item::selectFirst($fields, $condition);
+		} else {
+			$fields = ['uid', 'allow_cid', 'allow_gid', 'deny_cid', 'deny_gid'];
+			$model = DBA::selectFirst($type, $fields, $condition);
+		}
+
+		if (!DBA::isResult($model)) {
+			throw new HttpException\NotFoundException(DI::l10n()->t('Model not found'));
+		}
+
+		if (isset($model['psid'])) {
+			$permissionSet = DI::permissionSet()->selectFirst(['id' => $model['psid']]);
+			$model['allow_cid'] = $permissionSet->allow_cid;
+			$model['allow_gid'] = $permissionSet->allow_gid;
+			$model['deny_cid']  = $permissionSet->deny_cid;
+			$model['deny_gid']  = $permissionSet->deny_gid;
+		}
+
+		// Kept for backwards compatiblity
+		Hook::callAll('lockview_content', $model);
+
+		if ($model['uid'] != local_user() ||
+			isset($model['private'])
+			&& $model['private'] == Item::PRIVATE
+			&& empty($model['allow_cid'])
+			&& empty($model['allow_gid'])
+			&& empty($model['deny_cid'])
+			&& empty($model['deny_gid']))
+		{
+			echo DI::l10n()->t('Remote privacy information not available.');
+			exit;
+		}
+
+		$aclFormatter = DI::aclFormatter();
+
+		$allowed_users  = $aclFormatter->expand($model['allow_cid']);
+		$allowed_groups = $aclFormatter->expand($model['allow_gid']);
+		$deny_users     = $aclFormatter->expand($model['deny_cid']);
+		$deny_groups    = $aclFormatter->expand($model['deny_gid']);
+
+		$o = DI::l10n()->t('Visible to:') . '<br />';
+		$l = [];
+
+		if (count($allowed_groups)) {
+			$key = array_search(Group::FOLLOWERS, $allowed_groups);
+			if ($key !== false) {
+				$l[] = '<b>' . DI::l10n()->t('Followers') . '</b>';
+				unset($allowed_groups[$key]);
+			}
+
+			$key = array_search(Group::MUTUALS, $allowed_groups);
+			if ($key !== false) {
+				$l[] = '<b>' . DI::l10n()->t('Mutuals') . '</b>';
+				unset($allowed_groups[$key]);
+			}
+
+			foreach (DI::dba()->selectToArray('group', ['name'], ['id' => $allowed_groups]) as $group) {
+				$l[] = '<b>' . $group['name'] . '</b>';
+			}
+		}
+
+		foreach (DI::dba()->selectToArray('contact', ['name'], ['id' => $allowed_users]) as $contact) {
+			$l[] = $contact['name'];
+		}
+
+		if (count($deny_groups)) {
+			$key = array_search(Group::FOLLOWERS, $deny_groups);
+			if ($key !== false) {
+				$l[] = '<b><strike>' . DI::l10n()->t('Followers') . '</strike></b>';
+				unset($deny_groups[$key]);
+			}
+
+			$key = array_search(Group::MUTUALS, $deny_groups);
+			if ($key !== false) {
+				$l[] = '<b><strike>' . DI::l10n()->t('Mutuals') . '</strike></b>';
+				unset($deny_groups[$key]);
+			}
+
+			foreach (DI::dba()->selectToArray('group', ['name'], ['id' => $allowed_groups]) as $group) {
+				$l[] = '<b><strike>' . $group['name'] . '</strike></b>';
+			}
+		}
+
+		foreach (DI::dba()->selectToArray('contact', ['name'], ['id' => $deny_users]) as $contact) {
+			$l[] = '<strike>' . $contact['name'] . '</strike>';
+		}
+
+		echo $o . implode(', ', $l);
+		exit();
+	}
+}
diff --git a/static/routes.config.php b/static/routes.config.php
index 8c3fba99b7..ddfabd7780 100644
--- a/static/routes.config.php
+++ b/static/routes.config.php
@@ -237,6 +237,8 @@ return [
 	'/openid'         => [Module\Security\OpenID::class, [R::GET]],
 	'/opensearch'     => [Module\OpenSearch::class,      [R::GET]],
 
+	'/permission/tooltip/{type}/{id:\d+}' => [Module\PermissionTooltip::class, [R::GET]],
+
 	'/photo' => [
 		'/{name}'                    => [Module\Photo::class, [R::GET]],
 		'/{type}/{name}'             => [Module\Photo::class, [R::GET]],
diff --git a/view/js/main.js b/view/js/main.js
index 60337918b4..36c9cbb88d 100644
--- a/view/js/main.js
+++ b/view/js/main.js
@@ -750,26 +750,23 @@ function getPosition(e) {
 
 var lockvisible = false;
 
-function lockview(event,id) {
+function lockview(event, type, id) {
 	event = event || window.event;
 	cursor = getPosition(event);
 	if (lockvisible) {
-		lockviewhide();
+		lockvisible = false;
+		$('#panel').hide();
 	} else {
 		lockvisible = true;
-		$.get('lockview/' + id, function(data) {
-			$('#panel').html(data);
-			$('#panel').css({'left': cursor.x + 5 , 'top': cursor.y + 5});
-			$('#panel').show();
+		$.get('permission/tooltip/' + type + '/' + id, function(data) {
+			$('#panel')
+				.html(data)
+				.css({'left': cursor.x + 5 , 'top': cursor.y + 5})
+				.show();
 		});
 	}
 }
 
-function lockviewhide() {
-	lockvisible = false;
-	$('#panel').hide();
-}
-
 function post_comment(id) {
 	unpause();
 	commentBusy = true;
@@ -940,14 +937,6 @@ function groupChangeMember(gid, cid, sec_token) {
 	});
 }
 
-function profChangeMember(gid,cid) {
-	$('body .fakelink').css('cursor', 'wait');
-	$.get('profperm/' + gid + '/' + cid, function(data) {
-			$('#prof-update-wrapper').html(data);
-			$('body .fakelink').css('cursor', 'auto');
-	});
-}
-
 function contactgroupChangeMember(checkbox, gid, cid) {
 	let url;
 	// checkbox.checked is the checkbox state after the click
diff --git a/view/templates/photo_view.tpl b/view/templates/photo_view.tpl
index 7170ceb333..0d7ccb20fa 100644
--- a/view/templates/photo_view.tpl
+++ b/view/templates/photo_view.tpl
@@ -17,7 +17,7 @@
 		| <a id="photo-toprofile-link" href="{{$tools.profile.0}}">{{$tools.profile.1}}</a>
 	{{/if}}
 	{{if $tools.lock}}
-		| <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event,'photo/{{$id}}');" />
+		| <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event, 'photo', {{$id}});" />
 	{{/if}}
 {{/if}}
 </div>
diff --git a/view/templates/search_item.tpl b/view/templates/search_item.tpl
index 38aa947498..1a756db8a3 100644
--- a/view/templates/search_item.tpl
+++ b/view/templates/search_item.tpl
@@ -17,7 +17,7 @@
 			</div>
 			<div class="wall-item-photo-end"></div>	
 			<div class="wall-item-wrapper" id="wall-item-wrapper-{{$item.id}}" >
-				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event,{{$item.id}});" /></div>
+				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event, 'item', {{$item.id}});" /></div>
 				{{else}}<div class="wall-item-lock"></div>{{/if}}	
 				<div class="wall-item-location" id="wall-item-location-{{$item.id}}">{{$item.location nofilter}}</div>
 			</div>
diff --git a/view/templates/wall_thread.tpl b/view/templates/wall_thread.tpl
index 0d8c896e16..cec886253f 100644
--- a/view/templates/wall_thread.tpl
+++ b/view/templates/wall_thread.tpl
@@ -43,7 +43,7 @@
 			</div>
 			<div class="wall-item-photo-end"></div>
 			<div class="wall-item-wrapper" id="wall-item-wrapper-{{$item.id}}" >
-				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event,{{$item.id}});" /></div>
+				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event, 'item', {{$item.id}});" /></div>
 				{{else}}<div class="wall-item-lock"></div>{{/if}}
 				<div class="wall-item-location" id="wall-item-location-{{$item.id}}">{{$item.location nofilter}}</div>
 			</div>
diff --git a/view/theme/frio/templates/photo_view.tpl b/view/theme/frio/templates/photo_view.tpl
index 91e9dafe44..de45eecff0 100644
--- a/view/theme/frio/templates/photo_view.tpl
+++ b/view/theme/frio/templates/photo_view.tpl
@@ -37,7 +37,7 @@
 	{{/if}}
 	{{if $tools.lock}}
 		<span class="icon-padding"> </span>
-		<a id="photo-lock-link" onclick="lockview(event,'photo/{{$id}}');" title="{{$tools.lock}}" data-toggle="tooltip">
+		<a id="photo-lock-link" onclick="lockview(event, 'photo', {{$id}});" title="{{$tools.lock}}" data-toggle="tooltip">
 			<i class="page-action faded-icon fa fa-lock"></i>
 		</a>
 	{{/if}}
diff --git a/view/theme/frio/templates/search_item.tpl b/view/theme/frio/templates/search_item.tpl
index a5b6d52d6d..2cd231f8fb 100644
--- a/view/theme/frio/templates/search_item.tpl
+++ b/view/theme/frio/templates/search_item.tpl
@@ -1,7 +1,7 @@
 <!-- TODO => Unknow block -->
 <div class="wall-item-decor" style="display:none;">
 	<span class="icon s22 star {{$item.isstarred}}" id="starred-{{$item.id}}" title="{{$item.star.starred}}">{{$item.star.starred}}</span>
-	{{if $item.lock}}<span class="navicon lock fakelink" onclick="lockview(event, {{$item.id}});" title="{{$item.lock}}"></span><span class="fa fa-lock" aria-hidden="true"></span>{{/if}}
+	{{if $item.lock}}<span class="navicon lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}"></span><span class="fa fa-lock" aria-hidden="true"></span>{{/if}}
 </div>
 <!-- ./TODO => Unknow block -->
 
@@ -56,7 +56,7 @@
 					</a>
 				{{/if}}
 				{{if $item.lock}}
-					<span class="navicon lock fakelink" onClick="lockview(event, {{$item.id}});" title="{{$item.lock}}">
+					<span class="navicon lock fakelink" onClick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}">
 						&nbsp;<small><i class="fa fa-lock" aria-hidden="true"></i></small>
 					</span>
 				{{/if}}
diff --git a/view/theme/frio/templates/wall_thread.tpl b/view/theme/frio/templates/wall_thread.tpl
index c15b110ec9..5e7f49bf48 100644
--- a/view/theme/frio/templates/wall_thread.tpl
+++ b/view/theme/frio/templates/wall_thread.tpl
@@ -63,7 +63,7 @@ as the value of $top_child_total (this is done at the end of this file)
 	{{if $item.star}}
 	<span class="icon s22 star {{$item.isstarred}}" id="starred-{{$item.id}}" title="{{$item.star.starred}}">{{$item.star.starred}}</span>
 	{{/if}}
-	{{if $item.lock}}<span class="navicon lock fakelink" onclick="lockview(event,{{$item.id}});" title="{{$item.lock}}"></span><span class="fa fa-lock"></span>{{/if}}
+	{{if $item.lock}}<span class="navicon lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}"></span><span class="fa fa-lock"></span>{{/if}}
 </div>
 {{* /TODO => Unknown block *}}
 
@@ -138,7 +138,7 @@ as the value of $top_child_total (this is done at the end of this file)
 				</a>
 			{{/if}}
 			{{if $item.lock}}
-				<span class="navicon lock fakelink" onClick="lockview(event,{{$item.id}});" title="{{$item.lock}}" data-toggle="tooltip">
+				<span class="navicon lock fakelink" onClick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}" data-toggle="tooltip">
 					&nbsp;<small><i class="fa fa-lock" aria-hidden="true"></i></small>
 				</span>
 			{{/if}}
diff --git a/view/theme/quattro/templates/photo_view.tpl b/view/theme/quattro/templates/photo_view.tpl
index 1ce336b0a6..11947643c6 100644
--- a/view/theme/quattro/templates/photo_view.tpl
+++ b/view/theme/quattro/templates/photo_view.tpl
@@ -16,7 +16,7 @@
         | <a id="photo-toprofile-link" href="{{$tools.profile.0}}">{{$tools.profile.1}}</a>
     {{/if}}
     {{if $tools.lock}}
-        | <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event,'photo/{{$id}}');" />
+        | <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event, 'photo', {{$id}});" />
     {{/if}}
 {{/if}}
 </div>
diff --git a/view/theme/quattro/templates/search_item.tpl b/view/theme/quattro/templates/search_item.tpl
index 0e4aaaf8f7..cb400ac4f5 100644
--- a/view/theme/quattro/templates/search_item.tpl
+++ b/view/theme/quattro/templates/search_item.tpl
@@ -1,6 +1,6 @@
 <div class="wall-item-decor">
 	{{if $item.star}}<span class="icon s22 star {{$item.isstarred}}" id="starred-{{$item.id}}" title="{{$item.star.starred}}">{{$item.star.starred}}</span>{{/if}}
-	{{if $item.lock}}<span class="icon s22 lock fakelink" onclick="lockview(event,{{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
+	{{if $item.lock}}<span class="icon s22 lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
 	<img id="like-rotator-{{$item.id}}" class="like-rotator" src="images/rotator.gif" alt="{{$item.wait}}" title="{{$item.wait}}" style="display: none;" />
 </div>
 
diff --git a/view/theme/quattro/templates/wall_thread.tpl b/view/theme/quattro/templates/wall_thread.tpl
index e6d8b97547..b6907219ff 100644
--- a/view/theme/quattro/templates/wall_thread.tpl
+++ b/view/theme/quattro/templates/wall_thread.tpl
@@ -24,7 +24,7 @@
 
 <div class="wall-item-decor">
 	{{if $item.star}}<span class="icon s22 star {{$item.isstarred}}" id="starred-{{$item.id}}" title="{{$item.star.starred}}">{{$item.star.starred}}</span>{{/if}}
-	{{if $item.lock}}<span class="icon s22 lock fakelink" onclick="lockview(event,{{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
+	{{if $item.lock}}<span class="icon s22 lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
 	<img id="like-rotator-{{$item.id}}" class="like-rotator" src="images/rotator.gif" alt="{{$item.wait}}" title="{{$item.wait}}" style="display: none;" />
 </div>
 
diff --git a/view/theme/smoothly/templates/search_item.tpl b/view/theme/smoothly/templates/search_item.tpl
index 23af1b794b..b3ae01eb4e 100644
--- a/view/theme/smoothly/templates/search_item.tpl
+++ b/view/theme/smoothly/templates/search_item.tpl
@@ -18,7 +18,7 @@
 			<div class="wall-item-location" id="wall-item-location-{{$item.id}}">{{if $item.location}}<span class="icon globe"></span>{{$item.location nofilter}} {{/if}}</div>
 		</div>
 		<div class="wall-item-lock-wrapper">
-				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event,{{$item.id}});" /></div>
+				{{if $item.lock}}<div class="wall-item-lock"><img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event, 'item', {{$item.id}});" /></div>
 				{{else}}<div class="wall-item-lock"></div>{{/if}}			
 		</div>
 		<div class="wall-item-tools" id="wall-item-tools-{{$item.id}}">
diff --git a/view/theme/smoothly/templates/wall_thread.tpl b/view/theme/smoothly/templates/wall_thread.tpl
index 85d480c31f..4ff34aed11 100644
--- a/view/theme/smoothly/templates/wall_thread.tpl
+++ b/view/theme/smoothly/templates/wall_thread.tpl
@@ -42,7 +42,7 @@
 		<div class="wall-item-lock-wrapper">
 			{{if $item.lock}}
 			<div class="wall-item-lock">
-			<img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event,{{$item.id}});" />
+			<img src="images/lock_icon.gif" class="lockview" alt="{{$item.lock}}" onclick="lockview(event, 'item', {{$item.id}});" />
 			</div>
 			{{else}}
 			<div class="wall-item-lock"></div>
diff --git a/view/theme/vier/templates/photo_item.tpl b/view/theme/vier/templates/photo_item.tpl
index f0b7a60890..94e9232af5 100644
--- a/view/theme/vier/templates/photo_item.tpl
+++ b/view/theme/vier/templates/photo_item.tpl
@@ -11,7 +11,7 @@
 			<a href="{{$profile_url}}" target="redir" title="{{$linktitle}}" class="wall-item-name-link"><span class="wall-item-name{{$sparkle}}">{{$name}}</span></a>
 			<span class="wall-item-ago">
 				{{if $plink}}<a class="link" title="{{$plink.title}}" href="{{$plink.href}}" style="color: #999">{{$ago}}</a>{{else}} {{$ago}} {{/if}}
-				{{if $lock}}<span class="fakelink" style="color: #999" onclick="lockview(event,{{$id}});">{{$lock}}</span> {{/if}}
+				{{if $lock}}<span class="fakelink" style="color: #999" onclick="lockview(event, 'item', {{$id}});">{{$lock}}</span> {{/if}}
 			</span>
 		</div>
 		<div class="wall-item-content">
diff --git a/view/theme/vier/templates/photo_view.tpl b/view/theme/vier/templates/photo_view.tpl
index f70ec5b561..87501c031a 100644
--- a/view/theme/vier/templates/photo_view.tpl
+++ b/view/theme/vier/templates/photo_view.tpl
@@ -17,7 +17,7 @@
 		| <a id="photo-toprofile-link" href="{{$tools.profile.0}}">{{$tools.profile.1}}</a>
 	{{/if}}
 	{{if $tools.lock}}
-		| <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event,'photo/{{$id}}');" />
+		| <img src="images/lock_icon.gif" class="lockview" alt="{{$tools.lock}}" onclick="lockview(event, 'photo', {{$id}});" />
 	{{/if}}
 {{/if}}
 </div>
diff --git a/view/theme/vier/templates/search_item.tpl b/view/theme/vier/templates/search_item.tpl
index 1813dd49f6..1da18b0867 100644
--- a/view/theme/vier/templates/search_item.tpl
+++ b/view/theme/vier/templates/search_item.tpl
@@ -2,7 +2,7 @@
 
 <div class="wall-item-decor">
 	{{if $item.star}}<span class="icon star {{$item.isstarred}}" id="starred-{{$item.id}}" title="{{$item.star.starred}}">{{$item.star.starred}}</span>{{/if}}
-	{{if $item.lock}}<span class="icon lock fakelink" onclick="lockview(event,{{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
+	{{if $item.lock}}<span class="icon lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
 	<img id="like-rotator-{{$item.id}}" class="like-rotator" src="images/rotator.gif" alt="{{$item.wait}}" title="{{$item.wait}}" style="display: none;" />
 </div>
 
@@ -25,7 +25,7 @@
 			<a href="{{$item.profile_url}}" target="redir" title="{{$item.linktitle}}" class="wall-item-name-link"><span class="wall-item-name{{$item.sparkle}}">{{$item.name}}</span></a>
 			<span class="wall-item-ago">
 				{{if $item.plink}}<a class="link" title="{{$item.plink.title}}" href="{{$item.plink.href}}" style="color: #999">{{$item.ago}}</a>{{else}} {{$item.ago}} {{/if}}
-				{{if $item.lock}}<span class="fakelink" style="color: #999" onclick="lockview(event,{{$item.id}});">{{$item.lock}}</span> {{/if}}
+				{{if $item.lock}}<span class="fakelink" style="color: #999" onclick="lockview(event, 'item', {{$item.id}});">{{$item.lock}}</span> {{/if}}
 			</span>
 		</div>
 		<div class="wall-item-content">
diff --git a/view/theme/vier/templates/wall_thread.tpl b/view/theme/vier/templates/wall_thread.tpl
index 04ea7b424d..430a6943e3 100644
--- a/view/theme/vier/templates/wall_thread.tpl
+++ b/view/theme/vier/templates/wall_thread.tpl
@@ -65,7 +65,7 @@
 				{{/if}}
 				<span class="pinned">{{$item.pinned}}</span>
 			</span>
-			{{if $item.lock}}<span class="icon s10 lock fakelink" onclick="lockview(event,{{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
+			{{if $item.lock}}<span class="icon s10 lock fakelink" onclick="lockview(event, 'item', {{$item.id}});" title="{{$item.lock}}">{{$item.lock}}</span>{{/if}}
 			<span class="wall-item-network" title="{{$item.app}}">
 				{{$item.network_name}}
 			</span>