Throw when trying to overwrite a template context value. Fixes #1884

middleman-core/lib/middleman-core/template_context.rb

@@ -36,14 +36,14 @@ module Middleman
       @opts = opts
 
       @locs.each do |k, _|
-        if self.respond_to?(k)
+        next unless self.respond_to?(k)
-          msg = "Template local `#{k}` tried to overwrite an existing context value. Please renamed the key when passing to `locals`"
 
-          if app.build?
+        msg = "Template local `#{k}` tried to overwrite an existing context value. Please renamed the key when passing to `locals`"
-            throw msg
+
-          else
+        if app.build?
-            logger.error(msg)
+          throw msg
-          end
+        else
+          logger.error(msg)
         end
       end
     end