Thunderbird conversations support

README

@@ -1 +1,51 @@
 A Thunderbird add-on to help with bugmail
+
+This fork
+=========
+
+This fork adds preliminary support for Thunderbird Conversations
+(https://addons.mozilla.org/en-US/thunderbird/addon/gmail-conversation-view/),
+so far working only for Bugzilla and somewhat quick&dirty.
+
+Development
+=============
+
+To test the development version in thunderbird:
+
+- uninstall „normal” Bugmail if it is installed (using Thunderbird Add-ons Manager), 
+  restart Thunderbird so it acknowledges uninstall
+
+- in your extensions directory 
+  (~/.thunderbird/<Profile Name>/extensions/ or similar)
+  create the file named 
+      {32d96adf-23fc-403d-a1c8-e5cf6a019bee}
+  (yes, use this ugly name, it must match <em:id> from install.rdf)
+  which body is 
+      /path/to/the/devel/bugmail/
+  (replace with true full path to the directory this file lies in)
+
+- restart thunderbird again.
+
+More details: https://developer.mozilla.org/en-US/docs/Extensions/Thunderbird/Building_a_Thunderbird_extension_7:_Installation
+
+Some useful development tricks:
+
+- set javascript.options.showInConsole = true in about:config
+
+- set extensions.logging.enabled = true
+
+- start thunderbird with 
+
+    thunderbird --purgecaches
+
+Packaging
+=============
+
+Simply zip the whole extension folder, for example
+
+    cd path/to/bugmail
+    zip -r ../bugmail.xpi chrome/ chrome.manifest install.rdf LICENSE README
+
+
+
+

chrome.manifest

@@ -3,5 +3,3 @@ locale bugmail en-US chrome/locale/en-US/
 skin bugmail    default chrome/skin/
 overlay chrome://messenger/content/messageWindow.xul chrome://bugmail/content/overlay.xul appversion>=3.0b3
 overlay chrome://messenger/content/messenger.xul chrome://bugmail/content/overlay.xul appversion>=3.0b3
-overlay chrome://messenger/content/messageWindow.xul chrome://bugmail/content/overlay-tb2.xul appversion>=2.0
-overlay chrome://messenger/content/messenger.xul chrome://bugmail/content/overlay-tb2.xul appversion>=2.0

chrome/content/binding.js

@@ -30,6 +30,7 @@ function xsltBinding() {
 xsltBinding.prototype = {
 
   init: function(pid, psrc, pxsl, cb, params) {
+      console.log("bugmail/binding: init");
     this.id = pid;
     this.loaded = 0;
     this.needed = 2;
@@ -59,6 +60,7 @@ xsltBinding.prototype = {
   },
 
   initWithSrc: function(pid, psrc, pxsl, cb, params) {
+      console.log("bugmail/binding: initWithSrc. psrc=" + psrc);
     this.id = pid;
     this.loaded = 0;
     this.needed = 1;
@@ -79,16 +81,19 @@ xsltBinding.prototype = {
   },
 
   initWithDoc: function(doc, pid, psrc, pxsl, cb, params) {
+      console.log("bugmail/binding: initWithDoc");
     this.doc = doc;
     this.init(pid, psrc, pxsl, cb, params);
   },
 
   initSrcWithDoc: function(doc, pid, psrc, pxsl, cb, params) {
+      console.log("bugmail/binding: initSrcWithDoc");
     this.doc = doc;
     this.initWithSrc(pid, psrc, pxsl, cb, params);
   },
 
   refresh: function() {
+      console.log("bugmail/binding: refresh. this.src=" + this.src + ", this.srcreq=" + this.srcreq);
     this.xsl = this.xslreq.responseXML;
     if (!this.src)
       this.src = this.srcreq.responseXML;

chrome/content/bugzilla.js

@@ -106,8 +106,12 @@ var bugzillaEngine = {
     }
     else { // Extract bugzilla URI from mail content
        var doc = document.getElementById('messagepane').contentDocument;
-       uri = doc.querySelector("a[href*=show_bug]").href +
-             "&ctype=xml&excludefield=attachmentdata";
+        uri = doc.querySelector("a[href*=show_bug]").href;
+        // Note: on URIs containing comments (/path/to/show_bug.cgi?id=3333#c6 etc)
+        // bugzilla happens to ignore ctype=xml and return html. 
+        // Therefore we drop fragment if we happened to get one
+        uri = uri.replace(/#.*$/, "");
+        uri = uri + "&ctype=xml&excludefield=attachmentdata";
     }
 
     //uri = "https://bugzilla.mozilla.org/show_bug.cgi?id=379306&ctype=xml";