From d77cc20da62d15cb4a69878469dd2bf5e16e68a0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim=20Daubensch=C3=BCtz?= <tim@ascribe.io>
Date: Fri, 22 May 2015 12:58:06 +0200
Subject: [PATCH] Make Pagination more generic

---
 js/components/pagination.js        | 34 ++++++++---------------
 js/components/pagination_button.js | 44 ++++++++++++++++++++++++++++++
 js/components/piece_list.js        |  7 ++++-
 3 files changed, 61 insertions(+), 24 deletions(-)
 create mode 100644 js/components/pagination_button.js

diff --git a/js/components/pagination.js b/js/components/pagination.js
index 35a3d726..9ace55af 100644
--- a/js/components/pagination.js
+++ b/js/components/pagination.js
@@ -1,14 +1,10 @@
 import React from 'react';
-import Router from 'react-router';
-
-let Link = Router.Link;
 
+import PaginationButton from './pagination_button';
 
 let Pagination = React.createClass({
-
-    goToPage(page) {
-        return () => this.props.fetchList(page, this.props.pageSize, this.props.search,
-                                          this.props.orderBy, this.props.orderAsc);
+    propTypes: {
+        goToPage: React.PropTypes.func.isRequired
     },
 
     render() {
@@ -18,22 +14,14 @@ let Pagination = React.createClass({
         return(
             <nav>
                 <ul className="pager">
-                    <li className="previous">
-                        <Link to="pieces"
-                              query={{page: prev}}
-                              onClick={this.goToPage(prev)}>
-                            <span aria-hidden="true">&larr;</span>
-                            Previous
-                        </Link>
-                    </li>
-                    <li className="next">
-                        <Link to="pieces"
-                              query={{page: next}}
-                              onClick={this.goToPage(next)}>
-                            Next
-                            <span aria-hidden="true">&rarr;</span>
-                        </Link>
-                    </li>
+                    <PaginationButton 
+                        direction='previous'
+                        goToPage={this.props.goToPage}>
+                    </PaginationButton>
+                    <PaginationButton 
+                        direction='next'
+                        goToPage={this.props.goToPage}>
+                    </PaginationButton>
                 </ul>
             </nav>
         );
diff --git a/js/components/pagination_button.js b/js/components/pagination_button.js
new file mode 100644
index 00000000..a7488d0a
--- /dev/null
+++ b/js/components/pagination_button.js
@@ -0,0 +1,44 @@
+import React from 'react';
+import Router from 'react-router';
+
+let Link = Router.Link;
+
+let PaginationButton = React.createClass({
+    propTypes: {
+        direction: React.PropTypes.oneOf(['previous', 'next']),
+        goToPage: React.PropTypes.func.isRequired
+    },
+
+    render() {
+        let page = parseInt(this.props.page, 10);
+        let directionDisplay;
+
+        if(this.props.direction === 'previous') {
+            page -= 1;
+            directionDisplay = (
+                <span>
+                    <span aria-hidden="true">&larr;</span> Previous
+                </span>
+                );
+        } else {
+            page += 1;
+            directionDisplay = (
+                <span>
+                    Next <span aria-hidden="true">&rarr;</span>
+                </span>
+                );
+        }
+
+        return (
+            <li className={this.props.direction}>
+                <Link to="pieces"
+                      query={{page}}
+                      onClick={this.props.goToPage(page)}>
+                    {directionDisplay}
+                </Link>
+            </li>
+        );
+    }
+});
+
+export default PaginationButton;
\ No newline at end of file
diff --git a/js/components/piece_list.js b/js/components/piece_list.js
index 22f0ee3e..e4b8af64 100644
--- a/js/components/piece_list.js
+++ b/js/components/piece_list.js
@@ -26,6 +26,11 @@ let PieceList = React.createClass({
                                    this.state.orderBy, this.state.orderAsc);
     },
 
+    paginationGoToPage(page) {
+        return () => PieceListActions.fetchPieceList(page, this.state.pageSize, this.state.search,
+                                          this.state.orderBy, this.state.orderAsc);
+    },
+
     render() {
         let columnList = [
             new TableColumnModel('thumbnail', '', TableItemImg, 2, false),
@@ -49,7 +54,7 @@ let PieceList = React.createClass({
                     }
                 }}>
                 <Table columnList={columnList} />
-                <Pagination currentPage={this.props.query.page} />
+                <Pagination currentPage={this.props.query.page} goToPage={this.paginationGoToPage} />
             </AltContainer>
         );
     }