Hi,
Since yesterday I get a lot of bugs when I try to edit class definitions while having the slot template in class definition enabled. (See comment: https://github.com/pharo-project/pharo/pull/4391) This make it a little hard to develop :( Is there an easy fix? Else maybe we should revert the change until we have the fix? Have a nice day! -- Cyril Ferlicot https://ferlicot.fr |
Argh this is probably the changes using the class definition.
Denis can you check? Else I will check how to revert it and wait that it becoming more ready. Stef > On 23 Aug 2019, at 11:49, Cyril Ferlicot <[hidden email]> wrote: > > Hi, > > Since yesterday I get a lot of bugs when I try to edit class > definitions while having the slot template in class definition > enabled. (See comment: > https://github.com/pharo-project/pharo/pull/4391) > This make it a little hard to develop :( > > Is there an easy fix? Else maybe we should revert the change until we > have the fix? > > Have a nice day! > > -- > Cyril Ferlicot > https://ferlicot.fr > |
In reply to this post by CyrilFerlicot
> On 23 Aug 2019, at 11:49, Cyril Ferlicot <[hidden email]> wrote: > > Hi, > > Since yesterday I get a lot of bugs when I try to edit class > definitions while having the slot template in class definition > enabled. (See comment: > https://github.com/pharo-project/pharo/pull/4391) > This make it a little hard to develop :( > > Is there an easy fix? Else maybe we should revert the change until we > have the fix? > I have a fix, I will commit when the internet connection is good enough Marcus |
Hi пт, 23 авг. 2019 г., 11:06 Marcus Denker <[hidden email]>:
That's good. I guess it was a class parser bug? I only tested browser with classic definition.
|
yes, even with this fix I need to do some (larger) change to support Slots with parameters, e.g. {#lazy => LazySlot default: 5 . #computed => ComputedSlot with: [:object | object calculateSomething] } This need some change in how the class parser models slots, though… CDSlotNode is just modelling a name and the class of the slot, but not the fact that slot definitions can contain parameters which can contain even blocks… and with composition (not yet in Pharo8), we need to support things like #lazyAndObservable => SpObservableSlot + LazySlot default: 5 . but I think it is not hard to do. Marcus
|
I updated the fix: now it should work, there will be just no menu shown for these kinds of slots for now (to be fixed in a second step, we should add tests, too) Marcus |
Free forum by Nabble | Edit this page |