Merge pull request #31 from amcquade/update-fixes

Filter duplicate previous feeds, get previous feed when it is clicked

src/App.js

@@ -22,7 +22,8 @@ const App = ({ fetching }) => {
 
   const getFeed = (event) => {
     setFetching((prev) => !prev);
-    event.preventDefault();
+    if (event.preventDefault != null)
+      event.preventDefault();
     const feed_url = event.target.elements.feed_url.value;
     const Parser = require("rss-parser");
     const parser = new Parser({
@@ -44,7 +45,7 @@ const App = ({ fetching }) => {
             program_description: feed.description,
           });
           setFetching((prev) => !prev);
-          setPreviousFeeds([...previousFeeds, feed_url]);
+          setPreviousFeeds([...new Set([...previousFeeds, feed_url])]);
           setPast(true);
 
           return setError(false);

src/components/SearchHistory.jsx

@@ -8,13 +8,15 @@ export default function SearchHistory(props) {
     setAnchorEl(event.currentTarget);
   };
 
-  const handleClose = () => {
+  const handleClose = (event) => {
+    if (event.currentTarget.innerText != '') 
+      props.getFeed({target: {elements: {feed_url: {value: event.currentTarget.innerText}}}});
     setAnchorEl(null);
   };
 
   const renderItem = (item, i) => {
     return (
-      <MenuItem index={Math.random() * i} onClick={handleClose}>
+      <MenuItem key={i} index={i} onClick={handleClose}>
         {item}
       </MenuItem>
     );

src/components/UserForm.jsx

@@ -38,7 +38,7 @@ const UserForm = ({ getFeed, previous_feeds, past }) => {
         >
           Submit
         </Button>
-        {past ? <SearchHistory history={previous_feeds} /> : <div></div>}
+        {past ? <SearchHistory getFeed={getFeed} history={previous_feeds} /> : <div></div>}
       </form>
     </div>
   );