[SUCS Devel] [Git][sucssite/sucs-site][sucs-site] Don't need to check if they aren't in the singup table if we know they aren't in the members table

Imran Hussain imranh at sucs.org
Thu Oct 13 14:13:04 BST 2016


Imran Hussain pushed to branch sucs-site at sucssite / sucs-site


Commits:
7eee2a8f by Imran Hussain at 2016-10-13T14:14:45+01:00
Don't need to check if they aren't in the singup table if we know they aren't in the members table

- - - - -


1 changed file:

- components/susignup.php


Changes:

=====================================
components/susignup.php
=====================================
--- a/components/susignup.php
+++ b/components/susignup.php
@@ -52,7 +52,7 @@ if (!empty($_REQUEST['sid']) && !empty($_REQUEST['transactionID'])) {
             $smarty->assign("id", $signuptmpresult->fields["id"]);
             $smarty->assign("pass", $signuptmpresult->fields["password"]);
         // else if they aren't in the SUCS DB, then bootstrap signup process
-        } else if ($tmpresult->fields == false && $signuptmpresult->fields == false) {
+        } else if ($tmpresult->fields == false) {
             $mode = "form";
             $pass = make_password();
             $iddata = $sucsDB->Execute("insert into signup (password,sid,issuedby) values( ?, ?, ?) returning id", array($pass, $sid, "99999"));



View it on GitLab: https://projects.sucs.org/sucssite/sucs-site/commit/7eee2a8fb9959649af1e892975cfc85165889fcc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sucs.org/pipermail/devel/attachments/20161013/2ab1fb4f/attachment.html>


More information about the Devel mailing list