Little fixes

(and rebase) Signed-off-by: Thomas Citharel <tcit@tcit.fr>
nextcloud · Aug 6, 2018 · 55158ae · 55158ae
1 parent a75de29
commit 55158ae
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/js/app/controllers/calcontroller.js b/js/app/controllers/calcontroller.js
@@ -26,8 +26,8 @@
 * Description: The fullcalendar controller.
 */
 
-app.controller('CalController', ['$scope', 'Calendar', 'CalendarService', 'VEventService', 'SettingsService', 'TimezoneService', 'VEvent', 'is', 'fc', 'EventsEditorDialogService', 'PopoverPositioningUtility', '$window', 'isPublic', 'constants', 'settings', '$uibModal',
-	function ($scope, Calendar, CalendarService, VEventService, SettingsService, TimezoneService, VEvent, is, fc, EventsEditorDialogService, PopoverPositioningUtility, $window, isPublic, constants, settings) {
+app.controller('CalController', ['$scope', 'Calendar', 'CalendarService', 'VEventService', 'SettingsService', 'TimezoneService', 'VEvent', 'is', 'fc', 'EventsEditorDialogService', 'PopoverPositioningUtility', '$window', 'isPublic', 'constants', 'settings', '$http', '$uibModal',
+	function ($scope, Calendar, CalendarService, VEventService, SettingsService, TimezoneService, VEvent, is, fc, EventsEditorDialogService, PopoverPositioningUtility, $window, isPublic, constants, settings, $http, $uibModal) {
 		'use strict';
 
 		is.loading = true;
@@ -153,9 +153,9 @@ app.controller('CalController', ['$scope', 'Calendar', 'CalendarService', 'VEven
 
 		$scope.importFileByPath = function(path) {
 			return $http.get(
-				OC.linkToRemoteBase('dav') + '/files/' + OC.getCurrentUser().displayName + '/' + decodeURI(path)
+				OC.linkToRemoteBase('dav') + '/files/' + OC.getCurrentUser().uid + '/' + decodeURIComponent(path)
 			).then(function(response) {
-				const fileName = decodeURIComponent(path);
+				const fileName = decodeURIComponent(decodeURI(path)).split('/').pop();
 				const fileBody = response.data;
 				let file = new File([fileBody], fileName, {type: 'text/calendar'});
 
@@ -173,6 +173,8 @@ app.controller('CalController', ['$scope', 'Calendar', 'CalendarService', 'VEven
 					},
 					scope: $scope
 				});
+			}).catch(function() {
+				OC.Notification.showTemporary(t('calendar', 'There was an issue while importing your calendar'));
 			});
 		};
 

diff --git a/js/utility/registerUtility.js b/js/utility/registerUtility.js
@@ -30,7 +30,7 @@ let register = (function () {
 		actionHandler: function(filename, context) {
 			let path = $('#fileList').find('[data-file="'+filename+'"]').data('path');
 			path = path.substring(1);
-			window.location = OC.generateUrl('apps/calendar/#/import/{filename}', {filename: encodeURI(path + '/' + filename)});
+			window.location = OC.generateUrl('apps/calendar/#/import/{filename}', {filename: encodeURIComponent(path + '/' + filename)});
 		},
 		permissions: OC.PERMISSION_READ,
 		icon: function () {

diff --git a/templates/part.import.dialog.php b/templates/part.import.dialog.php
@@ -29,7 +29,7 @@
 		<tbody>
 		<tr ng-repeat="file in files" ng-show="!file.done">
 			<td class="name">
-				<span>{{ file.file.name | limitTo:nameSize }}{{file.file.name.length < nameSize ? '' : '...'}}</span>
+				<span>{{ file.file.name | limitTo:nameSize }}{{ file.file.name.length < nameSize ? '' : '...' }}</span>
 			</td>
 			<td class="calendartype">
 				<span ng-show="file.wasCanceled()">