PHP Classes

There are two major problems with this class: 1) you use $_S...

Recommend this page to a friend!

      MySQL cache class  >  All threads  >  There are two major problems with...  >  (Un) Subscribe thread alerts  
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

  1. There are two major problems with...   Reply   Report abuse  
Picture of Artur Graniszewski 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).

  2. Re: There are two major problems with...   Reply   Report abuse  
Picture of Pashkov Denis Igorevich 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..