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

firebus’s picture

firebus commented
Assigned: Unassigned » firebus
Status: Active » Fixed

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.

sgriffin’s picture

sgriffin commented

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.

firebus’s picture

firebus commented
Status: Fixed » Postponed (maintainer needs more info)

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...)

sgriffin’s picture

sgriffin commented

I downloaded the updated cck ip module just in case, and the new cck. It seems good. No issues.

firebus’s picture

firebus commented
Status: Postponed (maintainer needs more info) » Needs review
firebus’s picture

firebus commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.