Raise issue "does not support platform setup" (home-assistant#93585)

gjohansson-ST · web-flow · commit c6feb30c311a · 2023-05-28T21:53:32.000+02:00
* Raise issue on platform missing

* Modify issue

* Remove deprecation

* Fix strings

* Strings

* Last strings

* strings to common
diff --git a/homeassistant/components/alarm_control_panel/strings.json b/homeassistant/components/alarm_control_panel/strings.json
@@ -62,5 +62,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/binary_sensor/strings.json b/homeassistant/components/binary_sensor/strings.json
@@ -314,5 +314,11 @@
     "smoke": "smoke",
     "sound": "sound",
     "vibration": "vibration"
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/button/strings.json b/homeassistant/components/button/strings.json
@@ -18,5 +18,11 @@
     "update": {
       "name": "Update"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/calendar/strings.json b/homeassistant/components/calendar/strings.json
@@ -32,5 +32,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/camera/strings.json b/homeassistant/components/camera/strings.json
@@ -34,5 +34,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/climate/strings.json b/homeassistant/components/climate/strings.json
@@ -103,5 +103,11 @@
         "temperature": { "name": "Target temperature" }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/cover/strings.json b/homeassistant/components/cover/strings.json
@@ -76,5 +76,11 @@
     "window": {
       "name": "Window"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/date/strings.json b/homeassistant/components/date/strings.json
@@ -4,5 +4,11 @@
     "_": {
       "name": "[%key:component::date::title%]"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/device_tracker/strings.json b/homeassistant/components/device_tracker/strings.json
@@ -41,5 +41,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/fan/strings.json b/homeassistant/components/fan/strings.json
@@ -52,5 +52,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/humidifier/strings.json b/homeassistant/components/humidifier/strings.json
@@ -62,5 +62,11 @@
     "humidifier": {
       "name": "[%key:component::humidifier::entity_component::_::name%]"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/light/strings.json b/homeassistant/components/light/strings.json
@@ -86,5 +86,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/lock/strings.json b/homeassistant/components/lock/strings.json
@@ -34,5 +34,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/media_player/strings.json b/homeassistant/components/media_player/strings.json
@@ -159,5 +159,11 @@
     "receiver": {
       "name": "Receiver"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/number/strings.json b/homeassistant/components/number/strings.json
@@ -154,5 +154,11 @@
     "wind_speed": {
       "name": "[%key:component::sensor::entity_component::wind_speed::name%]"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/remote/strings.json b/homeassistant/components/remote/strings.json
@@ -24,5 +24,11 @@
         "on": "[%key:common::state::on%]"
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/select/strings.json b/homeassistant/components/select/strings.json
@@ -24,5 +24,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/sensor/strings.json b/homeassistant/components/sensor/strings.json
@@ -267,5 +267,11 @@
     "wind_speed": {
       "name": "Wind speed"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/siren/strings.json b/homeassistant/components/siren/strings.json
@@ -13,5 +13,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/switch/strings.json b/homeassistant/components/switch/strings.json
@@ -30,5 +30,11 @@
     "outlet": {
       "name": "Outlet"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/text/strings.json b/homeassistant/components/text/strings.json
@@ -27,5 +27,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/time/strings.json b/homeassistant/components/time/strings.json
@@ -4,5 +4,11 @@
     "_": {
       "name": "[%key:component::time::title%]"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/update/strings.json b/homeassistant/components/update/strings.json
@@ -14,5 +14,11 @@
     "firmware": {
       "name": "firmware"
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/vacuum/strings.json b/homeassistant/components/vacuum/strings.json
@@ -28,5 +28,11 @@
         "returning": "Returning to dock"
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/water_heater/strings.json b/homeassistant/components/water_heater/strings.json
@@ -18,5 +18,11 @@
         "performance": "Performance"
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/components/weather/strings.json b/homeassistant/components/weather/strings.json
@@ -62,5 +62,11 @@
         }
       }
     }
+  },
+  "issues": {
+    "platform_integration_no_support": {
+      "title": "[%key:common::issues::platform_integration_no_support_title%]",
+      "description": "[%key:common::issues::platform_integration_no_support_description%]"
+    }
   }
 }
diff --git a/homeassistant/helpers/entity_platform.py b/homeassistant/helpers/entity_platform.py
@@ -44,6 +44,7 @@
 from .device_registry import DeviceRegistry
 from .entity_registry import EntityRegistry, RegistryEntryDisabler, RegistryEntryHider
 from .event import async_call_later, async_track_time_interval
+from .issue_registry import IssueSeverity, async_create_issue
 from .typing import ConfigType, DiscoveryInfoType
 
 if TYPE_CHECKING:
@@ -211,6 +212,19 @@ async def async_setup(
                 self.platform_name,
                 self.domain,
             )
+            async_create_issue(
+                self.hass,
+                self.domain,
+                f"platform_integration_no_support_{self.domain}_{self.platform_name}",
+                is_fixable=False,
+                severity=IssueSeverity.ERROR,
+                translation_key="platform_integration_no_support",
+                translation_placeholders={
+                    "domain": self.domain,
+                    "platform": self.platform_name,
+                },
+            )
+
             return
 
         @callback
diff --git a/homeassistant/strings.json b/homeassistant/strings.json
@@ -87,6 +87,10 @@
         "unknown_authorize_url_generation": "Unknown error generating an authorize URL.",
         "cloud_not_connected": "Not connected to Home Assistant Cloud."
       }
+    },
+    "issues": {
+      "platform_integration_no_support_title": "Platform support not supported",
+      "platform_integration_no_support_description": "The {platform} platform for the {domain} integration does not support platform setup.\n\nPlease remove it from your configuration and restart Home Assistant to fix this issue."
     }
   }
 }
diff --git a/tests/helpers/test_entity_platform.py b/tests/helpers/test_entity_platform.py
@@ -14,6 +14,7 @@
     device_registry as dr,
     entity_platform,
     entity_registry as er,
+    issue_registry as ir,
 )
 from homeassistant.helpers.entity import (
     DeviceInfo,
@@ -1458,7 +1459,9 @@ async def test_override_restored_entities(
 
 
 async def test_platform_with_no_setup(
-    hass: HomeAssistant, caplog: pytest.LogCaptureFixture
+    hass: HomeAssistant,
+    caplog: pytest.LogCaptureFixture,
+    issue_registry: ir.IssueRegistry,
 ) -> None:
     """Test setting up a platform that does not support setup."""
     entity_platform = MockEntityPlatform(
@@ -1471,6 +1474,12 @@ async def test_platform_with_no_setup(
         "The mock-platform platform for the mock-integration integration does not support platform setup."
         in caplog.text
     )
+    issue = issue_registry.async_get_issue(
+        domain="mock-integration",
+        issue_id="platform_integration_no_support_mock-integration_mock-platform",
+    )
+    assert issue
+    assert issue.translation_key == "platform_integration_no_support"
 
 
 async def test_platforms_sharing_services(hass: HomeAssistant) -> None:

Original file line number	Diff line number	Diff line change
`@@ -62,5 +62,11 @@`
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`	`}`
	`65`	`+ },`
	`66`	`+ "issues": {`
	`67`	`+ "platform_integration_no_support": {`
	`68`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`69`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`70`	`+ }`
`65`	`71`	`}`
`66`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -314,5 +314,11 @@`
`314`	`314`	`"smoke": "smoke",`
`315`	`315`	`"sound": "sound",`
`316`	`316`	`"vibration": "vibration"`
	`317`	`+ },`
	`318`	`+ "issues": {`
	`319`	`+ "platform_integration_no_support": {`
	`320`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`321`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`322`	`+ }`
`317`	`323`	`}`
`318`	`324`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,5 +18,11 @@`
`18`	`18`	`"update": {`
`19`	`19`	`"name": "Update"`
`20`	`20`	`}`
	`21`	`+ },`
	`22`	`+ "issues": {`
	`23`	`+ "platform_integration_no_support": {`
	`24`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`25`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`26`	`+ }`
`21`	`27`	`}`
`22`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,5 +32,11 @@`
`32`	`32`	`}`
`33`	`33`	`}`
`34`	`34`	`}`
	`35`	`+ },`
	`36`	`+ "issues": {`
	`37`	`+ "platform_integration_no_support": {`
	`38`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`39`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`40`	`+ }`
`35`	`41`	`}`
`36`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,5 +34,11 @@`
`34`	`34`	`}`
`35`	`35`	`}`
`36`	`36`	`}`
	`37`	`+ },`
	`38`	`+ "issues": {`
	`39`	`+ "platform_integration_no_support": {`
	`40`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`41`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`42`	`+ }`
`37`	`43`	`}`
`38`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -103,5 +103,11 @@`
`103`	`103`	`"temperature": { "name": "Target temperature" }`
`104`	`104`	`}`
`105`	`105`	`}`
	`106`	`+ },`
	`107`	`+ "issues": {`
	`108`	`+ "platform_integration_no_support": {`
	`109`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`110`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`111`	`+ }`
