| Author |
Message |
blith Client

Joined: Jul 18, 2003 Posts: 977
|
Posted:
Wed Jun 02, 2004 8:00 am |
|
I would like to make it possible to not have to score. Right now you have to choose a numerical value. I would like no score to be an option. Maybe a - (dash) to be no score chosen. Any ideas? Thanks! |
|
|
|
 |
blith Client

Joined: Jul 18, 2003 Posts: 977
|
Posted:
Thu Jun 03, 2004 12:38 pm |
|
Anyone have any thoughts on this? Thanks! |
|
|
|
 |
Raven Site Admin/Owner

Joined: Aug 27, 2002 Posts: 16987 Location: Kansas
|
Posted:
Thu Jun 03, 2004 4:32 pm |
|
Okay, I'm going to hang out the carrot and expect you to get it
in modules/Reviews/index.php you should have a couple instances of | Code: | | <option name=\"score\" value=\"1\">1</option> | Add this line after it | Code: | | <option name=\"score\" value=\"0\">0</option> |
Then, find any code that looks like this (should only be 1) | Code: | | if (($score < 1) || ($score > 10)) { | and modify it to read | Code: | | if (($score < 0) || ($score > 10)) { |
Now I haven't tested this, just glanced at it  |
|
|
|
 |
blith Client

Joined: Jul 18, 2003 Posts: 977
|
Posted:
Fri Jun 04, 2004 7:46 am |
|
That's got it! BUT... I tried some non numerical values and it wouldn't take probably because of the "value" command... so I am going to look into being able to have a - (dash) as a value... maybe a hint on how to do that... |
|
|
|
 |
blith Client

Joined: Jul 18, 2003 Posts: 977
|
Posted:
Fri Jun 04, 2004 7:47 am |
|
Wait I am going to try and figure it out... if I can't then I will be back... trying to fish! |
|
|
|
 |
Raven Site Admin/Owner

Joined: Aug 27, 2002 Posts: 16987 Location: Kansas
|
Posted:
Fri Jun 04, 2004 7:53 am |
|
You really need to leave it as a value to avoid exploits that required using the intval() to fix. If you use alpha-numeric values, you can't use the intval() and will have to have some kind of a security trap. |
|
|
|
 |
blith Client

Joined: Jul 18, 2003 Posts: 977
|
Posted:
Fri Jun 04, 2004 8:03 am |
|
okay. Thank you! whew.. I am soo glad you are on the job!! |
|
|
|
 |
|
|
|
|