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

2 participants