Modified SignupController to pass vars into view

emiliodeg · niden · commit 3869174ba042 · 2018-11-22T11:36:51.000-05:00
Added users list
Added some styles
diff --git a/app/controllers/IndexController.php b/app/controllers/IndexController.php
@@ -4,8 +4,11 @@
 
 class IndexController extends Controller
 {
-
+    /**
+     * Welcome and user list
+     */
     public function indexAction()
     {
+        $this->view->users = Users::find();
     }
 }
diff --git a/app/controllers/SignupController.php b/app/controllers/SignupController.php
@@ -4,14 +4,18 @@
 
 class SignupController extends Controller
 {
-
+    /**
+     * Show form to register a new user
+     */
     public function indexAction()
     {
     }
 
+    /**
+     * Register new user and show message
+     */
     public function registerAction()
     {
-
         $user = new Users();
 
         // Store and check for errors
@@ -20,15 +24,16 @@ public function registerAction()
             ['name', 'email']
         );
 
+        // passing the result to the view
+        $this->view->success = $success;
+
         if ($success) {
-            echo "Thanks for registering!";
+            $message = "Thanks for registering!";
         } else {
-            echo "Sorry, the following problems were generated: ";
-            foreach ($user->getMessages() as $message) {
-                echo $message->getMessage(), "<br/>";
-            }
+            $message = "Sorry, the following problems were generated:<br>" . implode('<br>', $user->getMessages());
         }
 
-        $this->view->disable();
+        // passing a message to the view
+        $this->view->message = $message;
     }
 }
diff --git a/app/views/index.phtml b/app/views/index.phtml
@@ -0,0 +1,13 @@
+<!doctype html>
+<html lang="en">
+<head>
+    <meta charset="UTF-8">
+    <title>Phalcon Tutorial</title>
+    <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css">
+</head>
+<body>
+<div class="container">
+    <?php echo $this->getContent(); ?>
+</div>
+</body>
+</html>
diff --git a/app/views/index/index.phtml b/app/views/index/index.phtml
@@ -2,4 +2,32 @@
 
 echo "<h1>Hello!</h1>";
 
-echo $this->tag->linkTo("signup", "Sign Up Here!");
+echo $this->tag->linkTo(["signup", "Sign Up Here!", 'class' => 'btn btn-primary']);
+
+if ($users->count() > 0) {
+    ?>
+    <table class="table table-bordered table-hover">
+        <thead class="thead-light">
+        <tr>
+            <th>#</th>
+            <th>Name</th>
+            <th>Email</th>
+        </tr>
+        </thead>
+        <tfoot>
+        <tr>
+            <td colspan="3">Users quantity: <?php echo $users->count(); ?></td>
+        </tr>
+        </tfoot>
+        <tbody>
+        <?php foreach ($users as $user) { ?>
+            <tr>
+                <td><?php echo $user->id; ?></td>
+                <td><?php echo $user->name; ?></td>
+                <td><?php echo $user->email; ?></td>
+            </tr>
+        <?php } ?>
+        </tbody>
+    </table>
+    <?php
+}
diff --git a/app/views/signup/index.phtml b/app/views/signup/index.phtml
@@ -13,7 +13,7 @@
 </p>
 
 <p>
-    <?php echo $this->tag->submitButton("Register") ?>
+    <?php echo $this->tag->submitButton(["Register", 'class' => 'btn btn-primary']) ?>
 </p>
 
 <?php echo $this->tag->endForm() ?>
diff --git a/app/views/signup/register.phtml b/app/views/signup/register.phtml
@@ -0,0 +1,5 @@
+<div class="alert alert-<?php echo $success === true ? 'success' : 'danger'; ?>">
+    <?php echo $message; ?>
+</div>
+
+<?php echo $this->tag->linkTo(['/', 'Go back', 'class' => 'btn btn-primary']); ?>
diff --git a/public/index.php b/public/index.php
@@ -17,8 +17,7 @@
         APP_PATH . '/controllers/',
         APP_PATH . '/models/',
     ]
-)->register()
-;
+)->register();
 
 // Create a DI
 $di = new FactoryDefault();

Original file line number	Diff line number	Diff line change
`@@ -4,8 +4,11 @@`
`4`	`4`
`5`	`5`	`class IndexController extends Controller`
`6`	`6`	`{`
`7`		`-`
	`7`	`+ /**`
	`8`	`+ * Welcome and user list`
	`9`	`+ */`
`8`	`10`	`public function indexAction()`
`9`	`11`	`{`
	`12`	`+ $this->view->users = Users::find();`
`10`	`13`	`}`
`11`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,14 +4,18 @@`
`4`	`4`
`5`	`5`	`class SignupController extends Controller`
`6`	`6`	`{`
`7`		`-`
	`7`	`+ /**`
	`8`	`+ * Show form to register a new user`
	`9`	`+ */`
`8`	`10`	`public function indexAction()`
`9`	`11`	`{`
`10`	`12`	`}`
`11`	`13`
	`14`	`+ /**`
	`15`	`+ * Register new user and show message`
	`16`	`+ */`
`12`	`17`	`public function registerAction()`
`13`	`18`	`{`
`14`		`-`
`15`	`19`	`$user = new Users();`
`16`	`20`
`17`	`21`	`// Store and check for errors`
`@@ -20,15 +24,16 @@ public function registerAction()`
`20`	`24`	`['name', 'email']`
`21`	`25`	`);`
`22`	`26`
	`27`	`+ // passing the result to the view`
	`28`	`+ $this->view->success = $success;`
	`29`	`+`
`23`	`30`	`if ($success) {`
`24`		`- echo "Thanks for registering!";`
	`31`	`+ $message = "Thanks for registering!";`
`25`	`32`	`} else {`
`26`		`- echo "Sorry, the following problems were generated: ";`
`27`		`- foreach ($user->getMessages() as $message) {`
`28`		`- echo $message->getMessage(), "<br/>";`
`29`		`- }`
	`33`	`+ $message = "Sorry, the following problems were generated:<br>" . implode('<br>', $user->getMessages());`
`30`	`34`	`}`
`31`	`35`
`32`		`- $this->view->disable();`
	`36`	`+ // passing a message to the view`
	`37`	`+ $this->view->message = $message;`
`33`	`38`	`}`
`34`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,8 +17,7 @@`
`17`	`17`	`APP_PATH . '/controllers/',`
`18`	`18`	`APP_PATH . '/models/',`
`19`	`19`	`]`
`20`		`-)->register()`
`21`		`-;`
	`20`	`+)->register();`
`22`	`21`
`23`	`22`	`// Create a DI`
`24`	`23`	`$di = new FactoryDefault();`