| Subject: | 1) There is no mysql_real_escape()... |
|---|
| Summary: | Package rating comment |
|---|
| Messages: | 2 |
|---|
| Author: | Artur Graniszewski |
|---|
| Date: | 2011-03-04 15:55:31 |
|---|
| Update: | 2011-03-04 21:22:32 |
|---|
| |
|
|
Artur Graniszewski rated this package as follows:
| Utility: | Insufficient |
|---|
| Consistency: | Bad |
|---|
|
|
 Artur Graniszewski - 2011-03-04 15:55:31 1) There is no mysql_real_escape() equivalent in your class
2) Why are you mixing two different languages (english and ?spanish?) in method and property names is beyond me
3) This class should be marked as "Country specific"
4) You are using epreg_match(), which is obsolete since PHP 5.2
5) Why are you needing $query string in protected function fetch($query) ? It's not used in this method at all...
 Marcel - 2011-03-04 21:22:32 - In reply to message 1 from Artur GraniszewskiHello,
1) I've made another class just for do this, because I had problems using real_scape depending on the operating system.
2) Sorry...
3) Sorry...
4) Sorry... again.
5) Mistake! you can remove this.
thanks!
|
