mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Fix linked-URL attachments not being saved to groups without files

Browse source 
It looks like this may have been broken for years.
This commit is contained in:
Dan Stillman 2019-01-08 02:07:15 -05:00
parent 506ed313da
commit 5a6a772ca2
2 changed files with 53 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

46
test/tests/translateTest.js
View file

@ -6,7 +6,7 @@ Components.utils.import("resource://gre/modules/osfile.jsm");
* @param {String} translatorType - "import" or "web"
* @param {Object} items - items as translator JSON
*/
function saveItemsThroughTranslator(translatorType, items) {
function saveItemsThroughTranslator(translatorType, items, translateOptions = {}) {
let tyname;
if (translatorType == "web") {
tyname = "Web";
@ -35,7 +35,7 @@ function saveItemsThroughTranslator(translatorType, items) {
" item.complete();\n"+
" }\n"+
"}"));
return translate.translate().then(function(items) {
return translate.translate(translateOptions).then(function(items) {
if (browser) Zotero.Browser.deleteHiddenBrowser(browser);
return items;
});
@ -404,7 +404,47 @@ describe("Zotero.Translate", function() {
assert.equal(newItems[0].getField("title"), "Container Item");
assert.equal(newItems[0].getAttachments().length, 0);
});
it("import translators should save linked-URL attachments with savingAttachments: false", async function () {
var json = [
{
itemType: "journalArticle",
title: "Parent Item",
attachments: [
// snapshot: false
{
title: "Link",
mimeType: "text/html",
url: "http://example.com",
snapshot: false
},
// linkMode (used by RDF import)
{
title: "Link",
mimeType: "text/html",
url: "http://example.com",
linkMode: Zotero.Attachments.LINK_MODE_LINKED_URL
}
]
}
];
var newItems = itemsArrayToObject(
await saveItemsThroughTranslator(
"import",
json,
{
saveAttachments: false
}
)
);
var attachmentIDs = newItems["Parent Item"].getAttachments();
assert.lengthOf(attachmentIDs, 2);
var attachments = await Zotero.Items.getAsync(attachmentIDs);
assert.equal(attachments[0].attachmentLinkMode, Zotero.Attachments.LINK_MODE_LINKED_URL);
assert.equal(attachments[1].attachmentLinkMode, Zotero.Attachments.LINK_MODE_LINKED_URL);
});
it('web translators should set accessDate to current date', function* () {
let myItem = {
"itemType":"webpage",