Skip to content

Commit 38b18fa

Browse files
authored
Merge pull request KartikTalwar#329 from michi88/fix_raw_email_source
fix getting the original source from email
2 parents 16b5914 + 018aedd commit 38b18fa

File tree

1 file changed

+9
-24
lines changed

1 file changed

+9
-24
lines changed

src/gmail.js

Lines changed: 9 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ var Gmail_ = function(localJQuery) {
379379
else {
380380
hash = api.tools.parse_url(window.location.href).th;
381381
}
382-
382+
383383
return hash;
384384
};
385385

@@ -469,7 +469,7 @@ var Gmail_ = function(localJQuery) {
469469
var dom = $("div[role=navigation]").find("[title*='" + api.tools.i18n('inbox') + "']");
470470

471471
if(dom.length > 0) {
472-
if(dom[0].text.indexOf('(') != -1) {
472+
if(dom[0].text.indexOf('(') != -1) {
473473
return parseInt(dom[0].text.split(':')[0].replace(/[^0-9]/g, ''));
474474
}
475475
}
@@ -1558,12 +1558,12 @@ var Gmail_ = function(localJQuery) {
15581558
var page = api.get.current_page();
15591559
var url = window.location.origin + window.location.pathname + '?ui=2&ik=' + api.tracker.ik+'&rid=' + api.tracker.rid + '&view=tl&num=120&rt=1';
15601560
if (!!$('.Dj:visible').find("b:first").text()) {
1561-
url += '&start=' + + parseInt($('.Dj:visible').find("b:first").text() - 1) +
1561+
url += '&start=' + + parseInt($('.Dj:visible').find("b:first").text() - 1) +
15621562
'&start=' + parseInt($('.Dj:visible').find("b:first").text() - 1);
15631563
} else {
15641564
url += '&start=0';
15651565
}
1566-
1566+
15671567
if(page.indexOf('label/') == 0) {
15681568
url += '&cat=' + page.split('/')[1] +'&search=cat';
15691569
} else if(page.indexOf('category/') == 0) {
@@ -1904,33 +1904,18 @@ var Gmail_ = function(localJQuery) {
19041904

19051905
var url = null;
19061906
if(email_id != undefined) {
1907-
url = window.location.origin + window.location.pathname + '?ui=2&ik=' + api.tracker.ik + '&view=om&th=' + email_id;
1907+
// this is normally included also but doesn't seem to be needed: '&attid=0&safe=1&zw'
1908+
url = window.location.origin + window.location.pathname + '?view=att&th=' + email_id + '&disp=comp';
19081909
}
19091910

19101911
return url;
19111912
};
19121913

1913-
api.tools.reformat_source = function(responseText) {
1914-
try {
1915-
// if parsing suceeds, its a HTML-embedded MIME-message
1916-
var parser = new DOMParser();
1917-
var doc = parser.parseFromString(responseText, "text/html");
1918-
var elem = doc.getElementById("raw_message_text");
1919-
var source = elem.innerHTML;
1920-
return source;
1921-
}
1922-
catch (err) {
1923-
// if parsing fails, its a raw mime message
1924-
}
1925-
return responseText;
1926-
};
19271914

19281915
api.get.email_source = function(email_id) {
19291916
var url = api.helper.get.email_source_pre(email_id);
19301917
if (url != null) {
1931-
return api.tools.reformat_source(
1932-
api.tools.make_request(url)
1933-
);
1918+
return api.tools.make_request(url);
19341919
}
19351920
return '';
19361921
};
@@ -1940,7 +1925,7 @@ var Gmail_ = function(localJQuery) {
19401925
var url = api.helper.get.email_source_pre(email_id);
19411926
if (url != null) {
19421927
api.tools.make_request_async(url, 'GET', function(value) {
1943-
callback(api.tools.reformat_source(value));
1928+
callback(value);
19441929
});
19451930
} else {
19461931
callback('');
@@ -2629,7 +2614,7 @@ var Gmail_ = function(localJQuery) {
26292614

26302615
if(composeEl) {
26312616
api.helper.trigger_mouse_click(composeEl);
2632-
2617+
26332618
return true;
26342619
}
26352620
return false;

0 commit comments

Comments
 (0)