Merge pull request #15 from adri09070/9-remove-was-sent-to-nil-when-r…

…emoving-an-object-from-a-chest-with-the-tree-view-within-the-debugger fixing bugs in actions of the tree view
pharo-spec · Jul 31, 2023 · c6dd43f · c6dd43f
2 parents 7dae9c7 + c6da44d
commit c6dd43f
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/Chest/ChestTableWithContentPresenter.class.st b/src/Chest/ChestTableWithContentPresenter.class.st
@@ -722,14 +722,15 @@ ChestTableWithContentPresenter >> removeAllChests [
 { #category : #removing }
 ChestTableWithContentPresenter >> removeAllItemsFromSelectedChest [
 
-	| items |
+	| items chest |
 	self flag: 'to do with polymorphism'.
 
 	items := activePresenter == chestWithContentTreeTable
 		         ifTrue: [ 
 		         chestWithContentTreeTable childrenFor: self selectedChest ]
 		         ifFalse: [ chestContentsTable items ].
-	items do: [ :assoc | self selectedChest remove: assoc value ]
+	chest := self selectedChest.
+	items do: [ :assoc | chest remove: assoc value ]
 ]
 
 { #category : #removing }
@@ -829,7 +830,7 @@ ChestTableWithContentPresenter >> selectedChest [
 		 activePresenter == chestWithContentTreeTable ]) ifTrue: [ 
 		^ chestWithContentTreeTable selectedItem isChest
 			  ifTrue: [ chestWithContentTreeTable selectedItem ]
-			  ifFalse: [ nil ] ].
+			  ifFalse: [ chestWithContentTreeTable parentOfSelected ] ].
 	^ chestsTable selection selectedItem
 ]
 

diff --git a/src/Chest/SpTreeTablePresenter.extension.st b/src/Chest/SpTreeTablePresenter.extension.st
@@ -0,0 +1,11 @@
+Extension { #name : #SpTreeTablePresenter }
+
+{ #category : #'*Chest' }
+SpTreeTablePresenter >> parentOfSelected [
+
+	| parentPath |
+	parentPath := self selection selectedPath.
+	parentPath ifEmpty: [ ^ nil ].
+	parentPath := parentPath allButLast.
+	^ self itemAtPath: parentPath ifAbsent: nil
+]