fix: do not add /v1 twice to the service url (#207)

kyteinsky · julien-nc · web-flow · commit 9c272ab669a6 · 2025-04-09T20:37:32.000+05:30
* fix: do not add /v1 twice to the service url

Signed-off-by: Anupam Kumar &lt;kyteinsky@gmail.com&gt;

* Update lib/Migration/Version030104Date20241011192830.php

Co-authored-by: Julien Veyssier &lt;julien-nc@posteo.net&gt;
Signed-off-by: Anupam Kumar &lt;kyteinsky@gmail.com&gt;

* fix: use str_ends_with

Signed-off-by: Anupam Kumar &lt;kyteinsky@gmail.com&gt;

---------

Signed-off-by: Anupam Kumar &lt;kyteinsky@gmail.com&gt;
Co-authored-by: Julien Veyssier &lt;julien-nc@posteo.net&gt;
diff --git a/lib/Migration/Version030104Date20241011192830.php b/lib/Migration/Version030104Date20241011192830.php
@@ -28,7 +28,7 @@ public function __construct(
 	 */
 	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
 		$value = $this->openAiSettingsService->getServiceUrl();
-		if ($value !== '') {
+		if ($value !== '' && !str_ends_with(rtrim($value, '/ '), '/v1')) {
 			$newValue = rtrim($value, '/') . '/v1';
 			$this->openAiSettingsService->setServiceUrl($newValue);
 		}

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public function __construct(`
`28`	`28`	`*/`
`29`	`29`	`public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {`
`30`	`30`	`$value = $this->openAiSettingsService->getServiceUrl();`
`31`		`- if ($value !== '') {`
	`31`	`+ if ($value !== '' && !str_ends_with(rtrim($value, '/ '), '/v1')) {`
`32`	`32`	`$newValue = rtrim($value, '/') . '/v1';`
`33`	`33`	`$this->openAiSettingsService->setServiceUrl($newValue);`
`34`	`34`	`}`