| Subject: | In my opinion, package based only on... |
|---|
| Summary: | Package rating comment |
|---|
| Messages: | 1 |
|---|
| Author: | Rad Paluszak |
|---|
| Date: | 2015-07-20 17:14:06 |
|---|
| |
|
|
Rad Paluszak rated this package as follows:
| Utility: | Insufficient |
|---|
| Consistency: | Sufficient |
|---|
|
|
 Rad Paluszak - 2015-07-20 17:14:06 In my opinion, package based only on cookies is not enough. You should implement some other procedures as cookies are to easy to get rid off.
Moreover where you have set_cookie($cookiename, /* ... */, $writedb), this $writedb is so desired to be boolean parametr with default value:
set_cookie($cookiename, /* ... */, $writedb = false)
so then when there is if($writedb=="yes") you'd just have if($writedb). Using "yes" is not neat.
|
