Status: Accepted
Owner: [hidden email] Labels: Type-Bug Target-Spec Milestone-2.0 New issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 MethodBrowser works without that initialize (and uses initializeWidgets for the instantiation) where ClassMethodBrowser requires one (doesnt work without as models are nil). ClassMethodBrowser>>initialize "Initialization code for ClassMethodBrowser" super initialize. listModel := self instantiate: ListComposableModel. methodModel := self instantiate: MethodBrowser. self focusOrder add: listModel; add: methodModel. listModel whenSelectedItemChanged: [:selection | selection ifNotNil: [:class | methodModel methods: (class methodDict values sort: [:a :b | a selector < b selector]). methodModel listModel resetSelection ]]. methodModel wrapWith: [:method | method selector ]. And ClassMethodBrowser>>initializeWidgets self instantiateModels: #( listModel ListComposableModel methodModel MethodBrowser ). self focusOrder add: listModel; add: methodModel. methodModel displayBlock: [:method | method selector ]. has the instantiateModels. There seems to be a double instantiation in here... What is the right way to proceed? The Spec Report of June 2012 and the Spec tex thing in the SCM aren't right, so I wonder what the right way to do things is. TIA Phil -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: WorkNeeded Comment #1 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 (No comment was entered for this change.) -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Comment #2 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 SLICE-Issue-7535-SPEC-ClassMethodBrowser-double-initialization-PhilippeBack.1 in PharoInbox -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: FixReviewNeeded Comment #3 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 (No comment was entered for this change.) -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: MonkeyIsChecking Comment #4 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535#c4 The Monkey is currently checking this issue. Please don't change it! -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: ValidatedByTheMonkey Labels: CheckedIn20559 Comment #5 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535#c5 This Issue has been checked by Ulysse the Monkey 9908 tests passed in 00:00:00s: =============================== AST-Tests-Core (80) AST-Tests-Semantic (25) AndreasProfiler (8) Announcements-Tests-Core (29) Balloon-Fills-Tests (4) BalloonTests-Collections (34) CollectionsTests-Arrayed (553) CollectionsTests-Atomic (12) CollectionsTests-Sequenceable (912) CollectionsTests-SplitJoin (27) CollectionsTests-Stack (16) CollectionsTests-Streams (37) CollectionsTests-Strings (611) CollectionsTests-Support (12) CollectionsTests-Unordered (1954) CollectionsTests-Weak (739) CompilerTests (181) CompressionTests-Archive (7) ConfigurationCommandLineHandler-Tests (2) FileSystem-Tests-Core (186) FileSystem-Tests-Disk (53) FileSystem-Tests-Memory (50) FreeTypeTests-cache (23) FuelTests (333) FuelTests-Collections (5) FuelTests-Streams (3) Gofer-Tests (45) Graphics-Tests-Files (47) Graphics-Tests-Primitives (62) HelpSystem-Tests-Builders (5) HelpSystem-Tests-Core-Model (4) HelpSystem-Tests-Core-UI (6) HelpSystem-Tests-Core-Utilities (2) KernelTests-Chronology (593) KernelTests-Classes (68) KernelTests-Exception (3) KernelTests-Methods (180) KernelTests-Numbers (277) KernelTests-Objects (87) KernelTests-Pragmas (3) KernelTests-Processes (38) Keymapping-Tests (38) Manifest-Tests (25) MorphicTests-Basic (12) MorphicTests-Event (8) MorphicTests-Kernel (8) MorphicTests-Layouts (5) MorphicTests-Text Support (4) MorphicTests-Widgets (5) MorphicTests-WindowNotification (8) Multilingual-Tests-TextConversion (14) NECompletion-Tests (63) NativeBoost-Tests (65) Nautilus-Tests (8) NetworkTests-Kernel (32) NetworkTests-Mail (5) NetworkTests-Protocols (3) NetworkTests-RFC822 (1) NetworkTests-URI (105) NetworkTests-UUID (14) NetworkTests-Url (37) OSProcess-Tests (112) RPackage-SystemIntegration (80) RPackage-Tests (93) Refactoring-Tests-Changes (43) Refactoring-Tests-Core (216) Refactoring-Tests-Critics (57) Refactoring-Tests-Environment (20) Regex-Tests-Core (188) Ring-Tests-Containers (8) Ring-Tests-Kernel (61) Ring-Tests-Monticello (5) SUnit-Core-Extensions (3) SUnit-Core-Utilities (3) SUnit-Tests-Core (78) ShoutTests-Parsing (5) ShoutTests-Styling (2) Soup-Tests-Core (79) Spec-Tests (10) Spec-Tools-Senders-Tests (5) Tests-Bugs (29) Tests-CodeImport (9) Tests-Compression (10) Tests-Digital Signatures (8) Tests-Exceptions (28) Tests-Files (21) Tests-Finalization (1) Tests-Monticello (128) Tests-Multilingual (3) Tests-Object Events (24) Tests-ObjectsAsMethods (3) Tests-PackageInfo (2) Tests-Polymorph-Widgets (10) Tests-Release (4) Tests-ScriptLoader (4) Tests-System (268) Tests-System-CommandLine (15) Tests-SystemChangeNotification (1) Tests-SystemHistory (14) Tests-TextEditors (3) Tests-Traits (59) Tests-Traits-MOP (3) Tests-VM (10) TextTests (60) ToolsTest-Base (2) ToolsTest-Debugger (10) ToolsTest-Finder (9) ToolsTest-Inspector (1) ToolsTest-MessageTally (5) ToolsTest-PointerFinder (3) UpdateStreamer-Tests (2) Zinc-Character-Encoding-Tests (29) Zinc-Resource-Meta-FileSystem (8) Zinc-Resource-Meta-Tests (42) Zinc-Tests (164) Zinc-Zodiac (9) Zodiac-Tests (63) ---------------------------------------------------------- Loaded Source: SLICE-Issue-7535-SPEC-ClassMethodBrowser-double-initialization-PhilippeBack.1 from http://ss3.gemstone.com/ss/PharoInbox Tested using Pharo-2.0-20559-a on NBCoInterpreter NativeBoost-CogPlugin-IgorStasenko.15 uuid: 44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012 NBCogit NativeBoost-CogPlugin-IgorStasenko.15 uuid: 44b6b681-38f1-4a9e-b6ee-8769b499576a Nov 27 2012 https://git.gitorious.org/cogvm/blessed.git Commit: 40ac7e7bdec6fef0e934d2c019b86db996053912 Date: 2012-11-19 18:54:49 +0100 By: Mariano Martinez Peck <[hidden email]> Jenkins build #146 -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: FixToInclude Comment #6 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 (No comment was entered for this change.) -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Updates:
Status: Integrated Comment #7 on issue 7535 by [hidden email]: [SPEC] ClassMethodBrowser double initialization http://code.google.com/p/pharo/issues/detail?id=7535 in 20560 -- You received this message because this project is configured to send all issue notifications to this address. You may adjust your notification preferences at: https://code.google.com/hosting/settings _______________________________________________ Pharo-bugtracker mailing list [hidden email] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker |
Free forum by Nabble | Edit this page |