From 73354e39d02f52b329a96f7a16409e787e9c7b26 Mon Sep 17 00:00:00 2001
From: Mike Gerwitz <gerwitzm@lovullo.com>
Date: Thu, 7 Sep 2017 15:31:43 -0400
Subject: [PATCH] Fix restoring of document metadata after clearing

Previous to this change, we'd write the clear to the database, but
then immediately after, the quote save would restore it.  Oops.

* src/server/Server.js (_monitorMetadataPromise): Clear data in
    metabucket rather than directly invoking a DB call.  This will be
    applied when the quote is saved.
---
 src/server/Server.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/server/Server.js b/src/server/Server.js
index ae10d76..593d153 100644
--- a/src/server/Server.js
+++ b/src/server/Server.js
@@ -1148,9 +1148,7 @@ module.exports = Class( 'Server' )
 
     'private _monitorMetadataPromise'( quote, dapis, meta_clear )
     {
-        // save metadata clear to database to prevent stale data from being
-        // used while requests are pending
-        this.dao.saveQuoteMeta( quote, meta_clear, null, e => { throw e; } );
+        quote.getMetabucket().setValues( meta_clear );
 
         dapis.map( promise => promise
             .then( ( { field, index, data } ) =>