From: Michael Gapczynski <GapczynskiM@gmail.com>
Date: Tue, 15 May 2012 00:09:10 +0000 (-0400)
Subject: Fix private link emailing and clean up file name displayed in email
X-Git-Tag: v4.0.0RC2~51
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cda9dcc7aaa701cec214122ae7127767d23af2f4;p=nextcloud-server.git

Fix private link emailing and clean up file name displayed in email
---

diff --git a/apps/files_sharing/ajax/email.php b/apps/files_sharing/ajax/email.php
index 9eba203465e..823462b9921 100644
--- a/apps/files_sharing/ajax/email.php
+++ b/apps/files_sharing/ajax/email.php
@@ -3,9 +3,9 @@ OCP\JSON::checkLoggedIn();
 OCP\JSON::checkAppEnabled('files_sharing');
 $user = OCP\USER::getUser();
 // TODO translations
-$subject = $user + ' ' + 'shared a file with you';
-$link = $_POST['link'] + '&f=' + $_POST['f'];
-$text = $user + ' ' + 'shared the file' + ' ' + $_POST['f'] + ' ' + 'with you.' + ' ' + 'It is available for download here:' + ' ' + $link;
+$subject = $user.' shared a file with you';
+$link = $_POST['link'];
+$text = $user.' shared the file '.$_POST['file'].' with you. It is available for download here: '.$link;
 $fromaddress = OCP\Config::getUserValue($user, 'settings', 'email', 'sharing-noreply@'.$_SERVER['HTTP_HOST']);
 OC_Mail::send($_POST['toaddress'], $_POST['toaddress'], $subject, $text, $fromaddress, $user);
 
diff --git a/apps/files_sharing/js/share.js b/apps/files_sharing/js/share.js
index dbf5588c4e3..ac1209956d6 100644
--- a/apps/files_sharing/js/share.js
+++ b/apps/files_sharing/js/share.js
@@ -204,7 +204,9 @@ OC.Share={
 		$('#email').animate({ fontWeight: 'normal' }, 2000, function() {
 			$(this).val('');
 		}).val('Email sent');
-		$.post(OC.filePath('files_sharing', 'ajax', 'email.php'), 'toaddress='+$('#email').val()+'&link='+$('#link').val());
+		var link = $('#privateLinkText').val();
+		var file = link.substr(link.lastIndexOf('/') + 1).replace(/%20/g, ' ');
+		$.post(OC.filePath('files_sharing', 'ajax', 'email.php'), { toaddress: $('#email').val(), link: link, file: file } );
 	},
 	dirname:function(path) {
 		return path.replace(/\\/g,'/').replace(/\/[^\/]*$/, '');