Skip to content

fix: build folder path from Backend.Path and silence null-revision bindings in revision file tree - #2595

Merged
love-linger merged 1 commit into
sourcegit-scm:developfrom
gadfly3173:fix/revision-file-tree-path-and-binding
Jul 31, 2026
Merged

fix: build folder path from Backend.Path and silence null-revision bindings in revision file tree#2595
love-linger merged 1 commit into
sourcegit-scm:developfrom
gadfly3173:fix/revision-file-tree-path-and-binding

Conversation

@gadfly3173

Copy link
Copy Markdown
Contributor
  • RevisionFileTreeView.SetSearchResultAsync: concatenate Backend.Path instead of stringifying the Backend Object, whose default ToString() leaked "SourceGit.Models.Object/" into child node paths and corrupted the DirHistories window title.

  • RevisionFiles / Blame: add FallbackValue={x:Null} to Commit.SHA / Revision.SHA. These commits are loaded asynchronously and are null when the view first binds, which logged "Value is null" binding errors.

Before:
image
image

…ndings in revision file tree

- RevisionFileTreeView.SetSearchResultAsync: concatenate Backend.Path
  instead of stringifying the Backend Object, whose default ToString()
  leaked "SourceGit.Models.Object/" into child node paths and corrupted
  the DirHistories window title.

- RevisionFiles / Blame: add FallbackValue={x:Null} to Commit.SHA /
  Revision.SHA. These commits are loaded asynchronously and are null
  when the view first binds, which logged "Value is null" binding errors.
@love-linger love-linger self-assigned this Jul 31, 2026
@love-linger love-linger added the bug Something isn't working label Jul 31, 2026
@love-linger
love-linger merged commit 24fa13f into sourcegit-scm:develop Jul 31, 2026
14 checks passed
love-linger added a commit that referenced this pull request Jul 31, 2026
Use `ContentControl.DataTemplates` instead of `FallbackValue` when the child
control does not need to access the parent's DataContext

Signed-off-by: leo <longshuang@msn.cn>
@gadfly3173
gadfly3173 deleted the fix/revision-file-tree-path-and-binding branch July 31, 2026 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants