Rename controller methods and adjust routing

Signed-off-by: MB-Finski <sami.finnila@gmail.com>
nextcloud · Dec 21, 2023 · ad8dd89 · ad8dd89
1 parent edb729d
commit ad8dd89
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/appinfo/routes.php b/appinfo/routes.php
@@ -6,9 +6,9 @@
 		['name' => 'config#setConfig', 'url' => '/config', 'verb' => 'PUT'],
 		['name' => 'config#setAdminConfig', 'url' => '/admin-config', 'verb' => 'PUT'],
 
-		['name' => 'assistant#getTaskResultPage', 'url' => '/t/{taskId}', 'verb' => 'GET'],
-		['name' => 'assistant#runTask', 'url' => '/run', 'verb' => 'POST'],
-		['name' => 'assistant#runOrScheduleTask', 'url' => '/run-or-schedule', 'verb' => 'POST'],
+		['name' => 'assistant#getTextProcessingTaskResultPage', 'url' => '/t/{taskId}', 'verb' => 'GET'],
+		['name' => 'assistant#runTextProcessingTask', 'url' => '/run', 'verb' => 'POST'],
+		['name' => 'assistant#runOrScheduleTextProcessingTask', 'url' => '/run-or-schedule', 'verb' => 'POST'],
 
 		['name' => 'Text2Image#processPrompt', 'url' => '/i/process_prompt', 'verb' => 'POST'],
 		['name' => 'Text2Image#getPromptHistory', 'url' => '/i/prompt_history', 'verb' => 'GET'],

diff --git a/lib/Controller/AssistantController.php b/lib/Controller/AssistantController.php
@@ -34,7 +34,7 @@ public function __construct(
 	#[NoAdminRequired]
 	#[NoCSRFRequired]
 	#[BruteForceProtection(action: 'taskResultPage')]
-	public function getTaskResultPage(int $taskId): TemplateResponse {
+	public function getTextProcessingTaskResultPage(int $taskId): TemplateResponse {
 		$task = $this->assistantService->getTextProcessingTask($this->userId, $taskId);
 		if ($task === null) {
 			$response = new TemplateResponse(
@@ -59,7 +59,7 @@ public function getTaskResultPage(int $taskId): TemplateResponse {
 	 * @return DataResponse
 	 */
 	#[NoAdminRequired]
-	public function runTask(string $type, string $input, string $appId, string $identifier): DataResponse {
+	public function runTextProcessingTask(string $type, string $input, string $appId, string $identifier): DataResponse {
 		try {
 			$task = $this->assistantService->runTextProcessingTask($type, $input, $appId, $this->userId, $identifier);
 		} catch (\Exception | \Throwable $e) {
@@ -78,7 +78,7 @@ public function runTask(string $type, string $input, string $appId, string $iden
 	 * @return DataResponse
 	 */
 	#[NoAdminRequired]
-	public function runOrScheduleTask(string $type, string $input, string $appId, string $identifier): DataResponse {
+	public function runOrScheduleTextProcessingTask(string $type, string $input, string $appId, string $identifier): DataResponse {
 		try {
 			$task = $this->assistantService->runOrScheduleTextProcessingTask($type, $input, $appId, $this->userId, $identifier);
 		} catch (\Exception | \Throwable $e) {

diff --git a/lib/Notification/Notifier.php b/lib/Notification/Notifier.php
@@ -89,7 +89,7 @@ public function prepare(INotification $notification, string $languageCode): INot
 					? $l->t('Task for "%1$s" has finished', [$schedulingAppName])
 					: $l->t('"%1$s" task for "%2$s" has finished', [$taskTypeName, $schedulingAppName]);
 				$content = $l->t('Input: %1$s', [$params['input']]);
-				$link = $params['target'] ?? $this->url->linkToRouteAbsolute(Application::APP_ID . '.assistant.getTaskResultPage', ['taskId' => $params['id']]);
+				$link = $params['target'] ?? $this->url->linkToRouteAbsolute(Application::APP_ID . '.assistant.getTextProcessingTaskResultPage', ['taskId' => $params['id']]);
 				$iconUrl = $this->url->getAbsoluteURL($this->url->imagePath(Application::APP_ID, 'app-dark.svg'));
 
 				$notification
@@ -114,7 +114,7 @@ public function prepare(INotification $notification, string $languageCode): INot
 					? $l->t('Task for "%1$s" has failed', [$schedulingAppName])
 					: $l->t('"%1$s" task for "%2$s" has failed', [$taskTypeName, $schedulingAppName]);
 				$content = $l->t('Input: %1$s', [$params['input']]);
-				$link = $params['target'] ?? $this->url->linkToRouteAbsolute(Application::APP_ID . '.assistant.getTaskResultPage', ['taskId' => $params['id']]);
+				$link = $params['target'] ?? $this->url->linkToRouteAbsolute(Application::APP_ID . '.assistant.getTextProcessingTaskResultPage', ['taskId' => $params['id']]);
 				$iconUrl = $this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/error.svg'));
 
 				$notification