View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0006012 | SOGo Connector | with SOGo | public | 2024-08-27 09:17 | 2024-08-29 20:30 |
Reporter | cz | Assigned To | sebastien | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | Windows | OS | Windows 10 | OS Version | 22H2 |
Summary | 0006012: sogo-connector not working with Thunderbird 128 | ||||
Description | TB128 won't work with the latest sogo-connector [1], [Exception... "File error: Not found" nsresult: "0x80520012 (NS_ERROR_FILE_NOT_FOUND)" location: "JS frame :: chrome://sogo-connector/content/messenger/startup-overlay.js :: <TOP_LEVEL> :: line 22" data: no] startup-overlay.js:22:32 and [Exception... "File error: Not found" nsresult: "0x80520012 (NS_ERROR_FILE_NOT_FOUND)" location: "JS frame :: chrome://sogo-connector/content/calendar/calendar-overlay.js :: <TOP_LEVEL> :: line 21" data: no] calendar-overlay.js:21:32 It seems those errors mostly stem from the old variable declaration of "Services", citing compatibility issues with "Services.jsm" as described in Thunderbird's documentation [2]. e.g. "var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");" in chrome/content/sogo-connector/messenger/startup-overlay.js which would be "var Services = globalThis.Services || ChromeUtils.import("resource://gre/modules/Services.jsm").Services;" in the newer version according to TB's documentation [2] After manually patching all occurrences of these declarations, I still get other errors, because there's more to do than that: converter.convertToInputStream is not a function sogoWebDAV.js:203:36 Someone with better knowledge of the code base might be able to relatively easily patch these things, I reckon. [1] https://github.com/Alinto/sogo-connector/tree/tb115 | ||||
Steps To Reproduce |
| ||||
Tags | No tags attached. | ||||
Hi, Can you have a try with branch https://github.com/Alinto/sogo-connector/tree/tb128 ? Sebastien |
|
Hi, thanks for the quick reply! I built it via the tb128 branch and it works on my Windows test machine with a clean profile. Calendars and address books are syncing and are working. But I still have some errors in the console... first one: NS_ERROR_FILE_NOT_FOUND: Component returned failure code: 0x80520012 (NS_ERROR_FILE_NOT_FOUND) [nsIFileInputStream.init] and this one: ReferenceError: MozXULElement is not definededitMenuOverlay.js:45:20 After some quick testing, it seems everything is working though. |
|
I will have a look on those errors Sebastien |
|
I cannot reproduce those errors. For the first one I have added logs. Can you try with the attached patch ? fix.patch (1,348 bytes)
diff --git i/chrome/content/sogo-connector/addressbook/folder-handler.js w/chrome/content/sogo-connector/addressbook/folder-handler.js index f1dcdfe..ebe6098 100644 --- i/chrome/content/sogo-connector/addressbook/folder-handler.js +++ w/chrome/content/sogo-connector/addressbook/folder-handler.js @@ -103,6 +103,8 @@ AddressbookHandler.prototype = { directory.setBoolValue("readOnly", readOnly); } catch(e) { dump(e); + console.error(newDirs); + console.error(directory); } //if (readOnly) { diff --git i/chrome/content/sogo-connector/calendar/calendar-overlay.js w/chrome/content/sogo-connector/calendar/calendar-overlay.js index e99adb9..f99c599 100644 --- i/chrome/content/sogo-connector/calendar/calendar-overlay.js +++ w/chrome/content/sogo-connector/calendar/calendar-overlay.js @@ -57,8 +57,7 @@ registerJs = () => { "chrome://sogo-connector/content/general/sync.addressbook.groupdav.js", "chrome://sogo-connector/content/messenger/folders-update.js", "chrome://sogo-connector/content/global/sogo-config.js", - "chrome://global/content/globalOverlay.js", - "chrome://global/content/editMenuOverlay.js"], _this); + "chrome://global/content/globalOverlay.js"], _this); jsInclude(["chrome://sogo-connector/content/general/subscription-utils.js"], window); } |
|
Newest version with patch is working with TB128, but unsubscribing calendars does not work. Unfortunately, no errors in the console, it's just not happening... |
|
I have fixed the issue, can you have a try with uptodate tb128 branch ? Sebastien |
|
Ah, yes, works perfectly now, subscribing and unsubscribing both work as intended, thank you very much for the quick fix! |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2024-08-27 09:17 | cz | New Issue | |
2024-08-27 12:53 | sebastien | Assigned To | => sebastien |
2024-08-27 12:53 | sebastien | Status | new => assigned |
2024-08-27 12:54 | sebastien | Note Added: 0017830 | |
2024-08-27 12:54 | sebastien | Status | assigned => feedback |
2024-08-27 15:41 | cz | Note Added: 0017833 | |
2024-08-27 15:41 | cz | Status | feedback => assigned |
2024-08-27 17:16 | sebastien | Note Added: 0017835 | |
2024-08-27 19:23 | sebastien | Note Added: 0017837 | |
2024-08-27 19:23 | sebastien | File Added: fix.patch | |
2024-08-27 19:26 | sebastien | Status | assigned => feedback |
2024-08-29 07:52 | cz | Note Added: 0017843 | |
2024-08-29 07:52 | cz | Status | feedback => assigned |
2024-08-29 15:29 | sebastien | Note Added: 0017844 | |
2024-08-29 15:29 | sebastien | Status | assigned => feedback |
2024-08-29 15:37 | cz | Note Added: 0017845 | |
2024-08-29 15:37 | cz | Status | feedback => assigned |
2024-08-29 20:30 | sebastien | Status | assigned => resolved |
2024-08-29 20:30 | sebastien | Resolution | open => fixed |