Removed a few TODOs.

These really aren't worth doing. The comments have been expanded to better explain what's going on as well.
wbyoung · Jun 21, 2015 · 75bee04 · 75bee04
1 parent 44f6648
commit 75bee04
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/lib/relations/belongs_to_config.js b/lib/relations/belongs_to_config.js
@@ -43,11 +43,9 @@ module.exports = Mixin.create(/** @lends BelongsTo# */ {
   configure: function() {
     /* jshint expr: true */
 
-    // TODO: this could be refactored to actually do configuration in this
-    // method rather than simply calling each property.
-
     // configure each of the properties that are calculated on a delay by
-    // simply invoking the property once.
+    // simply invoking the property once. configure the inverse first as some
+    // other calculations may rely on it.
     this.inverse;
     this.primaryKey;
     this.primaryKeyAttr;

diff --git a/lib/relations/has_many_config.js b/lib/relations/has_many_config.js
@@ -43,11 +43,9 @@ module.exports = Mixin.create(/** @lends HasMany# */ {
   configure: function() {
     /* jshint expr: true */
 
-    // TODO: this could be refactored to actually do configuration in this
-    // method rather than simply calling each property.
-
     // configure each of the properties that are calculated on a delay by
-    // simply invoking the property once.
+    // simply invoking the property once. configure the inverse first as some
+    // other calculations may rely on it.
     this.inverse;
     this.primaryKey;
     this.primaryKeyAttr;