FIXUP: Trigger scan only on details page

It's too expensive and not useful enough on the overview. OTOH, trigger
a scan on each opening of a WiFi device details page, so that the
information there is current and useful.

This means that we the number of APs in the overview is less reliable,
but that's fine -- after a while NM forgets them anyway.

Curiously the tests (which check "3 networks" on the overview) are still
fine -- apparently NM does a passive scan automatically on startup, or
for a newly appearing device?

#22884 (comment)

Author: martinpitt

pkg/networkmanager/interfaces.js

@@ -1204,18 +1204,7 @@ export function NetworkManagerModel() {
                             console.warn("request_scan: scan failed for", this.Interface + ":", error.toString());
                         });
             }
-        },
-
-        exporters: [
-            function (obj) {
-                // Once we see any WiFi device, trigger one initial scan
-                if (obj.DeviceType === '802-11-wireless' && !priv(obj).wifi_scan_requested) {
-                    utils.debug("triggering initial WiFi scan for", obj.Interface);
-                    priv(obj).wifi_scan_requested = true;
-                    obj.request_scan();
-                }
-            }
-        ]
+        }
     };
 
     // The 'Interface' type does not correspond to any NetworkManager

pkg/networkmanager/network-interface.jsx

@@ -4,7 +4,7 @@
  */
 import cockpit from "cockpit";
 import React, { useContext, useEffect, useRef, useState } from "react";
-import { useEvent } from "hooks";
+import { useEvent, useInit } from "hooks";
 import { Breadcrumb, BreadcrumbItem } from "@patternfly/react-core/dist/esm/components/Breadcrumb/index.js";
 import { Button } from "@patternfly/react-core/dist/esm/components/Button/index.js";
 import { Card, CardBody, CardHeader, CardTitle } from '@patternfly/react-core/dist/esm/components/Card/index.js';
@@ -283,6 +283,14 @@ export const NetworkInterfacePage = ({
 
     const Dialogs = useDialogs();
 
+    // Trigger (passive) scan on page load for wireless devices on page load
+    useInit(() => {
+        if (dev?.DeviceType === '802-11-wireless') {
+            utils.debug("Requesting initial WiFi scan for", dev_name);
+            dev.request_scan();
+        }
+    });
+
     // WiFi scanning: re-enable button when APs change or after timeout
     useEffect(() => {
         if (isScanning) {