From 306b39ab7829edb48cdf18a18c7688ea6eec57ca Mon Sep 17 00:00:00 2001
From: jvelilla <javier.hector@gmail.com>
Date: Thu, 21 May 2015 16:46:06 -0300
Subject: [PATCH] Updated code based on Jocelyn's suggestions.

---
 modules/node/handler/node_page_builder.e      | 25 ++++++++++++-------
 ...ge_builder.e => node_pagination_builder.e} |  7 ++----
 2 files changed, 18 insertions(+), 14 deletions(-)
 rename modules/node/handler/{page_builder.e => node_pagination_builder.e} (92%)

diff --git a/modules/node/handler/node_page_builder.e b/modules/node/handler/node_page_builder.e
index f6f7715..923caa2 100644
--- a/modules/node/handler/node_page_builder.e
+++ b/modules/node/handler/node_page_builder.e
@@ -1,6 +1,5 @@
 note
-	description: "Summary description for {NODE_PAGE_BUILDER}."
-	author: ""
+	description: "Paginator builder for CMS nodes."
 	date: "$Date$"
 	revision: "$Revision$"
 
@@ -9,28 +8,36 @@ class
 
 inherit
 
-	PAGE_BUILDER [CMS_NODE]
-		rename
-			make as page_make
+	NODE_PAGINATION_BUILDER [CMS_NODE]
+
+	CMS_NODE_HANDLER
+		redefine
+			make
 		end
 create
-
 	make
 
 feature {NONE} -- Initialization
 
 	make (a_api: CMS_API; a_module_api: CMS_NODE_API)
+			-- Create an object.
 		do
-			page_make (a_api, a_module_api)
+			Precursor (a_api, a_module_api)
 			limit := 5
 			offset := 0
+		ensure then
+			limit_set: limit = 5
+			offset_set: offset = 0
 		end
 
 feature -- Pager
 
-	list: LIST[CMS_NODE]
+	list: LIST [CMS_NODE]
+			-- <Precursor>.
 		do
-			create {ARRAYED_LIST[CMS_NODE]}Result.make (0)
+				--NOTE: the current implementation does not use
+				-- order by and ordering.
+			create {ARRAYED_LIST [CMS_NODE]} Result.make (0)
 			Result := node_api.recent_nodes (offset.as_integer_32, limit.as_integer_32)
 		end
 
diff --git a/modules/node/handler/page_builder.e b/modules/node/handler/node_pagination_builder.e
similarity index 92%
rename from modules/node/handler/page_builder.e
rename to modules/node/handler/node_pagination_builder.e
index 61319c6..25fbbf8 100644
--- a/modules/node/handler/page_builder.e
+++ b/modules/node/handler/node_pagination_builder.e
@@ -1,13 +1,10 @@
 note
-	description: "Generic Page Builder Interface"
+	description: "Generic Node Pagination Builder Interface"
 	date: "$Date$"
 	revision: "$Revision$"
 
 deferred class
-	PAGE_BUILDER [G->CMS_NODE]
-
-inherit
-	CMS_NODE_HANDLER
+	NODE_PAGINATION_BUILDER [G]
 
 feature -- Access