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 |