Don't render dot for commit which have no space property in revision graph (fine fix for #11612)
Quoting Jens from #11612#note-9 where he suggested to not add a commit node on the revision graph when it doesn't have a space property, meaning it's not attached to any branch anymore:
We recently had the same problem - graph rendering broke completely for a repository due to JS errors because some commits were missing the
Turns out that on a fresh install, where the repository in question was re-imported, the problem didn't occur, because the problematic commits didnt show up at all. Further investigation showed that those commits were basically abandoned and not connected to any branch.
I think not rendering these commits in the graph at all is much better than putting them randomly on the first branch as the patch above does.
Here's the patch for 1.4:
Comments are welcome.
Updated by Stephane Lapie almost 6 years ago
+1, this needs a fix.
Updated by Toshi MARUYAMA almost 6 years ago
- Description updated (diff)