Handle a null or empty database name

If name is null then user cancelled the prompt so we
do nothing.

If name is zero length then display an error via
notifications.


Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/c585708f
Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/c585708f
Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/c585708f

Branch: refs/heads/fauxton
Commit: c585708f71ca4344abd0d51e86edecbc6ff16812
Parents: 08de3e7
Author: Mike Wallace <mikewallace1...@googlemail.com>
Authored: Sun Jan 20 22:26:48 2013 +0000
Committer: Mike Wallace <mikewallace1...@googlemail.com>
Committed: Mon Jan 21 13:12:10 2013 +0000

----------------------------------------------------------------------
 src/fauxton/app/modules/databases/views.js |   13 ++++++++++++-
 1 files changed, 12 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/couchdb/blob/c585708f/src/fauxton/app/modules/databases/views.js
----------------------------------------------------------------------
diff --git a/src/fauxton/app/modules/databases/views.js 
b/src/fauxton/app/modules/databases/views.js
index fafddc5..b069b85 100644
--- a/src/fauxton/app/modules/databases/views.js
+++ b/src/fauxton/app/modules/databases/views.js
@@ -98,13 +98,24 @@ function(app, FauxtonAPI) {
     },
 
     newDatabase: function() {
+      var notification;
       // TODO: use a modal here instead of the prompt
       var name = prompt('Name of database', 'newdatabase');
+      if (name === null) {
+        return;
+      } else if (name.length === 0) {
+        notification = FauxtonAPI.addNotification({
+          msg: "Please enter a valid database name",
+          type: "error",
+          clear: true
+        });
+        return;
+      }
       var db = new this.collection.model({
         id: encodeURIComponent(name),
         name: name
       });
-      var notification = FauxtonAPI.addNotification({msg: "Creating 
database."});
+      notification = FauxtonAPI.addNotification({msg: "Creating database."});
       db.save().done(function() {
         notification = FauxtonAPI.addNotification({
           msg: "Database created successfully",

Reply via email to