Merge pull request #1278 from nextcloud/bugfix/noid/empty-remote

4 files changed, 5 insertions, 2 deletions

diff --git a/lib/Capabilities.php b/lib/Capabilities.php
index e8ad527a..3cd1b8c5 100644
--- a/lib/Capabilities.php
+++ b/lib/Capabilities.php
@@ -91,6 +91,7 @@ class Capabilities implements ICapability {
 			$collaboraCapabilities = $this->capabilitiesService->getCapabilities();
 			$this->capabilities = [
 				'richdocuments' => [
+					'version' => \OC::$server->getAppManager()->getAppVersion('richdocuments'),
 					'mimetypes' => self::MIMETYPES,
 					'mimetypesNoDefaultOpen' => self::MIMETYPES_OPTIONAL,
 					'collabora' => $collaboraCapabilities,
diff --git a/lib/Controller/DocumentController.php b/lib/Controller/DocumentController.php
index 3f2071db..f9100bce 100644
--- a/lib/Controller/DocumentController.php
+++ b/lib/Controller/DocumentController.php
@@ -202,7 +202,7 @@ class DocumentController extends Controller {
 						'&richdocuments_remote_access=' . $remote;
 
 						$event = new BeforeFederationRedirectEvent(
-							$item, $relative, $remote
+							$item, $relativeFolderPath, $remote
 						);
 						$eventDispatcher = \OC::$server->getEventDispatcher();
 						$eventDispatcher->dispatch(BeforeFederationRedirectEvent::class, $event);
diff --git a/lib/Db/WopiMapper.php b/lib/Db/WopiMapper.php
index d2cd3842..dbd2f67d 100644
--- a/lib/Db/WopiMapper.php
+++ b/lib/Db/WopiMapper.php
@@ -82,6 +82,8 @@ class WopiMapper extends Mapper {
 			'direct' => $direct,
 			'isRemoteToken' => $isRemoteToken,
 			'templateId' => $templateId,
+			'remoteServer' => '',
+			'remoteServerToken' => '',
 			'share' => $share
 		]);
 
diff --git a/lib/Service/FederationService.php b/lib/Service/FederationService.php
index 1d18fdef..2a06ce5a 100644
--- a/lib/Service/FederationService.php
+++ b/lib/Service/FederationService.php
@@ -196,7 +196,7 @@ class FederationService {
 				} else {
 					$wopi = $this->tokenManager->getRemoteTokenFromDirect($item, $direct->getUid());
 				}
-				$url = $remote . 'index.php/apps/richdocuments/remote?shareToken=' . $item->getStorage()->getToken() .
+				$url = rtrim($remote, '') . '/index.php/apps/richdocuments/remote?shareToken=' . $item->getStorage()->getToken() .
 					'&remoteServer=' . $wopi->getServerHost() .
 					'&remoteServerToken=' . $wopi->getToken();
 				if ($item->getInternalPath() !== '') {