apply suggestion

Co-authored-by: Bill Thornton <thornbill@users.noreply.github.com>

src/apps/experimental/features/details/hooks/api/useGetItemByType.ts

@@ -13,8 +13,9 @@ const getItemByType = async (
     options?: AxiosRequestConfig
 ) => {
     const { api, user } = apiContext;
-    if (!api) throw new Error('No API instance available');
+
-    if (!user?.Id) throw new Error('No User ID provided');
+    if (!api) throw new Error('[getItemByType] No API instance available');
+    if (!user?.Id) throw new Error('[getItemByType] No User ID provided');
 
     let response;
     switch (itemType) {

src/hooks/api/libraryHooks/useGetDownload.ts

@@ -10,29 +10,25 @@ const getDownload = async (
     options?: AxiosRequestConfig
 ) => {
     const { api, user } = apiContext;
-    if (!api) throw new Error('No API instance available');
-    if (!user?.Id) throw new Error('No User ID provided');
 
-    const response = await getLibraryApi(api).getDownload(
+    if (!api) throw new Error('[getDownload] No API instance available');
-        params,
+    if (!user?.Id) throw new Error('[getDownload] No User ID provided');
-        options
+
-    );
+    const response = await getLibraryApi(api).getDownload(params, options);
     return response.data;
 };
 
 export const getDownloadQuery = (
     apiContext: JellyfinApiContext,
     params: LibraryApiGetDownloadRequest
-) => queryOptions({
+) =>
-    queryKey: ['Download', params.itemId],
+    queryOptions({
-    queryFn: ({ signal }) =>
+        queryKey: ['Download', params.itemId],
-        getDownload(apiContext, params, { signal }),
+        queryFn: ({ signal }) => getDownload(apiContext, params, { signal }),
-    enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.itemId
+        enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.itemId
-});
+    });
 
-export const useGetDownload = (
+export const useGetDownload = (params: LibraryApiGetDownloadRequest) => {
-    params: LibraryApiGetDownloadRequest
-) => {
     const apiContext = useApi();
     return useQuery(getDownloadQuery(apiContext, params));
 };

src/hooks/api/liveTvHooks/useCancelSeriesTimer.ts

@@ -8,10 +8,11 @@ const cancelSeriesTimer = async (
     params: LiveTvApiCancelSeriesTimerRequest
 ) => {
     const { api } = apiContext;
-    if (api) {
+
-        const response = await getLiveTvApi(api).cancelSeriesTimer(params);
+    if (!api) throw new Error('[cancelSeriesTimer] No API instance available');
-        return response.data;
+
-    }
+    const response = await getLiveTvApi(api).cancelSeriesTimer(params);
+    return response.data;
 };
 
 export const useCancelSeriesTimer = () => {

src/hooks/api/liveTvHooks/useCancelTimer.ts

@@ -8,10 +8,11 @@ const cancelTimer = async (
     params: LiveTvApiCancelTimerRequest
 ) => {
     const { api } = apiContext;
-    if (api) {
+
-        const response = await getLiveTvApi(api).cancelTimer(params);
+    if (!api) throw new Error('[cancelTimer] No API instance available');
-        return response.data;
+
-    }
+    const response = await getLiveTvApi(api).cancelTimer(params);
+    return response.data;
 };
 
 export const useCancelTimer = () => {

src/hooks/api/liveTvHooks/useGetChannel.ts

@@ -10,8 +10,9 @@ const getChannel = async (
     options?: AxiosRequestConfig
 ) => {
     const { api, user } = apiContext;
-    if (!api) throw new Error('No API instance available');
+
-    if (!user?.Id) throw new Error('No User ID provided');
+    if (!api) throw new Error('[getChannel] No API instance available');
+    if (!user?.Id) throw new Error('[getChannel] No User ID provided');
 
     const response = await getLiveTvApi(api).getChannel(
         {
@@ -26,15 +27,15 @@ const getChannel = async (
 export const getChannelQuery = (
     apiContext: JellyfinApiContext,
     params: LiveTvApiGetChannelRequest
-) => queryOptions({
+) =>
-    queryKey: ['Channel', params.channelId],
+    queryOptions({
-    queryFn: ({ signal }) => getChannel(apiContext, params, { signal }),
+        queryKey: ['Channel', params.channelId],
-    enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.channelId
+        queryFn: ({ signal }) => getChannel(apiContext, params, { signal }),
-});
+        enabled:
+            !!apiContext.api && !!apiContext.user?.Id && !!params.channelId
+    });
 
-export const useGetChannel = (
+export const useGetChannel = (params: LiveTvApiGetChannelRequest) => {
-    params: LiveTvApiGetChannelRequest
-) => {
     const apiContext = useApi();
     return useQuery(getChannelQuery(apiContext, params));
 };

src/hooks/api/liveTvHooks/useGetSeriesTimer.ts

@@ -10,25 +10,26 @@ const getSeriesTimer = async (
     options?: AxiosRequestConfig
 ) => {
     const { api } = apiContext;
-    if (!api) throw new Error('No API instance available');
-    const response = await getLiveTvApi(api).getSeriesTimer(
-        params,
-        options
-    );
 
+    if (!api) throw new Error('[getSeriesTimer] No API instance available');
+
+    const response = await getLiveTvApi(api).getSeriesTimer(params, options);
     return response.data;
 };
 
 export const getSeriesTimerQuery = (
     apiContext: JellyfinApiContext,
     params: LiveTvApiGetSeriesTimerRequest
-) => queryOptions({
+) =>
-    queryKey: ['SeriesTimer', params.timerId],
+    queryOptions({
-    queryFn: ({ signal }) => getSeriesTimer(apiContext, params, { signal }),
+        queryKey: ['SeriesTimer', params.timerId],
-    enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.timerId
+        queryFn: ({ signal }) => getSeriesTimer(apiContext, params, { signal }),
-});
+        enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.timerId
+    });
 
-export const useGetSeriesTimer = (requestParameters: LiveTvApiGetSeriesTimerRequest) => {
+export const useGetSeriesTimer = (
+    requestParameters: LiveTvApiGetSeriesTimerRequest
+) => {
     const apiContext = useApi();
     return useQuery(getSeriesTimerQuery(apiContext, requestParameters));
 };

src/hooks/api/liveTvHooks/useGetTimer.ts

@@ -10,23 +10,22 @@ const getTimer = async (
     options?: AxiosRequestConfig
 ) => {
     const { api } = currentApi;
-    if (!api) throw new Error('No API instance available');
-    const response = await getLiveTvApi(api).getTimer(
-        params,
-        options
-    );
 
+    if (!api) throw new Error('[getTimer] No API instance available');
+
+    const response = await getLiveTvApi(api).getTimer(params, options);
     return response.data;
 };
 
 export const getTimerQuery = (
     apiContext: JellyfinApiContext,
     params: LiveTvApiGetTimerRequest
-) => queryOptions({
+) =>
-    queryKey: ['Timer', params.timerId],
+    queryOptions({
-    queryFn: ({ signal }) => getTimer(apiContext, params, { signal }),
+        queryKey: ['Timer', params.timerId],
-    enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.timerId
+        queryFn: ({ signal }) => getTimer(apiContext, params, { signal }),
-});
+        enabled: !!apiContext.api && !!apiContext.user?.Id && !!params.timerId
+    });
 
 export const useGetTimer = (requestParameters: LiveTvApiGetTimerRequest) => {
     const apiContext = useApi();

src/hooks/api/videosHooks/useDeleteAlternateSources.ts

@@ -8,10 +8,11 @@ const deleteAlternateSources = async (
     params: VideosApiDeleteAlternateSourcesRequest
 ) => {
     const { api } = apiContext;
-    if (api) {
+
-        const response = await getVideosApi(api).deleteAlternateSources(params);
+    if (!api) throw new Error('[deleteAlternateSources] No API instance available');
-        return response.data;
+
-    }
+    const response = await getVideosApi(api).deleteAlternateSources(params);
+    return response.data;
 };
 
 export const useDeleteAlternateSources = () => {

src/strings/en-us.json

@@ -1045,8 +1045,8 @@
     "MessageAreYouSureDeleteSubtitles": "Are you sure you wish to delete this subtitle file?",
     "MessageAreYouSureYouWishToRemoveMediaFolder": "Are you sure you wish to remove this media folder?",
     "MessageBrowsePluginCatalog": "Browse our plugin catalog to view available plugins.",
-    "MessageCancelSeriesTimerError": "An error occurred while cancel series timer",
+    "MessageCancelSeriesTimerError": "An error occurred while canceling the series timer",
-    "MessageCancelTimerError": "An error occurred while cancel timer",
+    "MessageCancelTimerError": "An error occurred while canceling the timer",
     "MessageChangeRecordingPath": "Changing your recording folder will not migrate existing recordings from the old location to the new. You'll need to move them manually if desired.",
     "MessageConfirmAppExit": "Do you want to exit?",
     "MessageConfirmDeleteGuideProvider": "Are you sure you wish to delete this guide provider?",
@@ -1099,7 +1099,7 @@
     "MessageRenameMediaFolder": "Renaming a media library will cause all metadata to be lost, proceed with caution.",
     "MessageRepositoryInstallDisclaimer": "WARNING: Installing a third party plugin repository carries risks. It may contain unstable or malicious code, and may change at any time. Only install repositories from authors that you trust.",
     "MessageSent": "Message sent.",
-    "MessageSplitVersionsError": "An error occurred while split versions",
+    "MessageSplitVersionsError": "An error occurred while splitting versions",
     "MessageSyncPlayCreateGroupDenied": "Permission required to create a group.",
     "MessageSyncPlayDisabled": "SyncPlay disabled.",
     "MessageSyncPlayEnabled": "SyncPlay enabled.",