From 1a2cf00b5e1a9c00be823eb655a76f8625bf32b5 Mon Sep 17 00:00:00 2001 From: Malf Furious Date: Sun, 31 Jan 2016 12:47:04 -0500 Subject: Merge Auth MVC, initial_signup action into signup There was a mistake that caused the page notice about no accounts existing to sometimes not showup in error. This merge resolves that issue as well as tidys up the code a bit. --- app/controller/auth.control.php | 8 +------- app/model/auth.mod.php | 12 ++++-------- 2 files changed, 5 insertions(+), 15 deletions(-) diff --git a/app/controller/auth.control.php b/app/controller/auth.control.php index 693d190..9bb8349 100644 --- a/app/controller/auth.control.php +++ b/app/controller/auth.control.php @@ -36,13 +36,7 @@ class Auth extends Controller if ($mod->deflt()) include "view/auth/default.view.php"; else - $this->action_initial_signup($mod); - } - - function action_initial_signup($mod) - { - $mod->initialSignup(); - $this->action_signup($mod); + $this->action_signup($mod); } function action_signup($mod) diff --git a/app/model/auth.mod.php b/app/model/auth.mod.php index aa0adf3..cdf416b 100644 --- a/app/model/auth.mod.php +++ b/app/model/auth.mod.php @@ -22,19 +22,15 @@ class AuthModel extends CommonModel return true; } - /* - * Initial signup action - */ - function initialSignup() - { - $this->noaccounts = true; - } - /* * Signup action */ function signup() { + $userTbl = new User(); + + if (count($userTbl->getAllUsers_orderByName()) == 0) + $this->noaccounts = true; } /* -- cgit v1.2.3