Changeset 852

Timestamp:

12/07/06 04:03:24 (2 years ago)

Author:

gboggs

Message:

Throw (JS alert() and CMS errtrans()) errors if the template name and outfile are not set. BugId: 45258

Files:

• branches/wheeljack/lib/MT/App/CMS.pm (modified) (1 diff)
• branches/wheeljack/tmpl/cms/edit_template.tmpl (modified) (2 diffs)

branches/wheeljack/lib/MT/App/CMS.pm

@@ -4915 +4915 @@
-    }
-
+    return $app->errtrans('The Template Name and Output File fields are required.')
+        if $type eq 'template' && !$q->param('name') && !$q->param('outfile');
+
-    if ($type eq 'author') {
-        ## If we are saving an author profile, we need to do some

branches/wheeljack/tmpl/cms/edit_template.tmpl

@@ -82 +82 @@
-    initDirty();
-}
+
+function validate (f, rebuild) {
+    if (!f.name.value) {
+        alert('<MT_TRANS phrase="You must set the Template Name." escape="singlequotes">');
+        return false;
+    }
+    else if (!f.outfile.value) {
+        alert('<MT_TRANS phrase="You must set the template Output File." escape="singlequotes">');
+        return false;
+    }
+    else {
+        clearDirty();
+        if (rebuild) setRebuild(f);
+    }
+}
-//-->
-</script>
@@ -251 +266 @@
-</div>
-
-clearDirty()
+return validate(this.form);
-<TMPL_UNLESS NAME=NEW_OBJECT>
-<TMPL_UNLESS NAME=BUILD_DYNAMIC>
-<TMPL_IF NAME=HAS_REBUILD>
-<TMPL_IF NAME=CAN_REBUILD>
-clearDirty();setRebuild(this.form)
+return validate(this.form, 1);
-</TMPL_IF>
-</TMPL_UNLESS>