Don't extend PickHelper. Ref #7

thomthom · Feb 11, 2019 · 68984e6 · 68984e6
1 parent 19332f2
commit 68984e6
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/modules/pick_helper.rb b/modules/pick_helper.rb
@@ -4,9 +4,12 @@ module PickHelper
 
     # @param [Sketchup::View] view
     def self.new(view)
-      pick_helper = view.pick_helper
-      pick_helper.extend(self)
-      pick_helper
+      # TODO: Don't do this! `Sketchup::PickHelper` objects are reused. Cannot
+      # use `extend` as it with modify the global persistent object.
+      raise NotImplementedError
+      # pick_helper = view.pick_helper
+      # pick_helper.extend(self)
+      # pick_helper
     end
 
     # Return unique set of leaves from the pick paths.