Posted by sgriffin on September 10, 2007 at 10:19pm
Jump to:
| Project: | CCK IP Address |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | minor |
| Assigned: | firebus |
| Status: | closed (fixed) |
Issue Summary
When "Reject non-routable IP addresses" is checked..
You MUST enter in a valid ip on your node.
Otherwise, 0.0.0.0 is entered and fails the check.
This makes it so ip address or ip range becomes a required field which shouldn't be the case.
Comments
#1
ugh. i was lazy about comparing 0 to empty string AND i was casting empty strings as '000' with sprintf!
i have this fixed i think and will commit in a moment.
#2
I uploaded your commits for .module and .css but I still receive for a blank range...
Illegal value for range_ip 1, second address. Non-routable IP addresses are not allowed.
I went down from multiple to single and it left some weird ass field after the first ip range. I guess thats it.
Its a label for something I guess. So maybe thats not the issue.
#3
i'll do some more testing in the next week and see if i can reproduce.
can you tell me what version of cck you're using (i doubt that's the issue, but i have discovered some version specific cck issues in the past...)
#4
I downloaded the updated cck ip module just in case, and the new cck. It seems good. No issues.
#5
#6
#7
Automatically closed -- issue fixed for 2 weeks with no activity.