Page MenuHomePhorge

FileUpload: show File page instead of binary file
ClosedPublic

Authored by valerio.bozzolan on Mar 12 2023, 21:08.

Details

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

Diff Detail

Repository
rP Phorge
Branch
T15165-popup-link-to-file-page
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 130
Build 130: arc lint + arc unit

Event Timeline

avivey added inline comments.
webroot/rsrc/js/core/FileUpload.js
41

I think we call these "monograms".

rename getPageID() to getMonogram() and add few inline comments

This revision is now accepted and ready to land.Mar 17 2023, 08:06