From ca49fc9eb5cecac2d043d602871d802ac3d0243e Mon Sep 17 00:00:00 2001
From: Tom Hacohen <tom@stosb.com>
Date: Thu, 7 Dec 2017 15:17:03 +0000
Subject: [PATCH] Split the addressbook view from the syncentry handling code.

---
 src/AddressBook.tsx        | 42 +++++++++++++++++++++++++++++
 src/JournalAddressBook.tsx | 55 +++++++++++++-------------------------
 2 files changed, 60 insertions(+), 37 deletions(-)
 create mode 100644 src/AddressBook.tsx

diff --git a/src/AddressBook.tsx b/src/AddressBook.tsx
new file mode 100644
index 0000000..35dc2aa
--- /dev/null
+++ b/src/AddressBook.tsx
@@ -0,0 +1,42 @@
+import * as React from 'react';
+import { List, ListItem } from 'material-ui/List';
+
+import * as ICAL from 'ical.js';
+
+class AddressBook extends React.Component {
+  props: {
+    entries: Map<string, ICAL.Component>,
+    onItemClick: (contact: ICAL.Component) => void,
+  };
+
+  render() {
+    let entries = Array.from(this.props.entries.values()).sort((_a, _b) => {
+      const a = _a.getFirstPropertyValue('fn');
+      const b = _b.getFirstPropertyValue('fn');
+
+      if (a < b) {
+        return -1;
+      } else if (a > b) {
+        return 1;
+      } else {
+        return 0;
+      }
+    });
+
+    let itemList = entries.map((entry) => {
+      const uid = entry.getFirstPropertyValue('uid');
+      const name = entry.getFirstPropertyValue('fn');
+      return (
+        <ListItem key={uid} primaryText={name} onClick={() => (this.props.onItemClick(entry))} />
+      );
+    });
+
+    return (
+      <List>
+        {itemList}
+      </List>
+    );
+  }
+}
+
+export default AddressBook;
diff --git a/src/JournalAddressBook.tsx b/src/JournalAddressBook.tsx
index 9e2587b..32df51d 100644
--- a/src/JournalAddressBook.tsx
+++ b/src/JournalAddressBook.tsx
@@ -1,25 +1,34 @@
 import * as React from 'react';
-import { Route, Switch } from 'react-router';
-import { List, ListItem } from 'material-ui/List';
-import { Link } from 'react-router-dom';
+import { Route, Switch, withRouter } from 'react-router';
 
 import * as ICAL from 'ical.js';
 
 import * as EteSync from './api/EteSync';
 
 import { routeResolver } from './App';
+
+import AddressBook from './AddressBook';
 import Contact from './Contact';
 
 class JournalAddressBook extends React.Component {
-  static defaultProps = {
-    prevUid: null,
-  };
-
   props: {
     journal: EteSync.Journal,
     entries: Array<EteSync.SyncEntry>,
+    history?: any,
   };
 
+  constructor(props: any) {
+    super(props);
+    this.contactClicked = this.contactClicked.bind(this);
+  }
+
+  contactClicked(contact: ICAL.Component) {
+    const uid = contact.getFirstPropertyValue('uid');
+
+    this.props.history.push(
+      routeResolver.getRoute('journals._id.items._id', { journalUid: this.props.journal.uid, itemUid: uid }));
+  }
+
   render() {
     if (this.props.journal === undefined) {
       return (<div>Loading</div>);
@@ -40,32 +49,6 @@ class JournalAddressBook extends React.Component {
       }
     }
 
-    let entries = Array.from(items.values()).sort((_a, _b) => {
-      const a = _a.getFirstPropertyValue('fn');
-      const b = _b.getFirstPropertyValue('fn');
-
-      if (a < b) {
-        return -1;
-      } else if (a > b) {
-        return 1;
-      } else {
-        return 0;
-      }
-    });
-
-    let itemList = entries.map((entry) => {
-      const uid = entry.getFirstPropertyValue('uid');
-      const name = entry.getFirstPropertyValue('fn');
-      return (
-        <Link
-          key={uid}
-          to={routeResolver.getRoute('journals._id.items._id', { journalUid: this.props.journal.uid, itemUid: uid })}
-        >
-          <ListItem primaryText={name} />
-        </Link>
-      );
-    });
-
     return (
       <div>
         <Switch>
@@ -73,9 +56,7 @@ class JournalAddressBook extends React.Component {
             path={routeResolver.getRoute('journals._id')}
             exact={true}
             render={() => (
-                <List>
-                  {itemList}
-                </List>
+                <AddressBook entries={items} onItemClick={this.contactClicked} />
               )
             }
           />
@@ -95,4 +76,4 @@ class JournalAddressBook extends React.Component {
   }
 }
 
-export default JournalAddressBook;
+export default withRouter(JournalAddressBook);