added getById() to portForwardStore to eliminate direct items access from outside

Signed-off-by: Jim Ehrismann <jehrismann@mirantis.com>

src/renderer/components/+network-port-forwards/port-forwards.tsx

@@ -55,9 +55,7 @@ export class PortForwards extends React.Component<Props> {
   get selectedPortForward() {
     const { match: { params: { forwardport } } } = this.props;
 
-    return portForwardStore.items.find(pf => {
+    return portForwardStore.getById(forwardport);
-      return pf.getForwardPort() == Number(forwardport);
-    });
   }
 
   onDetails = (item: PortForwardItem) => {

src/renderer/port-forward/port-forward.store.ts

@@ -81,6 +81,16 @@ export class PortForwardStore extends ItemStore<PortForwardItem> {
   async removeSelectedItems() {
     return Promise.all(this.selectedItems.map(removePortForward));
   }
+
+  getById(id: string) {
+    const index = this.getIndexById(id);
+
+    if (index === -1) {
+      return null;
+    }
+    
+    return this.getItems()[index];
+  }
 }
 
 interface PortForwardResult {