Page MenuHomePhorge
Differential D232

Let Filehandler react to eventContent modification
ClosedPublic
Actions

Authored by nannanko on Sep 18 2025, 6:32 AM.
Tags
None
Referenced Files
F43164583: D232.1771376829.diff
Mon, Feb 16, 5:07 PM
F43164254: D232.1771376767.diff
Mon, Feb 16, 5:06 PM
F43164144: D232.1771376747.diff
Mon, Feb 16, 5:05 PM
F43139033: D232.1771371703.diff
Mon, Feb 16, 3:41 PM
F43139023: D232.1771371702.diff
Mon, Feb 16, 3:41 PM
F43139017: D232.1771371701.diff
Mon, Feb 16, 3:41 PM
F43082197: D232.1771360057.diff
Mon, Feb 16, 12:27 PM
F43082192: D232.1771360056.diff
Mon, Feb 16, 12:27 PM
View All 12 Files
Subscribers
None

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Maniphest Tasks
T152: FileHandler is not reactive against change in the mxc uri
Commits
rK57ef15344f92: Let Filehandler react to eventContent modification
Summary

Now FileHandler.qml can react to eventContent modification.

Type: fix

Test Plan

Varify unit tests pass.

Diff Detail

Repository
rK kazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

nannanko created this revision.Sep 18 2025, 6:32 AM
Owners added a reviewer: O1: the Kazv Project.Sep 18 2025, 6:32 AM
Herald added a reviewer: servant. · View Herald TranscriptSep 18 2025, 6:32 AM

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

nannanko edited the summary of this revision. (Show Details)Sep 18 2025, 6:32 AM
nannanko removed a reviewer: servant.
nannanko added a task: T152: FileHandler is not reactive against change in the mxc uri.Sep 18 2025, 6:35 AM
Harbormaster completed remote builds in B428: Diff 681.Sep 18 2025, 6:45 AM
nannanko requested review of this revision.Sep 18 2025, 6:45 AM
nannanko updated this revision to Diff 682.Sep 18 2025, 4:23 PM

Delete useless code.

Harbormaster completed remote builds in B429: Diff 682.Sep 18 2025, 4:37 PM
nannanko mentioned this in T152: FileHandler is not reactive against change in the mxc uri.Sep 18 2025, 11:46 PM
tusooa added a comment.Sep 20 2025, 7:19 PM

mxcUri, sha256, key, iv can be combined into one object.

properties can be turned into a helper function that returns the object.

nannanko updated this revision to Diff 687.Sep 22 2025, 6:55 AM

Use helper function instead of useless properties

Harbormaster completed remote builds in B434: Diff 687.Sep 22 2025, 7:08 AM
tusooa requested changes to this revision.Sep 27 2025, 6:58 PM
tusooa added inline comments.
src/contents/ui/FileHandler.qml
71
148–151

mediaId property has been removed, I don't think it can still be accessed

This revision now requires changes to proceed.Sep 27 2025, 6:58 PM
tusooa retitled this revision from Let Filehandler react to eventCnotent modification to Let Filehandler react to eventContent modification.Oct 4 2025, 7:02 PM
nannanko updated this revision to Diff 727.Oct 11 2025, 7:05 PM

Fixed incorrect use of mediaId, add test for auto track downloading job.

Harbormaster completed remote builds in B466: Diff 727.Oct 11 2025, 7:27 PM
nannanko updated this revision to Diff 782.Dec 27 2025, 7:10 PM
nannanko marked 2 inline comments as done.

Rebase to servant

Harbormaster failed remote builds in B505: Diff 782!Dec 27 2025, 7:10 PM
nannanko updated this revision to Diff 791.Jan 3 2026, 6:18 PM

Rebase to servant

Harbormaster completed remote builds in B510: Diff 791.Jan 3 2026, 7:08 PM
tusooa accepted this revision.Jan 6 2026, 9:15 PM
This revision is now accepted and ready to land.Jan 6 2026, 9:15 PM
Closed by commit rK57ef15344f92: Let Filehandler react to eventContent modification. · Explain WhyJan 7 2026, 4:06 AM
This revision was automatically updated to reflect the committed changes.
nannanko added a commit: rK57ef15344f92: Let Filehandler react to eventContent modification.

Revision Contents
Changeset List

Diff 803

View Options

src/contents/ui/FileHandler.qml

Loading...
View Options

src/contents/ui/event-types/Image.qml

Loading...
View Options

src/contents/ui/event-types/MediaBubble.qml

Loading...
View Options

src/tests/quick-tests/test-helpers/KazvIOJobMock.qml

Loading...
View Options

src/tests/quick-tests/test-helpers/KazvIOManagerMock.qml

Loading...
View Options

src/tests/quick-tests/tst_FileHandler.qml

Loading...