[squeak-dev] The Inbox: Tools-ct.980.mcz

commits at source.squeak.org commits at source.squeak.org
Sat Jun 27 19:46:17 UTC 2020


A new version of Tools was added to project The Inbox:
http://source.squeak.org/inbox/Tools-ct.980.mcz

==================== Summary ====================

Name: Tools-ct.980
Author: ct
Time: 27 June 2020, 9:46:13.766705 pm
UUID: ad6a8c37-fddf-fa4f-96ca-6b3c64d4b594
Ancestors: Tools-mt.979

Fixes opening a TextLink pointing to a class from within a MessageSet. In addition, only select the new item for a consistent experience.

Steps to reproduce the original bug:
1. Insert a TextLink pointing to a class comment into any method comment and save the method with style.
2. Browse the method in a message trace.
3. Click the link.
Before this fix, the message trace raised a draw error because an internal conversion step was forgotten. See also Model >> #addItem: which already respected that conversion.

=============== Diff against Tools-mt.979 ===============

Item was changed:
  ----- Method: MessageSet>>addItem: (in category 'message list') -----
  addItem: classAndMethod
  	"Append a classAndMethod string to the list.  Select the new item."
  
+ 	"Do some checks on the input"
+ 	classAndMethod isMethodReference ifFalse: [
+ 		^ self class parse: classAndMethod toClassAndSelector: [:class :sel |
+ 			class ifNotNil: [
+ 				self addItem: (MethodReference class: class selector: sel)]]].
+ 	
- 	"Do some checks on the input?"
  	self okToChange ifFalse: [^ self].
+ 	self deselectAll.
+ 	self basicAddItem: classAndMethod.
- 	messageList add: classAndMethod.
  	self changed: #messageList.
  	self messageListIndex: messageList size.!

Item was added:
+ ----- Method: MessageSet>>basicAddItem: (in category 'private') -----
+ basicAddItem: classAndMethod
+ 
+ 	messageList add: classAndMethod.!

Item was added:
+ ----- Method: MessageTrace>>basicAddItem: (in category 'message list') -----
+ basicAddItem: classAndMethod
+ 
+ 	super basicAddItem: classAndMethod.
+ 	
+ 	autoSelectStrings add: nil.
+ 	messageSelections add: true.!



More information about the Squeak-dev mailing list