Subject: | There are two major problems with... |
Summary: | Package rating comment |
Messages: | 2 |
Author: | Artur Graniszewski |
Date: | 2010-08-19 09:55:44 |
Update: | 2010-08-19 18:32:45 |
|
|
|
Artur Graniszewski rated this package as follows:
Utility: | Insufficient |
Consistency: | Insufficient |
Examples: | Sufficient |
|
Artur Graniszewski - 2010-08-19 09:55:44
There are two major problems with this class:
1) you use $_SESSION for storing and retrieving SQL data, so when you want to clean your cache, you clear entire user session (see session_destroy()) - this is a major problem for most of the homebrew PHP scripts which, and even some advanced (which use custom session handlers and database storage)
2) there is a potential problem with mysql_real_escape, you really should use connection handler in all of your mysql_* functions, because some scripts will be using two or more different MySQL connections (for example with different encodings).
Pashkov Denis Igorevich - 2010-08-19 18:32:45 - In reply to message 1 from Artur Graniszewski
thank u , i will fix this, it was very basic example, so i don't think about complex projects..
|