[FIX] document_url: Migration was not done properly

This commit is contained in:
Enric Tobella 2023-06-06 09:31:41 +02:00 committed by Bhavesh Heliconia
parent ab87e3e499
commit a6e79f21de
4 changed files with 44 additions and 54 deletions

View File

@ -15,9 +15,8 @@
"assets": {
"web.assets_backend": [
"document_url/static/src/js/url.esm.js",
],
"web.assets_qweb": [
"document_url/static/src/xml/url.xml",
"document_url/static/src/scss/document_url.scss",
],
},
"installable": True,

View File

@ -3,33 +3,28 @@
import {AttachmentBox} from "@mail/components/attachment_box/attachment_box";
import {AttachmentCard} from "@mail/components/attachment_card/attachment_card";
import {patch} from "web.utils";
import {url} from "@web/core/utils/urls";
patch(AttachmentBox.prototype, "document_url/static/src/js/url.js", {
_onAddUrl(event) {
event.preventDefault();
event.stopPropagation();
this.env.bus.trigger("do-action", {
action: "document_url.action_ir_attachment_add_url",
options: {
additional_context: {
active_id: this.messaging.models["mail.chatter"].get(
this.props.chatterLocalId
).threadId,
active_ids: [
this.messaging.models["mail.chatter"].get(
this.props.chatterLocalId
).threadId,
],
active_model: this.messaging.models["mail.chatter"].get(
this.props.chatterLocalId
).threadModel,
},
on_close: this._onAddedUrl.bind(this),
},
});
if (this.env.model) {
this.env.services.action.doAction(
"document_url.action_ir_attachment_add_url",
{
additionalContext: {
active_id: this.env.model.root.data.id,
active_ids: [this.env.model.root.data.id],
active_model: this.env.model.root.resModel,
},
onClose: this._onAddedUrl.bind(this),
}
);
}
},
_onAddedUrl() {
this.trigger("reload");
this.props.record.chatter.refresh();
},
});
@ -41,7 +36,7 @@ patch(AttachmentCard.prototype, "document_url/static/src/js/url.js", {
* @returns {String}
*/
get attachmentUrl() {
return this.env.session.url("/web/content", {
return url("/web/content", {
id: this.attachmentCard.attachment.id,
download: true,
});

View File

@ -1,15 +1,8 @@
.o_AttachmentBox_content {
text-align: center;
display: block;
}
.o_AttachmentList_partialListNonImages {
.o_Attachment_image {
cursor: pointer;
}
.o_Attachment_filename a {
color: #4c4c4c;
.o_image.o_Attachment_url_ico {
background-image: none;
i {
display: block;
text-align: center;
line-height: revert;
}
}
.o_Attachment_url_icon {
margin: 4px 2px 0px 4px;
}

View File

@ -3,6 +3,7 @@
Copyright 2021 Tecnativa - Víctor Martínez
License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
<templates>
<t t-inherit="mail.AttachmentBox" t-inherit-mode="extension" owl="1">
<xpath
expr="//button[hasclass('o_AttachmentBox_buttonAdd')]"
@ -12,10 +13,11 @@
<button
class="o_AttachmentBox_buttonAdd btn btn-link"
type="button"
t-on-click="_onClickAdd"
t-on-click="attachmentBoxView.onClickAddAttachment"
t-att-disabled="!attachmentBoxView.chatter.isTemporary and !attachmentBoxView.chatter.hasWriteAccess"
>
<i class="fa fa-plus-square" />
Add attachments
Attach files
</button>
<button
type="button"
@ -28,29 +30,30 @@
</div>
</xpath>
</t>
<t t-inherit="mail.AttachmentCard" t-inherit-mode="extension" owl="1">
<xpath
expr="//div[hasclass('o_AttachmentCard_details')]/t/div"
position="replace"
>
<a t-att-href="attachmentUrl" target="new">
<t t-esc="attachmentCard.attachment.displayName" />
</a>
</xpath>
<xpath
expr="//div[hasclass('o_AttachmentCard_details')]/t[2]/div"
position="replace"
>
</xpath>
<xpath
expr="//div[hasclass('o_AttachmentCard_asideItemDownload')]"
expr="//button[hasclass('o_AttachmentCard_asideItemDownload')]"
position="attributes"
>
<attribute
name="t-if"
>attachmentCard.attachment.mimetype != 'application/link'</attribute>
</xpath>
<xpath
expr="//button[hasclass('o_AttachmentCard_asideItemDownload')]"
position="after"
>
<button
class="o_AttachmentCard_asideItem o_AttachmentCard_asideItemDownload btn d-flex justify-content-center align-items-center w-100 h-100 rounded-0 bg-300"
t-on-click="() => window.open(attachmentUrl, '_blank')"
title="Open"
t-if="attachmentCard.attachment.mimetype === 'application/link'"
target="new"
>
<i class="fa fa-link" role="img" aria-label="Open" />
</button>
</xpath>
<xpath expr="//div[hasclass('o_AttachmentCard_image')]" position="attributes">
<attribute
name="t-if"
@ -58,7 +61,7 @@
</xpath>
<xpath expr="//div[hasclass('o_AttachmentCard_image')]" position="before">
<a
class="o_Attachment_url_icon"
class="o_Attachment_url_ico o_image flex-shrink-0 m-1"
t-att-href="attachmentUrl"
target="new"
t-if="attachmentCard.attachment.mimetype=='application/link'"