Re: [GS/SS Beta] Displaying Dynamic Instance Variables in the Seaside inspector

Dale,

I made a change to the following method so the inspector displays dynamicInstVars in the seaside inspector:

This is in the 'seaside-gemStone-tools-production’ package.

Object>>inspectorFields
        | members |
        members := Array new writeStream.
        self class allInstVarNames withIndexDo: [ :each :index |
                members nextPut: each -> (self instVarAt: index) ].
        self dynamicInstanceVariables withIndexDo: [:each :index |
                members nextPut: each -> (self dynamicInstVarAt: each) ].
        self class isVariable ifTrue: [
                1 to: self size do: [ :index |
                        members nextPut: index -> (self at: index) ] ].
        ^ members contents

If this okay would it be possible for you to add this to the standard code.

If there is a better way to send this change to you please let me know.

Regards,

Reg

Good point!

Which version of Seaside are you using ... for one method it is probably easier for my to do it then to have you submit a pull request on github:)

Dale

On Mon, Jun 23, 2014 at 12:03 PM, Reg Krock <[hidden email]> wrote:

Dale,

I made a change to the following method so the inspector displays dynamicInstVars in the seaside inspector:

This is in the 'seaside-gemStone-tools-production’ package.

Object>>inspectorFields
        | members |
        members := Array new writeStream.
        self class allInstVarNames withIndexDo: [ :each :index |
                members nextPut: each -> (self instVarAt: index) ].
        self dynamicInstanceVariables withIndexDo: [:each :index |
                members nextPut: each -> (self dynamicInstVarAt: each) ].
        self class isVariable ifTrue: [
                1 to: self size do: [ :index |
                        members nextPut: index -> (self at: index) ] ].
        ^ members contents

If this okay would it be possible for you to add this to the standard code.

If there is a better way to send this change to you please let me know.

Regards,

Reg

Hi!

I would prefer to mark the dynamic vars ..

I am on Seaside 3.1

| members |

members := Array new writeStream.

self class allInstVarNames withIndexDo: [ :each :index |

members nextPut: each -> (self instVarAt: index) ].

self dynamicInstanceVariables withIndexDo: [:each :index |

members nextPut: ('*', each) -> (self dynamicInstVarAt: each) ].

self class isVariable ifTrue: [

1 to: self size do: [ :index |

members nextPut: index -> (self at: index) ] ].

^ members contents

regards

Gerhard

On Mon, Jun 23, 2014 at 9:27 PM, Dale Henrichs <[hidden email]> wrote:

Good point!

Which version of Seaside are you using ... for one method it is probably easier for my to do it then to have you submit a pull request on github:)

Dale

On Mon, Jun 23, 2014 at 12:03 PM, Reg Krock <[hidden email]> wrote:

Dale,

I made a change to the following method so the inspector displays dynamicInstVars in the seaside inspector:

This is in the 'seaside-gemStone-tools-production’ package.

Object>>inspectorFields
        | members |
        members := Array new writeStream.
        self class allInstVarNames withIndexDo: [ :each :index |
                members nextPut: each -> (self instVarAt: index) ].
        self dynamicInstanceVariables withIndexDo: [:each :index |
                members nextPut: each -> (self dynamicInstVarAt: each) ].
        self class isVariable ifTrue: [
                1 to: self size do: [ :index |
                        members nextPut: index -> (self at: index) ] ].
        ^ members contents

If this okay would it be possible for you to add this to the standard code.

If there is a better way to send this change to you please let me know.

Regards,

Reg

That's a very good idea ....

On Mon, Jun 23, 2014 at 12:48 PM, Gerhard Obermann <[hidden email]> wrote:

Hi!

I would prefer to mark the dynamic vars ..

I am on Seaside 3.1

| members |

members := Array new writeStream.

self class allInstVarNames withIndexDo: [ :each :index |

members nextPut: each -> (self instVarAt: index) ].

self dynamicInstanceVariables withIndexDo: [:each :index |

members nextPut: ('*', each) -> (self dynamicInstVarAt: each) ].

self class isVariable ifTrue: [

1 to: self size do: [ :index |

members nextPut: index -> (self at: index) ] ].

^ members contents

regards

Gerhard

On Mon, Jun 23, 2014 at 9:27 PM, Dale Henrichs <[hidden email]> wrote:

Good point!

Which version of Seaside are you using ... for one method it is probably easier for my to do it then to have you submit a pull request on github:)

Dale

On Mon, Jun 23, 2014 at 12:03 PM, Reg Krock <[hidden email]> wrote:

Dale,

I made a change to the following method so the inspector displays dynamicInstVars in the seaside inspector:

This is in the 'seaside-gemStone-tools-production’ package.

Object>>inspectorFields
        | members |
        members := Array new writeStream.
        self class allInstVarNames withIndexDo: [ :each :index |
                members nextPut: each -> (self instVarAt: index) ].
        self dynamicInstanceVariables withIndexDo: [:each :index |
                members nextPut: each -> (self dynamicInstVarAt: each) ].
        self class isVariable ifTrue: [
                1 to: self size do: [ :index |
                        members nextPut: index -> (self at: index) ] ].
        ^ members contents

If this okay would it be possible for you to add this to the standard code.

If there is a better way to send this change to you please let me know.

Regards,

Reg