Fix calls of bugzilla.updateAttachMimeType to the new API.

Bugzilla guys are working on fix, but they provided workaround.

Fixes #124

1 files changed, 19 insertions, 5 deletions

diff --git a/data/rhlib/fixingAttMIME.js b/data/rhlib/fixingAttMIME.js
index 220b7ad..56e10ba 100644
--- a/data/rhlib/fixingAttMIME.js
+++ b/data/rhlib/fixingAttMIME.js
@@ -44,24 +44,37 @@ function XMLRPCcallback() {
  * this change };
  *
  */
-function fixAttachById(id, type, email) {
+function fixAttachById(bugId, id, type, email) {
+  myDebug("bugId = " + bugId + ", id = " + id);
   if (type === undefined) {
     type = "text/plain";
   }
   if (email === undefined) {
-    email = false;
+    email = 0;
+  }
+
+  // Logic of nomail is negative ... turn around
+  email = !email;
+
+  if (email === true) {
+    email = 1;
+  }
+  else {
+    email = 0;
   }
 
   // https://bugzilla.redhat.com/\
   // docs/en/html/api/extensions/compat_xmlrpc/code/webservice.html
   // test on https://bugzilla.redhat.com/show_bug.cgi?id=485145
   var params = {
+    'id': bugId,
     'attach_id' : id,
     'mime_type' : type,
-    'nomail' : !email
+    'nomail' : email
   };
 
-  makeJSONRPCCall("bugzilla.updateAttachMimeType", params, XMLRPCcallback);
+  // makeJSONRPCCall("bugzilla.updateAttachMimeType", params, XMLRPCcallback);
+  makeJSONRPCCall("RedHat.updateAttachMimeType", params, XMLRPCcallback);
   reqCounter++;
 }
 
@@ -75,6 +88,7 @@ function fixAttachById(id, type, email) {
 function addTextLink(att) {
   var elemS = att.element.getElementsByTagName("td");
   var elem = elemS[elemS.length - 1];
+  myDebug("att = " + att.toSource());
   createDeadLink("addFix2TextLink", "text", elem, fixAttachById,
-      [ att.id ], "br");
+      [ att.bugId, att.id ], "br");
 }