[Seaside] GRPharoPlatform>>isIpAddress bug?

Johan Brichau johan at inceptive.be
Tue Sep 20 17:39:13 UTC 2011


Marco,

You should first download the latest configuration of Seaside as well. The one with Pharo probably is not the latest one.
You can find it in the MetacelloRepository on squeaksource.

cheers
Johan

On 20 Sep 2011, at 17:09, Johan Brichau wrote:

> Hi Marco,
> 
> You should update Seaside. This was fixed 6 months ago: http://code.google.com/p/seaside/issues/detail?id=646
> 
> Load the most recent ConfigurationOfSeaside30 and execute 
> ConfigurationOfSeaside30 load
> 
> best regards,
> Johan
> 
> On 20 Sep 2011, at 16:59, Marco A. Gonzalez wrote:
> 
>> Hi,
>> 
>> I think I've discovered an problem in the following class>>method but still considering myself somewhat of a newbie, I'm not entirely sure:
>> 
>> GRPharoPlatform>>isIpAddress: aString
>> 	| ip |
>> 	ip := aString findTokens: '.'.
>> 	^ ip size = 4
>> 		and: [ ip allSatisfy: [ :each | each greaseInteger notNil
>> 			and: [ each greaseInteger notNil between: 0 and: 255] ] ]
>> 
>> 
>> The problem is the last line "each greaseInteger notNil between: 0 and: 255"... where the notNil message returns a true or false, and it does not respond to "between:and:". I believe the fix ought to be to remove the "notNil" in the last line to result in the line:
>> 	 "each greaseInteger between: 0 and: 255" 
>> since the "notNil" is already considered before the "and:".
>> 
>> 
>> This is from the most recent One-click download:
>> 	Pharo1.3
>> 	Latest update: #13307
>> 
>> ------------------------------------------------------
>> Marco A. Gonzalez
>> 
>> 
>> 
>> _______________________________________________
>> seaside mailing list
>> seaside at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> 
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside



More information about the seaside mailing list