1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 14:52:41 +01:00

FileUpload: show File page instead of binary file

Summary:
This modification improves a single link shown to the user.

The goal is to make it easier to change file permissions or
file name or whatever just after the upload of the file itself.

Before this change, doing that was difficult because clicking
on the popup would send you back to the binary file only.

Closes T15165

Test Plan:
- upload a file using drag & drop (for example in a comment)
- click on the popup's link
- you see the File page, not the binary file

Reviewers: O1 Blessed Committers, avivey

Reviewed By: O1 Blessed Committers, avivey

Subscribers: avivey, speck, tobiaswiese, Matthew, Cigaryno

Maniphest Tasks: T15165

Differential Revision: https://we.phorge.it/D25074
This commit is contained in:
Valerio Bozzolan 2023-03-17 08:29:35 +01:00
parent 5cba56182f
commit 4f356838be

View file

@ -38,6 +38,20 @@ JX.install('PhabricatorFileUpload', {
return this; return this;
}, },
/**
* Get the File Monogram - like 'F123'
*/
getMonogram: function() {
return 'F' + this.getID();
},
/**
* Get the File page URI - like '/F123'
*/
getPageURI: function() {
return '/' + this.getMonogram();
},
setChunks: function(chunks) { setChunks: function(chunks) {
var chunk; var chunk;
for (var ii = 0; ii < chunks.length; ii++) { for (var ii = 0; ii < chunks.length; ii++) {
@ -101,7 +115,7 @@ JX.install('PhabricatorFileUpload', {
switch (this.getStatus()) { switch (this.getStatus()) {
case 'done': case 'done':
var link = JX.$N('a', {href: this.getURI()}, 'F' + this.getID()); var link = JX.$N('a', {href: this.getPageURI()}, this.getMonogram());
content = [ content = [
JX.$N('strong', {}, ['Upload Complete (', link, ')']), JX.$N('strong', {}, ['Upload Complete (', link, ')']),