From 405e90c5d5ef383f1034e64ad3780d0e012a8927 Mon Sep 17 00:00:00 2001
From: "bbaetz%acm.org" <>
Date: Fri, 29 May 2009 00:59:39 +0000
Subject: Bug 494751 - Toggling Add/Don't add attachment doesn't reset JS state

r/a=mkanat
---
 js/attachment.js                                            | 7 ++++++-
 template/en/default/attachment/create.html.tmpl             | 3 +--
 template/en/default/attachment/createformcontents.html.tmpl | 6 ++++++
 3 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/js/attachment.js b/js/attachment.js
index c05d1d3ea..254331679 100644
--- a/js/attachment.js
+++ b/js/attachment.js
@@ -100,7 +100,12 @@ function clearAttachmentFields() {
         URLFieldHandler();
     }
     document.getElementById('description').value = '';
-    document.getElementById('ispatch').checked = '';
+    /* Fire onchange so that the disabled state of the content-type
+     * radio buttons are also reset 
+     */
+    element = document.getElementById('ispatch');
+    element.checked = '';
+    bz_fireEvent(element, 'change');
     if ((element = document.getElementById('isprivate')))
         element.checked = '';
 }
diff --git a/template/en/default/attachment/create.html.tmpl b/template/en/default/attachment/create.html.tmpl
index 10648159b..687cd7cbf 100644
--- a/template/en/default/attachment/create.html.tmpl
+++ b/template/en/default/attachment/create.html.tmpl
@@ -33,9 +33,8 @@
   title = title
   header = header
   subheader = subheader
-  onload="setContentTypeDisabledState(document.entryform);"
   style_urls = [ 'skins/standard/create_attachment.css' ]
-  javascript_urls = [ "js/attachment.js" ]
+  javascript_urls = [ "js/attachment.js", "js/util.js" ]
   doc_section = "attachments.html"
 %]
 
diff --git a/template/en/default/attachment/createformcontents.html.tmpl b/template/en/default/attachment/createformcontents.html.tmpl
index 2fc80a518..2cef632d1 100644
--- a/template/en/default/attachment/createformcontents.html.tmpl
+++ b/template/en/default/attachment/createformcontents.html.tmpl
@@ -70,6 +70,12 @@
     <input type="checkbox" id="ispatch" name="ispatch" value="1"
            onchange="setContentTypeDisabledState(this.form);">
     <label for="ispatch">patch</label><br><br>
+    [%# Reset this whenever the page loads so that the JS state is up to date %]
+    <script type="text/javascript">
+      YAHOO.util.Event.onDOMReady(function() {
+          bz_fireEvent(document.getElementById('ispatch'), 'change');
+      });
+    </script>
 
     <em>Otherwise, choose a method for determining the content type.</em><br>
     <input type="radio" id="autodetect"
-- 
cgit v1.2.3-24-g4f1b