Navigation Menu

Skip to content

Commit

Permalink
Receive synonyms directly by updateSynonymsSync()
Browse files Browse the repository at this point in the history
  • Loading branch information
piroor committed Aug 9, 2012
1 parent 17e59aa commit e846db7
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 12 deletions.
2 changes: 1 addition & 1 deletion lib/api/2011-02-01/configuration.js
Expand Up @@ -402,7 +402,7 @@ handlers.UpdateSynonymOptions = function(context, request, response) {

var updatedAt = new Date();

domain.updateSynonymsSync(synonymOptions);
domain.updateSynonymsSync(synonymOptions.synonyms);

response.send(createUpdateSynonymOptionsResponse({
synonymOptions: JSON.stringify(synonymOptions),
Expand Down
4 changes: 1 addition & 3 deletions lib/database/domain.js
Expand Up @@ -273,12 +273,10 @@ Domain.prototype = {
return formattedSynonyms;
},

updateSynonymsSync: function(synonymOptions) {
updateSynonymsSync: function(synonyms) {
if (!this.context)
throw new Error('no context');

var synonyms = synonymOptions.synonyms;

try {
this.context.commandSync('table_remove', {
table: this.synonymsTableName
Expand Down
12 changes: 4 additions & 8 deletions test/database-domain.test.js
Expand Up @@ -307,10 +307,8 @@ suite('database', function() {
assert.isFalse(domain.isSynonymsTableAvailableSync());

domain.updateSynonymsSync({
synonyms: {
tokio: ['tokyo'],
dekkaido: 'hokkaido'
}
tokio: ['tokyo'],
dekkaido: 'hokkaido'
});
assert.isTrue(domain.isSynonymsTableAvailableSync());

Expand All @@ -334,10 +332,8 @@ suite('database', function() {
test('getSynonymsSync', function() {
var domain = new Domain('companies', context);
domain.updateSynonymsSync({
synonyms: {
tokio: ['tonkin', 'tokyo'],
dekkaido: 'hokkaido'
}
tokio: ['tonkin', 'tokyo'],
dekkaido: 'hokkaido'
});

var expectedSynonyms = {
Expand Down

0 comments on commit e846db7

Please sign in to comment.