DSVColumnView.st
changeset 5195 1d15a878af42
parent 5194 b28b44a4a21e
child 5204 fccb63f32188
--- a/DSVColumnView.st	Fri Jul 29 19:01:59 2016 +0200
+++ b/DSVColumnView.st	Fri Jul 29 19:04:14 2016 +0200
@@ -452,10 +452,6 @@
         selectedColIndexHolder := ValueHolder new.
     ].
     ^ selectedColIndexHolder
-!
-
-sortListInPlace:something
-    sortListInPlace := something.
 ! !
 
 !DSVColumnView methodsFor:'accessing-actions'!
@@ -670,6 +666,32 @@
     selectRowOnDefault := aBool
 !
 
+sortListInPlace
+    "when false (default for backward compatibility): 
+     if sorting, create a copy
+     of the list, which is wrong when useIndex is on,
+     as the application will get the index in the sorted
+     list, which is probably different from tha apps list mode.
+     When true (should be default, but that might break many
+     users), the passed in list is sorted in place (i.e. possibly
+     sorting the application's list)."
+     
+    ^ sortListInPlace
+!
+
+sortListInPlace:aBoolean
+    "when false (default for backward compatibility): 
+     if sorting, create a copy
+     of the list, which is wrong when useIndex is on,
+     as the application will get the index in the sorted
+     list, which is probably different from tha apps list mode.
+     When true (should be default, but that might break many
+     users), the passed in list is sorted in place (i.e. possibly
+     sorting the application's list)."
+     
+    sortListInPlace := aBoolean.
+!
+
 tabIntern
     "returns true if tabing is supported in the widget"