From 4cf328e3b941139f3a41a8a3631f3c74544eb094 Mon Sep 17 00:00:00 2001
From: Michael Gapczynski <mtgap@owncloud.com>
Date: Wed, 1 May 2013 21:15:32 -0400
Subject: [PATCH 1/5] Don't return links for shared files if disabled

---
 lib/public/share.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/public/share.php b/lib/public/share.php
index 1b6f5d05f1..2847f9e5ec 100644
--- a/lib/public/share.php
+++ b/lib/public/share.php
@@ -960,6 +960,10 @@ class Share {
 				$queryArgs = array($itemType);
 			}
 		}
+		if (\OC_Appconfig::getValue('core', 'shareapi_allow_links', 'yes') !== 'yes') {
+			$where .= ' AND `share_type` != ?';
+			$queryArgs[] = self::SHARE_TYPE_LINK;
+		}
 		if (isset($shareType)) {
 			// Include all user and group items
 			if ($shareType == self::$shareTypeUserAndGroups && isset($shareWith)) {

From a2c80bbc687af788d8a5a3c520a24ac223ddc714 Mon Sep 17 00:00:00 2001
From: Michael Gapczynski <mtgap@owncloud.com>
Date: Thu, 2 May 2013 11:45:12 -0400
Subject: [PATCH 2/5] Don't show links option in dropdown if disabled

---
 core/js/share.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/core/js/share.js b/core/js/share.js
index 270eab356e..734bbbfe0b 100644
--- a/core/js/share.js
+++ b/core/js/share.js
@@ -203,7 +203,12 @@ OC.Share={
 			html += '<input id="shareWith" type="text" placeholder="'+t('core', 'Share with')+'" />';
 			html += '<ul id="shareWithList">';
 			html += '</ul>';
-			if (link) {
+			var linksAllowed = false;
+			$.ajax({type: 'GET', url: OC.filePath('core', 'ajax', 'appconfig.php'), data: { action:'getValue', app:'core', key:'shareapi_allow_links', defaultValue:'yes' }, async: false, success: function(result) {
+				if (result && result.status === 'success' && result.data === 'yes') {
+					linksAllowed = true;
+				}
+			}});
 				html += '<div id="link">';
 				html += '<input type="checkbox" name="linkCheckbox" id="linkCheckbox" value="1" /><label for="linkCheckbox">'+t('core', 'Share with link')+'</label>';
 				html += '<br />';

From 7f9ff51308eaa2a32314bf45198487e479500b9d Mon Sep 17 00:00:00 2001
From: Michael Gapczynski <mtgap@owncloud.com>
Date: Thu, 2 May 2013 11:59:11 -0400
Subject: [PATCH 3/5] Fix accidental deletion of if

---
 core/js/share.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/core/js/share.js b/core/js/share.js
index 734bbbfe0b..12c35511fb 100644
--- a/core/js/share.js
+++ b/core/js/share.js
@@ -209,6 +209,7 @@ OC.Share={
 					linksAllowed = true;
 				}
 			}});
+			if (link && linksAllowed) {
 				html += '<div id="link">';
 				html += '<input type="checkbox" name="linkCheckbox" id="linkCheckbox" value="1" /><label for="linkCheckbox">'+t('core', 'Share with link')+'</label>';
 				html += '<br />';

From 0febccee49faa63beff1de253957a193a904fd4a Mon Sep 17 00:00:00 2001
From: Morris Jobke <morris.jobke@gmail.com>
Date: Fri, 18 Oct 2013 10:36:08 +0200
Subject: [PATCH 4/5] readability: single line -> multiple lines

---
 core/js/share.js | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/core/js/share.js b/core/js/share.js
index 12c35511fb..25126d3c23 100644
--- a/core/js/share.js
+++ b/core/js/share.js
@@ -204,11 +204,17 @@ OC.Share={
 			html += '<ul id="shareWithList">';
 			html += '</ul>';
 			var linksAllowed = false;
-			$.ajax({type: 'GET', url: OC.filePath('core', 'ajax', 'appconfig.php'), data: { action:'getValue', app:'core', key:'shareapi_allow_links', defaultValue:'yes' }, async: false, success: function(result) {
-				if (result && result.status === 'success' && result.data === 'yes') {
-					linksAllowed = true;
+			$.ajax({
+				type: 'GET',
+				url: OC.filePath('core', 'ajax', 'appconfig.php'),
+				data: { action:'getValue', app:'core', key:'shareapi_allow_links', defaultValue:'yes' },
+				async: false,
+				success: function(result) {
+					if (result && result.status === 'success' && result.data === 'yes') {
+						linksAllowed = true;
+					}
 				}
-			}});
+			});
 			if (link && linksAllowed) {
 				html += '<div id="link">';
 				html += '<input type="checkbox" name="linkCheckbox" id="linkCheckbox" value="1" /><label for="linkCheckbox">'+t('core', 'Share with link')+'</label>';

From 1f1fcc61298e1ce8b32a16824da0e44671bb30f5 Mon Sep 17 00:00:00 2001
From: Morris Jobke <morris.jobke@gmail.com>
Date: Fri, 18 Oct 2013 11:37:13 +0200
Subject: [PATCH 5/5] Remove "link" shares right after disabling the
 "allow_link_share" setting

---
 apps/files_sharing/appinfo/app.php    |  2 ++
 apps/files_sharing/lib/maintainer.php | 44 +++++++++++++++++++++++++++
 lib/private/appconfig.php             |  6 ++++
 lib/public/share.php                  | 12 ++++++++
 4 files changed, 64 insertions(+)
 create mode 100644 apps/files_sharing/lib/maintainer.php

diff --git a/apps/files_sharing/appinfo/app.php b/apps/files_sharing/appinfo/app.php
index ffdcbf0510..bdaea64bb9 100644
--- a/apps/files_sharing/appinfo/app.php
+++ b/apps/files_sharing/appinfo/app.php
@@ -8,6 +8,7 @@ OC::$CLASSPATH['OC\Files\Cache\Shared_Permissions'] = 'files_sharing/lib/permiss
 OC::$CLASSPATH['OC\Files\Cache\Shared_Updater'] = 'files_sharing/lib/updater.php';
 OC::$CLASSPATH['OC\Files\Cache\Shared_Watcher'] = 'files_sharing/lib/watcher.php';
 OC::$CLASSPATH['OCA\Files\Share\Api'] = 'files_sharing/lib/api.php';
+OC::$CLASSPATH['OCA\Files\Share\Maintainer'] = 'files_sharing/lib/maintainer.php';
 OCP\Util::connectHook('OC_Filesystem', 'setup', '\OC\Files\Storage\Shared', 'setup');
 OCP\Share::registerBackend('file', 'OC_Share_Backend_File');
 OCP\Share::registerBackend('folder', 'OC_Share_Backend_Folder', 'file');
@@ -17,3 +18,4 @@ OCP\Util::addScript('files_sharing', 'share');
 \OC_Hook::connect('OC_Filesystem', 'post_rename', '\OC\Files\Cache\Shared_Updater', 'renameHook');
 \OC_Hook::connect('OCP\Share', 'post_shared', '\OC\Files\Cache\Shared_Updater', 'shareHook');
 \OC_Hook::connect('OCP\Share', 'pre_unshare', '\OC\Files\Cache\Shared_Updater', 'shareHook');
+\OC_Hook::connect('OC_Appconfig', 'post_set_value', '\OCA\Files\Share\Maintainer', 'configChangeHook');
diff --git a/apps/files_sharing/lib/maintainer.php b/apps/files_sharing/lib/maintainer.php
new file mode 100644
index 0000000000..bbb3268410
--- /dev/null
+++ b/apps/files_sharing/lib/maintainer.php
@@ -0,0 +1,44 @@
+<?php
+/**
+ * ownCloud
+ *
+ * @author Morris Jobke
+ * @copyright 2013 Morris Jobke morris.jobke@gmail.com
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU AFFERO GENERAL PUBLIC LICENSE for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public
+ * License along with this library.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+namespace OCA\Files\Share;
+
+/**
+ * Maintains stuff around the sharing functionality
+ *
+ * for example: on disable of "allow links" it removes all link shares
+ */
+
+class Maintainer {
+
+	/**
+	 * Keeps track of the "allow links" config setting
+	 * and removes all link shares if the config option is set to "no"
+	 *
+	 * @param array with app, key, value as named values
+	 */
+	static public function configChangeHook($params) {
+		if($params['app'] === 'core' && $params['key'] === 'shareapi_allow_links' && $params['value'] === 'no') {
+			\OCP\Share::removeAllLinkShares();
+		}
+	}
+
+}
diff --git a/lib/private/appconfig.php b/lib/private/appconfig.php
index e615d83817..4f170e054e 100644
--- a/lib/private/appconfig.php
+++ b/lib/private/appconfig.php
@@ -134,6 +134,12 @@ class OC_Appconfig{
 				.' WHERE `appid` = ? AND `configkey` = ?' );
 			$query->execute( array( $value, $app, $key ));
 		}
+		// TODO where should this be documented?
+		\OC_Hook::emit('OC_Appconfig', 'post_set_value', array(
+			'app' => $app,
+			'key' => $key,
+			'value' => $value
+		));
 	}
 
 	/**
diff --git a/lib/public/share.php b/lib/public/share.php
index 2847f9e5ec..59150e1964 100644
--- a/lib/public/share.php
+++ b/lib/public/share.php
@@ -1721,6 +1721,18 @@ class Share {
 		}
 	}
 
+	/**
+	 * Delete all shares with type SHARE_TYPE_LINK
+	 */
+	public static function removeAllLinkShares() {
+		// Delete any link shares
+		$query = \OC_DB::prepare('SELECT `id` FROM `*PREFIX*share` WHERE `share_type` = ?');
+		$result = $query->execute(array(self::SHARE_TYPE_LINK));
+		while ($item = $result->fetchRow()) {
+			self::delete($item['id']);
+		}
+	}
+
 	/**
 	* Hook Listeners
 	*/