Report an error if the user attempts to shadow a

core API fn.
sonic-pi-net · Nov 20, 2014 · c41eaa5 · c41eaa5
1 parent d367540
commit c41eaa5
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/app/server/sonicpi/lib/sonicpi/spiderapi.rb b/app/server/sonicpi/lib/sonicpi/spiderapi.rb
@@ -277,7 +277,13 @@ def ndefine(name, &block)
 
     def define(name, &block)
       raise "define must be called with a code block" unless block
-      if @user_methods.method_defined? name
+      already_defined = @user_methods.method_defined? name
+
+      if !already_defined && self.respond_to?(name)
+        raise "A function called #{name} is already part of Sonic Pi's core API. Please choose another name."
+      end
+
+      if already_defined
         __info "Redefining #{name}"
       else
         __info "Defining #{name}"