Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
  • Sign in
S
sucs-site
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 25
    • Issues 25
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Security & Compliance
    • Security & Compliance
    • Dependency List
    • License Compliance
  • Analytics
    • Analytics
    • CI / CD
    • Code Review
    • Insights
    • Issues
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • sucssite
  • sucs-site
  • Issues
  • #33

Closed
Open
Opened Oct 13, 2016 by Imran Hussain@imranh
  • Report abuse
  • New issue
Report abuse New issue

SUSignup code can't handle people (student numbers) being in the signup table more than once

So they became a full member a few years back, got their account deleted now is still a student and has paid via the su and wants to signup again.

The code goes with the assumption that sid (student number) is a primary key in that table but in reality it's signupid+sid as the primary key.

So when it does a search for them $signuptmpresult = $sucsDB->Execute("SELECT * FROM signup WHERE sid=?",array($sid)); it returns more than one row of data so bits of code like $signuptmpresult->fields["sid"] == $sid will blow up.

My suggestion would be to write a foreach loop and look for a place where in the feilds activated == NULL and use that.

Of course you have to account for not finding activated == NULL or finding it multiple times etc...

EDIT: all the code you should care about is in components/susignup.php

Linked issues

  • Discussion
  • Designs

The one place for your designs

To enable design management, you'll need to meet the requirements. If you need help, reach out to our support team for assistance.

Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
3
Labels
bug confirmed critical
Assign labels
  • View project labels
Reference: sucssite/sucs-site#33