Fix missing variable

groonga · Nov 21, 2012 · ace939f · ace939f
1 parent acdef01
commit ace939f
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/lib/api/2011-02-01/configuration.js b/lib/api/2011-02-01/configuration.js
@@ -334,11 +334,11 @@ function assertHaveNoMismatchedTypeOption(request, type) {
       });
 
   if (type == 'text' && (literalOptions.length || uintOptions.length))
-    throw new error.FieldOptionConflictError('A text IndexField may only specify textOptions');
+    throw new errors.FieldOptionConflictError('A text IndexField may only specify textOptions');
   if (type == 'literal' && (textOptions.length || uintOptions.length))
-    throw new error.FieldOptionConflictError('A literal IndexField may only specify literalOptions');
+    throw new errors.FieldOptionConflictError('A literal IndexField may only specify literalOptions');
   if (type == 'uint' && (textOptions.length || literalOptions.length))
-    throw new error.FieldOptionConflictError('A uint IndexField may only specify uintOptions');
+    throw new errors.FieldOptionConflictError('A uint IndexField may only specify uintOptions');
 }
 
 handlers.DefineIndexField = function(context, request, response, config) {
@@ -352,11 +352,11 @@ handlers.DefineIndexField = function(context, request, response, config) {
   var fieldName = request.query['IndexField.IndexFieldName'] || '';
   var fieldType = request.query['IndexField.IndexFieldType'] || '';
   var field = handleIndexFieldValidationError(function() {
-        var field = domain.getIndexField(fieldName).setType(fieldType).validate();
-        assertHaveNoMismatchedTypeOption(request, fieldType);
-        return field;
+        return domain.getIndexField(fieldName).setType(fieldType).validate();
       });
 
+  assertHaveNoMismatchedTypeOption(request, fieldType);
+
   var facetEnabled = getFieldOption('FacetEnabled', request, fieldType);
   if (facetEnabled !== undefined)
     field.facetEnabled = facetEnabled.toLowerCase() == 'true';