Mantis Bugtracker
Main | My View | View Issues | Change Log | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Issue History ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0006730 [Squeak] Etoys minor always 10-14-07 02:46 10-16-07 20:29
Reporter wiz View Status public  
Assigned To
Priority normal Resolution open  
Status new   Product Version
Summary 0006730: GeeMailMorph initialzation causes nil to be used as a dicitonary key
Description nk 11/1/2004 07:48 UndefinedObject initialize {*eToys-*Morphic-customevents-class initialization}'

initialize
    "AlansTextPlusMorph initialize"
    ScriptingSystem addCustomEventFor: self named: #scrolledIntoView help: 'when I am scrolled into view in a GeeMailMorph' targetMorphClass: Morph.
    ScriptingSystem addCustomEventFor: self named: #scrolledOutOfView help: 'when I am scrolled out of view in a GeeMailMorph' targetMorphClass: Morph.
Additional Information Apparently when AlansTextPlusMorph was changed or removed the initialization responsibility was given to the instance side of UndefinedObject.

It clearly doesn't belong there.

I found this when writing tests that check the integrity of Identity dictionary instances.

iDictionary keys breaks when nil is used as a key. An identity set can not have nil as a valid member.

This is also broken in squeakland and olpc images.

So where does this initialization belong?
Or can it be removed entirely?

Yours in curiosity and service, --Jerome Peace
Attached Files  nilKeyFix2-wiz-M6730.3.cs [^] (2,886 bytes) 10-16-07 03:04

- Relationships
related to 0006721feedback  In 7149 Identity dictionarys sometimes can't find their keys 

- Notes
(0011314 - 58 - 82 - 82 - 82 - 82 - 82)
wiz
10-14-07 04:26
Reminder sent to: ScottWallace

Hi Scott,

I could use your help on this.

Cheers -Jer
 
(0011317 - 578 - 648 - 648 - 648 - 648 - 648)
ScottWallace
10-15-07 10:52

 Hi, Jerome.

This was almost certainly simply an editing error made in update 6458-0367specificEvents-nk back in November 2004. The method (UndefinedObject >> initialize) is obviously there in error should simply removed.

In Squeakland and olpc/etoys images, at any rate, the necessary initialization for the custom #scrolledIntoView and #scrolledOutOfView events is lodged in GeeMailMorph's class-side initialization. In any images that do not have GeeMailMorph, the point is presumably moot.

So, again, UndefinedObject >> should undoubtedly be removed.

  -- Scott
 
(0011324 - 139 - 175 - 175 - 175 - 175 - 175)
wiz
10-15-07 18:11

 Hi Scott,

Thanks for your help.

I'll look at the oplc gmail init and take my cues from there.

Yours in curiosity and service, -Jer
 
(0011329 - 429 - 451 - 451 - 451 - 451 - 451)
ScottWallace
10-15-07 21:19

 Jerome: Again, it's really okay, and desirable, simply to remove the method UndefinedObject >> initialize, which obviously should not be there. IMO no need to spend more time trying to delve more deeply into this issue -- the method clearly got compiled into UndefinedObject either because of operator error or because of tools failure, but in any case it simply does not belong there, and should simply be removed.
  -- Scott
 
(0011331 - 716 - 788 - 788 - 788 - 788 - 788)
wiz
10-16-07 03:30

 Hi Scott,

I've got my fix.

I was interested in preserving the GeeMailMorph as a working entity in 3dot10 so I did not want to just remove the code.

I have uploaded a fix with the code moved to class side of Gmail and the unload method added there as well. I don't know if this is all that is needed but here I am just concerned with fixing the nil key problem without taking responsibility for making anything else worse.

I would like at some point to see oplc/etoys/squeakland reconciled with squeakdevs stream of stuff. I don't know if there is a community will or resources to do that but I don't wish to push code that gets us further from the possibility.

Thanks again for your support.

--Jer
 
(0011338 - 639 - 711 - 711 - 711 - 711 - 711)
wiz
10-16-07 20:29

 Hmmm.

I went back and checked the general health of GeeMail in 3dot9 and 3dot10.

GeeMail is not readily findable in 3dot9 (It is not on the widgets flap or in any of the object catagories.)

In 3dot10 it is now back in the widget flap but works wierdly.

Grabbing a GeeMail out of the flap and then dropping a polygon or curvier on it gets a error box. But the object gets imbedded. Then lifting the object out of the morph act wierd. The hand will pick it up but it is displaced way to the top left of the screen.

So it doesn't seem GeeMail is working in 3dot9 or 3dot10 and its existence is better hidden in 3dot9.

Sigh.
 

- Issue History
Date Modified Username Field Change
10-14-07 02:46 wiz New Issue
10-14-07 04:24 wiz Relationship added related to 0006721
10-14-07 04:26 wiz Issue Monitored: ScottWallace
10-14-07 04:26 wiz Note Added: 0011314
10-15-07 10:52 ScottWallace Note Added: 0011317
10-15-07 18:11 wiz Note Added: 0011324
10-15-07 21:19 ScottWallace Note Added: 0011329
10-16-07 03:04 wiz File Added: nilKeyFix2-wiz-M6730.3.cs
10-16-07 03:30 wiz Note Added: 0011331
10-16-07 03:30 wiz Note Added: 0011332
10-16-07 20:12 wiz Note Deleted: 0011332
10-16-07 20:29 wiz Note Added: 0011338

Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
67 total queries executed.
40 unique queries executed.
Powered by Mantis Bugtracker