Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion #14

Open
KBischof opened this issue Oct 5, 2011 · 1 comment
Open

Suggestion #14

KBischof opened this issue Oct 5, 2011 · 1 comment

Comments

@KBischof
Copy link

KBischof commented Oct 5, 2011

Ivan, thanks for adding my suggested changes yesterday!

Sorry this is not an issue, but I wanted to make another suggestion for the project's roadmap and figured this was the best way to make this suggestion. I think the project's database calls should be using PDO instead of mysql_query calls. The main reason is that the mysql_ library has been obsolete for six years and less secure than PDO. This article inspired me to begin programming all my latest projects in PDO, you might be interested: http://webdevrefinery.com/forums/topic/1272-your-mysql-code-sucks

I was able to convert the database file to use PDO but got hung up with how it interacts with the session file. I would be willing to send you my converted database.php file if you are interested in pursing this..

Thanks again!

KB

@ivannovak
Copy link
Owner

I like the idea. Go ahead and make it an issue :)


KBischof
mailto:reply@reply.github.com
October 4, 2011 9:59 PM

Ivan, thanks for adding my suggested changes yesterday!

Sorry this is not an issue, but I wanted to make another suggestion
for the project's roadmap and figured this was the best way to make
this suggestion. I think the project's database calls should be using
PDO instead of mysql_query calls. The main reason is that the mysql_
library has been obsolete for six years and less secure than PDO. This
article inspired me to begin programming all my latest projects in
PDO, you might be interested:
http://webdevrefinery.com/forums/topic/1272-your-mysql-code-sucks

I was able to convert the database file to use PDO but got hung up
with how it interacts with the session file. I would be willing to
send you my converted database.php file if you are interested in
pursing this..

Thanks again!

KB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants