MBS-10644: Fix chatgpt azure model registration (#120)

Author: Snl-FGH

classes/base_instance.php

@@ -680,10 +680,6 @@ final public function supported_purposes(): array {
             return [];
         }
         $connector = \core\di::get(connector_factory::class)->get_connector_by_connectorname($this->connector);
-        if (!in_array($this->get_model(), $connector->get_models())) {
-            // This typically is the case if we are using a model that is preconfigured (for example when using Azure).
-            return array_keys($connector->get_models_by_purpose());
-        }
         $purposesofcurrentmodel = [];
         foreach ($connector->get_models_by_purpose() as $purpose => $models) {
             if (in_array($this->get_model(), $models)) {

tools/chatgpt/classes/connector.php

@@ -40,7 +40,7 @@ class connector extends \local_ai_manager\base_connector {
     public function get_models_by_purpose(): array {
         $chatgptmodels =
                 ['gpt-3.5-turbo', 'gpt-4-turbo', 'gpt-4o', 'gpt-4o-mini', 'o1', 'o1-mini', 'o3', 'o3-mini', 'o4-mini'];
-        return [
+        $modelsbypurpose = [
                 'chat' => $chatgptmodels,
                 'feedback' => $chatgptmodels,
                 'singleprompt' => $chatgptmodels,
@@ -52,7 +52,13 @@ public function get_models_by_purpose(): array {
                 'agent' => $chatgptmodels,
         ];
         foreach ($modelsbypurpose as $purpose => $models) {
-            $modelsbypurpose[$purpose][] = aitool_option_azure::get_azure_model_name('chatgpt');
+            // We assume that the azure models support all the purposes. This is kind of a blind guess, because in case
+            // of azure we do not have any information which model lies behind the azure endpoint. So we will go for the less
+            // restrictive definition, but of course it could be that there's a model behind the azure endpoint that, for example,
+            // does not support vision (itt). In this case we let the user run into an error, but that's not avoidable right now.
+            if (!empty($models)) {
+                $modelsbypurpose[$purpose][] = aitool_option_azure::get_azure_model_name('chatgpt');
+            }
         }
         return $modelsbypurpose;
     }

tools/chatgpt/tests/connector_test.php

@@ -0,0 +1,50 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace aitool_chatgpt;
+
+use local_ai_manager\local\aitool_option_azure;
+use local_ai_manager\local\connector_factory;
+
+/**
+ * Tests for ChatGPT connector.
+ *
+ * @package   aitool_chatgpt
+ * @copyright 2026 ISB Bayern
+ * @author    Philipp Memmel
+ * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+final class connector_test extends \advanced_testcase {
+    /**
+     * Test that the Azure model is only available for supported purposes.
+     *
+     * @throws \coding_exception
+     * @covers \aitool_chatgpt\connector::get_models_by_purpose
+     */
+    public function test_get_models_by_purpose_contains_azure_model_only_for_supported_purposes(): void {
+        $connectorfactory = \core\di::get(connector_factory::class);
+        $connector = $connectorfactory->get_connector_by_connectorname('chatgpt');
+        $modelname = aitool_option_azure::get_azure_model_name('chatgpt');
+        $modelsbypurpose = $connector->get_models_by_purpose();
+
+        foreach (['chat', 'feedback', 'singleprompt', 'translate', 'itt', 'questiongeneration', 'agent'] as $purpose) {
+            $this->assertContains($modelname, $modelsbypurpose[$purpose]);
+        }
+
+        $this->assertNotContains($modelname, $modelsbypurpose['tts']);
+        $this->assertNotContains($modelname, $modelsbypurpose['imggen']);
+    }
+}