use showMessageBox instead of notifier to ask user for feedback.

Author: rueckstiess

src/connect/behavior.js

@@ -3,7 +3,7 @@ var assert = require('assert');
 var Connection = require('../models/connection');
 var _ = require('lodash');
 
-// var debug = require('debug')('scout:connect:behavior');
+var debug = require('debug')('scout:connect:behavior');
 
 module.exports = State.extend({
   props: {
@@ -81,7 +81,7 @@ module.exports = State.extend({
   dispatch: function(action) {
     var newState = this.reduce(this.state, action);
     // if (newState !== this.state) {
-    //   debug('transition: (%s, %s) ==> %s', this.state, action, newState);
+    debug('transition: (%s, %s) ==> %s', this.state, action, newState);
     // }
     this.state = newState;
     return this.state;

src/connect/index.js

@@ -8,12 +8,15 @@ var View = require('ampersand-view');
 
 var _ = require('lodash');
 var app = require('ampersand-app');
-var debug = require('debug')('scout:connect:index');
 var format = require('util').format;
 var metrics = require('mongodb-js-metrics');
 
 var remote = window.require('remote');
+var dialog = remote.require('dialog');
 var Clipboard = remote.require('clipboard');
+var BrowserWindow = remote.require('browser-window');
+
+var debug = require('debug')('scout:connect:index');
 
 /**
  * AuthenticationOptionCollection
@@ -66,6 +69,10 @@ var ConnectView = View.extend({
     previousSslMethod: {
       type: 'string',
       default: null
+    },
+    clipboardText: {
+      type: 'string',
+      default: ''
     }
   },
   derived: {
@@ -233,36 +240,53 @@ var ConnectView = View.extend({
     this.listenToAndRun(this, 'change:sslMethod',
       this.replaceSslMethodFields.bind(this));
 
-    this.listenTo(app, 'autofill-connection-from-clipboard',
-      this.autofillFromClipboard.bind(this));
+    // this.listenTo(app, 'autofill-connection-from-clipboard',
+    //   this.autofillFromClipboard.bind(this));
+
     this.listenTo(app, 'connect-window-focused',
-      this.checkIfCanAutofillFromClipboard.bind(this));
+      this.onConnectWindowFocused.bind(this));
 
     // always start in NEW_EMPTY state
     this.dispatch('new connection clicked');
   },
 
+  // === MongoDB URI clipboard Handling
+
+  /**
+   * Called when the user clicked "YES" in the message dialog after
+   * a MongoDB URI was detected.
+   */
   autofillFromClipboard: function() {
-    var connectionFromCB = MongoDBConnection.from(this.lastClipboardTxt).toJSON();
-    this.connection = new Connection();
-    /* eslint guard-for-in: 0 */
-    for (var attr in connectionFromCB) {
-      this.connection[attr] = connectionFromCB[attr];
-    }
-    /* eslint guard-for-in: 1 */
+    this.connection = MongoDBConnection.from(this.clipboardText);
     this.updateForm();
   },
 
-  checkIfCanAutofillFromClipboard: function() {
-    var cbTxt = Clipboard.readText();
-    if (cbTxt === this.lastClipboardTxt) {
+  /**
+   * Called when the Connect Window receives focus.
+   */
+  onConnectWindowFocused: function() {
+    var clipboardText = Clipboard.readText();
+    if (clipboardText === this.clipboardText) {
+      // we have seen this value already, don't ask user again
       return;
     }
-    this.lastClipboardTxt = cbTxt;
-
-    if (MongoDBConnection.isURI(cbTxt)) {
-      debug('mongoURI detected.');
-      app.sendMessage('show autofill connection notification');
+    this.clipboardText = clipboardText;
+
+    if (MongoDBConnection.isURI(clipboardText)) {
+      debug('MongoDB URI detected.', clipboardText);
+      // ask user if Compass should use it to fill out form
+      dialog.showMessageBox(BrowserWindow.getFocusedWindow(), {
+        type: 'info',
+        message: 'MongoDB connection string detected',
+        detail: 'Compass detected a MongoDB connection string in your '
+          + 'clipboard. Do you want to use the connection string to '
+          + 'fill out this form?',
+        buttons: ['Yes', 'No']
+      }, function(response) {
+        if (response === 0) {
+          this.autofillFromClipboard();
+        }
+      }.bind(this));
     }
   },
 

src/electron/window-manager.js

@@ -167,21 +167,21 @@ app.on('show about dialog', function() {
   });
 });
 
-app.on('show autofill connection notification', function() {
-  Notifier.notify({
-    'icon': SCOUT_ICON_PATH,
-    'message': 'Click this notification to autofill the connection fields from your clipboard.',
-    'title': 'Autofill Connection',
-    'wait': true
-  }, function(err, resp) {
-    if (err) {
-      debug(err);
-    }
-    if (resp === 'Activate\n') {
-      connectWindow.webContents.send('message', 'autofill-connection-from-clipboard');
-    }
-  });
-});
+// app.on('show autofill connection notification', function() {
+//   Notifier.notify({
+//     'icon': SCOUT_ICON_PATH,
+//     'message': 'Click this notification to autofill the connection fields from your clipboard.',
+//     'title': 'Autofill Connection',
+//     'wait': true
+//   }, function(err, resp) {
+//     if (err) {
+//       debug(err);
+//     }
+//     if (resp === 'Activate\n') {
+//       connectWindow.webContents.send('message', 'autofill-connection-from-clipboard');
+//     }
+//   });
+// });
 
 app.on('hide connect submenu', function() {
   AppMenu.hideConnect();