From b3a05969496f859c136ce755736dca2087bca095 Mon Sep 17 00:00:00 2001
From: aaronsamuel137 <aaron.davis.samuel@gmail.com>
Date: Wed, 11 Nov 2015 09:58:23 -0700
Subject: [PATCH] fix abstract type anti-pattern, see #189

---
 js/common/module/ModeConfig.js | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/js/common/module/ModeConfig.js b/js/common/module/ModeConfig.js
index bd97b3eb..0d6b9438 100644
--- a/js/common/module/ModeConfig.js
+++ b/js/common/module/ModeConfig.js
@@ -1,7 +1,9 @@
 // Copyright 2014-2015, University of Colorado Boulder
 
 /**
- * Configuration for setting up a particular GravityAndOrbitsMode, enumerated in ModeList
+ * Configuration for setting up a particular GravityAndOrbitsMode, enumerated in ModeList.
+ * This is an abstract class, and is intended only to be used by sub-types.
+ * @abstract
  *
  * @author Sam Reid
  * @author Aaron Davis
@@ -69,10 +71,12 @@ define( function( require ) {
     },
 
     /**
-     * @protected
+     * @returns {BodyConfiguration[]}
+     * @public
      * @abstract
-     * @returns {Array<BodyConfiguration}
      */
-    getBodies: function() {}
+    getBodies: function() {
+      throw new Error( 'must be implemented by subtype' );
+    }
   } );
 } );