From 0a7e6f09f8e01983a4d962df6cdbead9add66fb5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lo=C3=AFc=20Blot=20=28=40U-Exp=29?=
 <loic.blot@unix-experience.fr>
Date: Tue, 16 Sep 2014 20:54:47 +0200
Subject: [PATCH] Improve mailbox management for navigation

---
 controller/smscontroller.php  | 14 +++++++++++++-
 templates/main.php            |  5 ++++-
 templates/part.navigation.php | 27 ++++++++++-----------------
 3 files changed, 27 insertions(+), 19 deletions(-)

diff --git a/controller/smscontroller.php b/controller/smscontroller.php
index 66ac20a..af37a61 100644
--- a/controller/smscontroller.php
+++ b/controller/smscontroller.php
@@ -35,8 +35,20 @@ class SmsController extends Controller {
 	 * @NoCSRFRequired
 	 */
 	public function index () {
+		$mboxes = array(
+			'PNLConversations' => array(
+				'label' => 'Conversations',
+				'phoneNumbers' => $this->smsMapper->getAllPeersPhoneNumbers($this->userId),
+				'url' => \OCP\Util::linkToAbsolute('ocsms', 'index.php', array('feed' => 'conversations'))
+			),
+			'PNLDrafts' => array(
+				'label' => 'Drafts',
+				'phoneNumbers' => array(),
+				'url' => \OCP\Util::linkToAbsolute('ocsms', 'index.php', array('feed' => 'drafts'))
+			)
+		)
 		$params = array('user' => $this->userId,
-			"PNLConversations" => $this->smsMapper->getAllPeersPhoneNumbers($this->userId)
+			'mailboxes' => $mboxes
 		);
 		return new TemplateResponse($this->appName, 'main', $params);
 	}
diff --git a/templates/main.php b/templates/main.php
index d975252..cb0debf 100644
--- a/templates/main.php
+++ b/templates/main.php
@@ -6,7 +6,10 @@
 <div id="app">
 	<div id="app-navigation">
 		<?php print_unescaped($this->inc('part.navigation')); ?>
-		<?php print_unescaped($this->inc('part.settings')); ?>
+		<?php /*print_unescaped($this->inc('part.settings'));*/ ?>
+	</div>
+
+	<div id="app-mailbox-content">
 	</div>
 
 	<div id="app-content">
diff --git a/templates/part.navigation.php b/templates/part.navigation.php
index f0262b0..4992ab2 100644
--- a/templates/part.navigation.php
+++ b/templates/part.navigation.php
@@ -1,19 +1,12 @@
 <ul>
-	<?php if (count($_["PNLConversations"]) > 0) { ?>
-	<li><a href="#">Conversations</a>
-	<ul>
-	<?php foreach ($_["PNLConversations"] as $number) { ?>
-		<li><?php p($number); ?></li>
-	<?php } ?>
-	</ul></li>
-	<?php }	?>
-	
-	<?php if (count($_["PNLDrafts"]) > 0) { ?>
-	<li><a href="#">Drafts</a>
-	<ul>
-	<?php foreach ($_["PNLDrafts"] as $number) { ?>
-		<li><?php p($number); ?></li>
-	<?php } ?>
-	</ul></li>
-	<?php }	?>
+	<?php foreach ($_['mailboxes'] as $mboxName => $mailbox) { ?>
+		<?php if (count($mailbox['phoneNumbers']) > 0) { ?>
+		<li><a href="<?php p($mailbox['url']); ?>"><?php p($mailbox['label']); ?></a>
+		<ul>
+		<?php foreach ($mailbox['phoneNumbers'] as $number) { ?>
+			<li><?php p($number); ?></li>
+		<?php } ?>
+		</ul></li>
+		<?php }
+	} ?>
 </ul>