Fixes #494: Forcing named classes in CoffeeScript (named functions) .…

….. even in IE.
jashkenas · Dec 19, 2011 · 1018c51 · 1018c51 · michaelficarra · Dec 19, 2011
1 parent 8d8c345
commit 1018c51
Show file tree

Hide file tree

Showing 7 changed files with 76 additions and 6 deletions.
diff --git a/lib/coffee-script/lexer.js b/lib/coffee-script/lexer.js
diff --git a/lib/coffee-script/nodes.js b/lib/coffee-script/nodes.js
diff --git a/lib/coffee-script/optparse.js b/lib/coffee-script/optparse.js
diff --git a/lib/coffee-script/rewriter.js b/lib/coffee-script/rewriter.js
diff --git a/lib/coffee-script/scope.js b/lib/coffee-script/scope.js
diff --git a/src/nodes.coffee b/src/nodes.coffee
@@ -928,6 +928,7 @@ exports.Class = class Class extends Base
     @ensureConstructor name
     @body.spaced = yes
     @body.expressions.unshift @ctor unless @ctor instanceof Code
+    @body.expressions.unshift new Literal "#{name}.name = '#{name}'" if decl
     @body.expressions.push lname
     @addBoundFunctions o
 

diff --git a/test/classes.coffee b/test/classes.coffee
@@ -593,4 +593,14 @@ test "#1813: Passing class definitions as expressions", ->
     x = 1
 
   eq result, B
-
+
+test "#494: Named classes", ->
+
+  class A
+  eq A.name, 'A'
+
+  class A.B
+  eq A.B.name, 'B'  
+
+  class A.B["C"]        
+  ok A.B.C.name isnt 'C'