🔧 Fixing the AirDCPP socket issue on AcquisitionPanel on ComicDetails page

src/client/components/AirDCPPSettings/AirDCPPHubsForm.tsx

@@ -10,12 +10,14 @@ export const AirDCPPHubsForm = (airDCPPClientUserSettings): ReactElement => {
   const dispatch = useDispatch();
   const [hubList, setHubList] = useState([]);
   const airDCPPConfiguration = useContext(AirDCPPSocketContext);
-  const { AirDCPPSocket, settings } = airDCPPConfiguration;
+  const {
+    airDCPPState: { settings, socket },
+  } = airDCPPConfiguration;
 
   useEffect(() => {
     (async () => {
       if (!isEmpty(settings)) {
-        const hubs = await AirDCPPSocket.get(`hubs`);
+        const hubs = await socket.get(`hubs`);
         const hubSelectionOptions = hubs.map(({ hub_url, identity }) => ({
           value: hub_url,
           label: identity.name,

src/client/components/AirDCPPSettings/AirDCPPSettingsForm.tsx

@@ -27,7 +27,9 @@ export const AirDCPPSettingsForm = (): ReactElement => {
   }, []);
   const validate = async () => {};
   console.log(airDCPPSettings.airDCPPState);
-  const initFormData = !isUndefined(airDCPPSettings.airDCPPState.settings.directConnect)
+  const initFormData = !isUndefined(
+    airDCPPSettings.airDCPPState.settings.directConnect,
+  )
     ? airDCPPSettings.airDCPPState.settings.directConnect.client.host
     : {};
 

src/client/components/ComicDetail/AcquisitionPanel.tsx

@@ -46,24 +46,27 @@ export const AcquisitionPanel = (
 
   // const settings = useSelector((state: RootState) => state.settings.data);
   const airDCPPConfiguration = useContext(AirDCPPSocketContext);
-  const { AirDCPPSocket, settings } = airDCPPConfiguration;
+
   const dispatch = useDispatch();
   const [dcppQuery, setDcppQuery] = useState({});
 
   useEffect(() => {
-    if (!isNil(settings)) {
+    if (!isEmpty(airDCPPConfiguration.airDCPPState.settings)) {
       // AirDC++ search query
       const dcppSearchQuery = {
         query: {
           pattern: `${sanitizedIssueName.replace(/#/g, "")}`,
           extensions: ["cbz", "cbr", "cb7"],
         },
-        hub_urls: map(settings.directConnect.client.hubs, (item) => item.value),
+        hub_urls: map(
+          airDCPPConfiguration.airDCPPState.settings.directConnect.client.hubs,
+          (item) => item.value,
+        ),
         priority: 5,
       };
       setDcppQuery(dcppSearchQuery);
     }
-  }, []);
+  }, [airDCPPConfiguration]);
 
   const getDCPPSearchResults = useCallback(
     async (searchQuery) => {
@@ -72,17 +75,20 @@ export const AcquisitionPanel = (
           pattern: `${searchQuery.issueName}`,
           extensions: ["cbz", "cbr", "cb7"],
         },
-        hub_urls: map(settings.directConnect.client.hubs, (item) => item.value),
+        hub_urls: map(
+          airDCPPConfiguration.airDCPPState.settings.directConnect.client.hubs,
+          (item) => item.value,
+        ),
         priority: 5,
       };
       dispatch(
-        search(manualQuery, AirDCPPSocket, {
+        search(manualQuery, airDCPPConfiguration.airDCPPState.socket, {
-          username: `${settings.directConnect.client.host.username}`,
+          username: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.username}`,
-          password: `${settings.directConnect.client.host.password}`,
+          password: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.password}`,
         }),
       );
     },
-    [dispatch, AirDCPPSocket],
+    [dispatch, airDCPPConfiguration],
   );
 
   // download via AirDC++
@@ -94,19 +100,23 @@ export const AcquisitionPanel = (
           resultId,
           comicBookObjectId,
           comicObject,
-          AirDCPPSocket,
+          airDCPPConfiguration.airDCPPState.socket,
           {
-            username: `${settings.directConnect.client.host.username}`,
+            username: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.username}`,
-            password: `${settings.directConnect.client.host.password}`,
+            password: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.password}`,
           },
         ),
       );
       // this is to update the download count badge on the downloads tab
       dispatch(
-        getBundlesForComic(comicBookObjectId, AirDCPPSocket, {
+        getBundlesForComic(
-          username: `${settings.directConnect.client.host.username}`,
+          comicBookObjectId,
-          password: `${settings.directConnect.client.host.password}`,
+          airDCPPConfiguration.airDCPPState.socket,
-        }),
+          {
+            username: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.username}`,
+            password: `${airDCPPConfiguration.airDCPPState.settings.directConnect.client.host.password}`,
+          },
+        ),
       );
     },
     [],
@@ -114,7 +124,7 @@ export const AcquisitionPanel = (
   return (
     <>
       <div className="comic-detail columns">
-        {!isEmpty(AirDCPPSocket) ? (
+        {!isEmpty(airDCPPConfiguration.airDCPPState.socket) ? (
           <Form
             onSubmit={getDCPPSearchResults}
             initialValues={{
@@ -185,11 +195,13 @@ export const AcquisitionPanel = (
                 <dl>
                   <dt>
                     <div className="tags mb-1">
-                      {settings.directConnect.client.hubs.map(({ value }) => (
+                      {airDCPPConfiguration.airDCPPState.settings.directConnect.client.hubs.map(
-                        <span className="tag is-warning" key={value}>
+                        ({ value }) => (
-                          {value}
+                          <span className="tag is-warning" key={value}>
-                        </span>
+                            {value}
-                      ))}
+                          </span>
+                        ),
+                      )}
                     </div>
                   </dt>
                   <dt>