Manuscript (Case [Issue]22395) _Inbox - Logic of declaring actions for nodes in SugsSuggestions is not really good

Classic

List

Threaded

♦

♦

Locked 1 message

Pharo Issue Tracker

Reply | Threaded

Open this post in threaded view

♦

♦

|

Manuscript (Case [Issue]22395) _Inbox - Logic of declaring actions for nodes in SugsSuggestions is not really good

Marcus Denker edited Case 22395: Logic of declaring actions for nodes in SugsSuggestions is not really good : Bug in  _Inbox: Not Spam  •  You are subscribed to this case The SugsSuggestionFactory was >300 lines of code, *a lot* of class instance variables, and a huge mess. For the redesign, it was not known that AST nodes are ever possible that are not subclasses of RBProgramNode. It am sure that we can find a solution for this that is not as bad as SugsSuggestionFactory was. Priority: 5 – Fix If Time Status: Work Needed Assigned to: Everyone Milestone: Later Go to Case No longer need updates? Unsubscribe from this case. Don't want Manuscript notifications anymore? Update your preferences. A Manuscript case was edited by Marcus Denker. Case ID: 22395 Title: Logic of declaring actions for nodes in SugsSuggestions is not really good Status: Work Needed Category: Bug Project: _Inbox Area: Not Spam Priority: 5 - Fix If Time Milestone: Later Assigned To: Everyone URL: https://pharo.manuscript.com/f/cases/22395 The SugsSuggestionFactory was >300 lines of code, *a lot* of class instance variables, and a huge mess. For the redesign, it was not known that AST nodes are ever possible that are not subclasses of RBProgramNode. It am sure that we can find a solution for this that is not as bad as SugsSuggestionFactory was. You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.manuscript.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&command=view&ixBug=22395) from this case. _______________________________________________ Pharo-bugtracker mailing list [hidden email] https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker