Replace '::' with '_' in schema name

ManageIQ · Jan 31, 2024 · c5ffa6e · c5ffa6e
1 parent 1557f37
commit c5ffa6e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/config/openapi.json b/config/openapi.json
@@ -276,7 +276,7 @@
         },
         "additionalProperties": false
       },
-      "ManageIQ::Providers::CloudManager::AuthKeyPair": {
+      "ManageIQ_Providers_CloudManager_AuthKeyPair": {
         "type": "object",
         "properties": {
           "id": {
@@ -1209,7 +1209,7 @@
         },
         "additionalProperties": false
       },
-      "ManageIQ::Providers::CloudManager::Template": {
+      "ManageIQ_Providers_CloudManager_Template": {
         "type": "object",
         "properties": {
           "id": {
@@ -4256,7 +4256,7 @@
         },
         "additionalProperties": false
       },
-      "ManageIQ::Providers::CloudManager::Vm": {
+      "ManageIQ_Providers_CloudManager_Vm": {
         "type": "object",
         "properties": {
           "id": {

diff --git a/lib/manageiq/api/open_api/generator.rb b/lib/manageiq/api/open_api/generator.rb
@@ -46,9 +46,10 @@ def build_schemas
           ::Api::ApiConfig.collections.each do |_collection_name, collection|
             next unless collection.klass
 
-            model = collection.klass.constantize
+            model       = collection.klass.constantize
+            schema_name = model.name.gsub("::", "_")
 
-            schemas[model.name] = {
+            schemas[schema_name] = {
               "type"                 => "object",
               "properties"           => build_schema_properties(model),
               "additionalProperties" => false