jacob-hegna/bvswnhs — pull request #11 by jacob-hegna (2014-08-07 04:53:11+00:00)
Title: Edit data in member pages

So it works (theoretically). I'm creating this pull request to discuss issues with the code because I know @killmous will find issues with it. Here are the problems I have identified thus far
- Need to make it so not every user can edit data
- Should make the edit boxes `

`s so that the user can just hit `enter` to update the data
- it is **ugly** code
- ***ugly***
- make sure there aren't any security holes with `edit_sql`
- have there be a placeholder for how to properly format the data being input
- I had to change the `	`s because they are used for what column is affected in sql, we should fix that. I'm thinking an associative array/javascript object to compare the table heading to a sql heading would be cool (this could even be done server side).
- **DATA VERIFICATION**. can't have people put in 2 digit `bvid`s or phone numbers. We could theoretically use one function for all data verification and just pass it an array, it would return a negative number for failure, `0` for success.