search

pharo-project / pharo

Pharo is a dynamic reflective pure object-oriented language supporting live programming inspired by Smalltalk.
http://pharo.org
Other
1.21k stars 356 forks source link

Breakpoint should get an application and not cross layers #17393

Open Ducasse opened 3 hours ago

Ducasse commented 3 hours ago

Or it should remove the browseAll method. We should stop to cross layers this way. A domain object should not make reference to UI elements freely.

browseAll
    <script>

    | methods |
    methods := self all flatCollect: [ :breakpoint | breakpoint link methods ].
    Smalltalk tools messageList browse: methods

=> 

browseAll
    <script>

    | methods |
    methods := self all flatCollect: [ :breakpoint | breakpoint link methods ].
    self application tools messageList browse: methods
Ducasse commented 3 hours ago

http://rmod-pharo-mooc.lille.inria.fr/AdvancedDesignMooc/Slides/M9-4-CaseStudy-06-LayeredSettingsArchitecture.pdf