[SUCS Devel] [Git][sucssite/sucs-site][master] Fix old members returning with the same student id while still being a student.

[Imran Hussain]

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


Commits:
f772dc7a by Imran Hussain at 2016-10-13T10:29:44+01:00
Fix old members returning with the same student id while still being a student.

- - - - -


1 changed file:

- components/susignup.php


Changes:

=====================================
components/susignup.php
=====================================
--- a/components/susignup.php
+++ b/components/susignup.php
@@ -40,19 +40,13 @@ if (!empty($_REQUEST['sid']) && !empty($_REQUEST['transactionID'])) {
         if ($tmpresult->fields["sid"] == $sid && $tmpresult->fields["paid"] == paidUntil(time())) {
             // let them know they are already signed up and renewed
             message_flash("You are a numpty and have already signed up and paid for this year.");
-            // else if check to see if they have signedup and paid for the new year but haven't renewed
+        // else if check to see if they have signedup and paid for the new year but haven't renewed
         } else if ($tmpresult->fields["sid"] == $sid && $tmpresult->fields["paid"] != paidUntil(time())) {
             // renew them!
             renew_membership($tmpresult->fields["username"]);
             // let them know that their account has been renewed
             message_flash("Your SUCS account has been renewed.");
-            // else if they aren't in the SUCS DB but have a signup slip, take them back to that part of signup
-            // I don't know how you would end up in a state like this but the old code dealt with it so I will as well
-        } else if ($tmpresult->fields == false && $signuptmpresult->fields["sid"] == $sid) {
-            $mode = "form";
-            $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 they aren't in the SUCS DB, then bootstrap signup process
         } else if ($tmpresult->fields == false && $signuptmpresult->fields == false) {
             $mode = "form";
             $pass = make_password();



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