11 Jul
2005
11 Jul
'05
3:41 p.m.
Hi,
with the last image from
http://www.iam.unibe.ch/~scg/Teaching/AdvancedLabs/swimages/
and the last monticello versions of seaside (63), magritte (28) and
smallwiki2 (78) an error is printed just when the website is
displayed.
This is because the field 'embedded' is not initialised in the class
SW2Link.
I have modified SW2Link>>embedded to made it work :
embedded
embedded isNil ifTrue: [ embedded := true ].
^ embedded
--
Damien
11 Jul
11 Jul
3:51 p.m.
This is because the field 'embedded' is not
initialised in the
class SW2Link.
I have modified SW2Link>>embedded to made it work :
embedded
embedded isNil ifTrue: [ embedded := true ].
^ embedded
This is just when you load a newer version of SmallWiki 2 into the
current development version. Please note that the development
versions are not always backward compatible and that in some cases
(like this one) you need to reinitialize your model.
Cheers,
Lukas
--
Lukas Renggli
http://www.lukas-renggli.ch
4:18 p.m.
This is just when you load a newer version of
SmallWiki 2 into the
current development version. Please note that the development
versions are not always backward compatible and that in some cases
(like this one) you need to reinitialize your model.
So, the only thing I had to do is to modify the field in the instance ?
Thanks
--
Damien
4:38 p.m.
This is just
when you load a newer version of SmallWiki 2 into the
current development version. Please note that the development
versions are not always backward compatible and that in some cases
(like this one) you need to reinitialize your model.
So, the only thing I had to do is to modify the field in the
instance ?
5:03 p.m.
Your fix is a bit strange, because not all instances
of a link want
to be embedded. This will certainly lead to problems as soon as the
renderer takes care about this new property. It is probably better to
use false as a default instance, but then it will break the environment.
So if you really don't want to drop your existing model instances and
migrate to the latest version I suggest to do something along the lines:
SW2Link allSubInstancesDo: [ :e |
e instVarNamed: 'embedded' put: false ].
SW2SeasideFrame initializeEnvironment.
Thank you
--
Damien
5:51 p.m.
Damien
Your fix is a bit strange, because not all instances
of a link want
to be embedded. This will certainly lead to problems as soon as the
renderer takes care about this new property. It is probably better
to use false as a default instance, but then it will break the
environment.
So if you really don't want to drop your existing model instances
and migrate to the latest version I suggest to do something along
the lines:
SW2Link allSubInstancesDo: [ :e |
e instVarNamed: 'embedded' put: false ].
SW2SeasideFrame initializeEnvironment.
Most of the time, I would suggest to drop the existing instances, but
this is a nice examples of how we
can keep a system running two years and upgrade it on the fly. When
you have to do this kind of
things this is quite powerful.
Stef
6:34 p.m.
Most of the
time, I would suggest to drop the existing instances,
How do you drop existing instances ?
6:46 p.m.
On Mon, Jul 11, 2005 at 06:34:45PM +0200, Lukas Renggli wrote:
You open an inspector on
SW2Kernel instances
and remove your the old kernels from the ordered-collection. Then you
evaluate
SW2SeasideFrame initialize
that will setup a new Seaside application and ask you if it should create a
new kernel, since you deleted the old ones.
Thanks a lot for this explanations
--
Damien
Most of the time, I would suggest to drop the existing
instances,
How do you drop existing instances ?
10:49 p.m.
Normally this should be based on a GC if you do not have a root
structure somewhere.
You can also nil out some instance variables and do a GC
SmalltalkImage or Smalltalk garbage....
stef
On 11 juil. 05, at 18:04, Damien Cassou wrote:
Most of the
time, I would suggest to drop the existing instances,
How do you drop existing instances ?
--
Damien
7146
days inactive
7146
days old
9 comments
4 participants
participants (4)
-
Damien Cassou -
Lukas Renggli -
Lukas Renggli -
stéphane ducasse