mirror of
https://github.com/home-assistant/core.git
synced 2025-07-23 13:17:32 +00:00
Use loader.async_suggest_report_issue in stt (#101390)
This commit is contained in:
parent
0c40c8465e
commit
589fd58137
@ -26,6 +26,7 @@ from homeassistant.helpers import config_validation as cv
|
|||||||
from homeassistant.helpers.entity_component import EntityComponent
|
from homeassistant.helpers.entity_component import EntityComponent
|
||||||
from homeassistant.helpers.restore_state import RestoreEntity
|
from homeassistant.helpers.restore_state import RestoreEntity
|
||||||
from homeassistant.helpers.typing import ConfigType
|
from homeassistant.helpers.typing import ConfigType
|
||||||
|
from homeassistant.loader import async_suggest_report_issue
|
||||||
from homeassistant.util import dt as dt_util, language as language_util
|
from homeassistant.util import dt as dt_util, language as language_util
|
||||||
|
|
||||||
from .const import (
|
from .const import (
|
||||||
@ -264,7 +265,7 @@ class SpeechToTextView(HomeAssistantView):
|
|||||||
raise HTTPBadRequest(text=str(err)) from err
|
raise HTTPBadRequest(text=str(err)) from err
|
||||||
|
|
||||||
if not provider_entity:
|
if not provider_entity:
|
||||||
stt_provider = self._get_provider(provider)
|
stt_provider = self._get_provider(hass, provider)
|
||||||
|
|
||||||
# Check format
|
# Check format
|
||||||
if not stt_provider.check_metadata(metadata):
|
if not stt_provider.check_metadata(metadata):
|
||||||
@ -297,7 +298,7 @@ class SpeechToTextView(HomeAssistantView):
|
|||||||
raise HTTPNotFound()
|
raise HTTPNotFound()
|
||||||
|
|
||||||
if not provider_entity:
|
if not provider_entity:
|
||||||
stt_provider = self._get_provider(provider)
|
stt_provider = self._get_provider(hass, provider)
|
||||||
|
|
||||||
return self.json(
|
return self.json(
|
||||||
{
|
{
|
||||||
@ -321,7 +322,7 @@ class SpeechToTextView(HomeAssistantView):
|
|||||||
}
|
}
|
||||||
)
|
)
|
||||||
|
|
||||||
def _get_provider(self, provider: str) -> Provider:
|
def _get_provider(self, hass: HomeAssistant, provider: str) -> Provider:
|
||||||
"""Get provider.
|
"""Get provider.
|
||||||
|
|
||||||
Method for legacy providers.
|
Method for legacy providers.
|
||||||
@ -331,7 +332,7 @@ class SpeechToTextView(HomeAssistantView):
|
|||||||
|
|
||||||
if not self._legacy_provider_reported:
|
if not self._legacy_provider_reported:
|
||||||
self._legacy_provider_reported = True
|
self._legacy_provider_reported = True
|
||||||
report_issue = self._suggest_report_issue(provider, stt_provider)
|
report_issue = self._suggest_report_issue(hass, provider, stt_provider)
|
||||||
# This should raise in Home Assistant Core 2023.9
|
# This should raise in Home Assistant Core 2023.9
|
||||||
_LOGGER.warning(
|
_LOGGER.warning(
|
||||||
"Provider %s (%s) is using a legacy implementation, "
|
"Provider %s (%s) is using a legacy implementation, "
|
||||||
@ -344,19 +345,13 @@ class SpeechToTextView(HomeAssistantView):
|
|||||||
|
|
||||||
return stt_provider
|
return stt_provider
|
||||||
|
|
||||||
def _suggest_report_issue(self, provider: str, provider_instance: object) -> str:
|
def _suggest_report_issue(
|
||||||
|
self, hass: HomeAssistant, provider: str, provider_instance: object
|
||||||
|
) -> str:
|
||||||
"""Suggest to report an issue."""
|
"""Suggest to report an issue."""
|
||||||
report_issue = ""
|
return async_suggest_report_issue(
|
||||||
if "custom_components" in type(provider_instance).__module__:
|
hass, integration_domain=provider, module=type(provider_instance).__module__
|
||||||
report_issue = "report it to the custom integration author."
|
)
|
||||||
else:
|
|
||||||
report_issue = (
|
|
||||||
"create a bug report at "
|
|
||||||
"https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue"
|
|
||||||
)
|
|
||||||
report_issue += f"+label%3A%22integration%3A+{provider}%22"
|
|
||||||
|
|
||||||
return report_issue
|
|
||||||
|
|
||||||
|
|
||||||
def _metadata_from_header(request: web.Request) -> SpeechMetadata:
|
def _metadata_from_header(request: web.Request) -> SpeechMetadata:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user