Skip to content

Commit fdb0200

Browse files
committed
fix(cosmetic): remove unnecessary comments
Signed-off-by: codewithvk <[email protected]>
1 parent 85483d6 commit fdb0200

File tree

3 files changed

+3
-8
lines changed

3 files changed

+3
-8
lines changed

lib/Controller/WopiController.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,7 @@ public function uploadSettingsFile(string $fileId, string $access_token): JSONRe
437437
$wopi = $this->wopiMapper->getWopiForToken($access_token);
438438

439439
$userId = $wopi->getEditorUid();
440-
// TODO: auth - for admin??
440+
441441
$content = fopen('php://input', 'rb');
442442
if (!$content) {
443443
throw new \Exception('Failed to read input stream.');
@@ -985,10 +985,11 @@ private function getWopiUrlForTemplate(Wopi $wopi): string {
985985
$nextcloudUrl = $this->appConfig->getNextcloudUrl() ?: trim($this->urlGenerator->getAbsoluteURL(''), '/');
986986
return $nextcloudUrl . '/index.php/apps/richdocuments/wopi/template/' . $wopi->getTemplateId() . '?access_token=' . $wopi->getToken();
987987
}
988+
988989
private function generateSettingToken(string $userId): string {
989990
return $this->settingsService->generateIframeToken('user', $userId)['token'];
990991
}
991-
// todo extract nextcloud url from everything
992+
992993
private function generateSettings(string $userId, string $type): array {
993994
$nextcloudUrl = $this->appConfig->getNextcloudUrl() ?: trim($this->urlGenerator->getAbsoluteURL(''), '/');
994995
$uri = $nextcloudUrl . '/index.php/apps/richdocuments/wopi/settings' . '?type=' . $type . '&access_token=' . $this->generateSettingToken($userId) . '&fileId=' . '-1';

lib/Service/SettingsService.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,6 @@ public function __construct(
4949
$this->cache = $cacheFactory->createDistributed(Application::APPNAME);
5050
}
5151

52-
// TODO: Implement file caching...
53-
5452
/**
5553
* Ensure the settings directory exists, if it doesn't exist then create it.
5654
*
@@ -322,7 +320,4 @@ public function deleteSettingsFile(string $type, string $category, string $name,
322320
throw $e;
323321
}
324322
}
325-
326-
// TODO: Handle installDefaultSystemFiles setting
327-
328323
}

src/components/CoolFrame.vue

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
<input type="hidden" name="wopi_setting_base_url" :value="wopiSettingBaseUrl">
1414
<input type="hidden" name="iframe_type" :value="iframeType">
1515
<input type="hidden" name="css_variables" :value="cssVariables">
16-
<!-- TODO: Include any other necessary hidden inputs - send css variables -->
1716
</form>
1817
<iframe :id="iframeName"
1918
:name="iframeName"

0 commit comments

Comments
 (0)