Reject empty names for dashboards (re getredash#76)

client/app/components/dashboards/edit-dashboard-dialog.html

@@ -4,7 +4,7 @@ <h4 class="modal-title">Edit: {{$ctrl.dashboard.name}}</h4>
 </div>
 <div class="modal-body">
   <p>
-    <input type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" autofocus>
+    <input type="text" class="form-control" placeholder="Dashboard Name" ng-model="$ctrl.dashboard.name" autofocus required>
   </p>
 
   <p ng-if="$ctrl.dashboard.id">

redash/handlers/dashboards.py

@@ -10,6 +10,7 @@
 from redash.permissions import (can_modify, require_admin_or_owner,
                                 require_object_modify_permission,
                                 require_permission)
+from sqlalchemy.exc import IntegrityError
 from sqlalchemy.orm.exc import StaleDataError
 
 
@@ -154,6 +155,8 @@ def post(self, dashboard_slug):
             models.db.session.commit()
         except StaleDataError:
             abort(409)
+        except IntegrityError:
+            abort(400)
 
         result = dashboard.to_dict(with_widgets=True, user=self.current_user)
         return result

redash/models.py

@@ -1264,7 +1264,7 @@ class Dashboard(ChangeTrackingMixin, TimestampMixin, BelongsToOrgMixin, db.Model
     org_id = Column(db.Integer, db.ForeignKey("organizations.id"))
     org = db.relationship(Organization, backref="dashboards")
     slug = Column(db.String(140), index=True, default=generate_slug)
-    name = Column(db.String(100))
+    name = Column(db.String(100), db.CheckConstraint("name<>''", name="dashboard_name_c"))
     user_id = Column(db.Integer, db.ForeignKey("users.id"))
     user = db.relationship(User)
     # TODO: The layout should dynamically be built from position and size information on each widget.