`106`	`112`	`}`
`107`	`113`	`}`
Original file line number	Diff line number	Diff line change
`@@ -76,5 +76,11 @@`
`76`	`76`	`"window": {`
`77`	`77`	`"name": "Window"`
`78`	`78`	`}`
	`79`	`+ },`
	`80`	`+ "issues": {`
	`81`	`+ "platform_integration_no_support": {`
	`82`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`83`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`84`	`+ }`
`79`	`85`	`}`
`80`	`86`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,5 +4,11 @@`
`4`	`4`	`"_": {`
`5`	`5`	`"name": "[%key:component::date::title%]"`
`6`	`6`	`}`
	`7`	`+ },`
	`8`	`+ "issues": {`
	`9`	`+ "platform_integration_no_support": {`
	`10`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`11`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`12`	`+ }`
`7`	`13`	`}`
`8`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,5 +41,11 @@`
`41`	`41`	`}`
`42`	`42`	`}`
`43`	`43`	`}`
	`44`	`+ },`
	`45`	`+ "issues": {`
	`46`	`+ "platform_integration_no_support": {`
	`47`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`48`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`49`	`+ }`
`44`	`50`	`}`
`45`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,5 +52,11 @@`
`52`	`52`	`}`
`53`	`53`	`}`
`54`	`54`	`}`
	`55`	`+ },`
	`56`	`+ "issues": {`
	`57`	`+ "platform_integration_no_support": {`
	`58`	`+ "title": "[%key:common::issues::platform_integration_no_support_title%]",`
	`59`	`+ "description": "[%key:common::issues::platform_integration_no_support_description%]"`
	`60`	`+ }`
`55`	`61`	`}`
`56`	`62`	`}`