fork-dev / Tracker

Bug and issue tracker for Fork for Mac
512 stars 12 forks source link

Repository Statistics showing empty charts. #1725

Open rogerwcpt opened 2 years ago

rogerwcpt commented 2 years ago

I have a few very old repositories and their statistics are fine. However for repositories created in the last 6 months or so with large amounts of commits, don't have any history. The repo is my company Git repo, which is from Azure DevOps.

image

image

DominikPalo commented 1 year ago

I'm having the same issue... The issue doesn’t affect all repositories - the statistics functionality works fine in some repositories, but in others, it doesn’t work. In my case, I don't see any correlation with the age of repositories or the number of commits.

The issue is still present in the latest version (2.25.5)

image

I can see many JS errors in the ~/Library/Logs/Fork.log log file:

⚠️[RepositoryStatisticsViewController.swift:180] > Error Domain=WKErrorDomain Code=4 "A JavaScript exception occurred" UserInfo={WKJavaScriptExceptionLineNumber=1, WKJavaScriptExceptionMessage=ReferenceError: Can't find variable: updateLineChart, WKJavaScriptExceptionColumnNumber=16, WKJavaScriptExceptionSourceURL=undefined, NSLocalizedDescription=A JavaScript exception occurred}
DominikPalo commented 1 year ago

@DanPristupov do you have any updates? It used to be a really useful feature, but it still doesn't work on most of our repositories 😟

DanPristupov commented 1 year ago

@DominikPalo can you find any open repository which has a problem with statistics, so I could reproduce the problem?

DominikPalo commented 1 year ago

Of course, for example, this one: https://github.com/slidoapp/oauth2-swift/

image
DanPristupov commented 1 year ago

MacOS 12.6.3 :(

Screenshot 2023-04-27 at 11 27 23

I'll try on macOS 13.

DanPristupov commented 1 year ago

Could you check if the statistics is fixed in Fork 2.30?

DominikPalo commented 1 year ago

@DanPristupov thank you, I've just tested v2.30 and the initial attempt failed as before - with an empty chart, so I closed the statistics window, reopened it again for the same repo and the second attempt worked. So, it appears that the issue is not completely resolved yet

Logs:

 /$$$$$$$$                     /$$
| $$_____/                    | $$             Fork 2.30
| $$        /$$$$$$   /$$$$$$ | $$   /$$
| $$$$$    /$$__  $$ /$$__  $$| $$  /$$/       Git: internal
| $$__/   | $$  \ $$| $$  \__/| $$$$$$/        Update Channel: develop
| $$      | $$  | $$| $$      | $$_  $$
| $$      |  $$$$$$/| $$      | $$ \  $$          https://fork.dev
|__/       \______/ |__/      |__/  \__/
.
.
.
⚠️[RepositoryStatisticsViewController.swift:180] > Error Domain=WKErrorDomain Code=4 "A JavaScript exception occurred" UserInfo={WKJavaScriptExceptionLineNumber=1, WKJavaScriptExceptionMessage=ReferenceError: Can't find variable: updateLineChart, WKJavaScriptExceptionColumnNumber=16, WKJavaScriptExceptionSourceURL=undefined, NSLocalizedDescription=A JavaScript exception occurred}
DominikPalo commented 1 year ago

@DanPristupov it seems the issue is back - I have version 2.34.1 and the chars are empty again 😟

image
⚠️[RepositoryStatisticsViewController.swift:180] > Error Domain=WKErrorDomain Code=4 "A JavaScript exception occurred" UserInfo={WKJavaScriptExceptionLineNumber=1, WKJavaScriptExceptionMessage=ReferenceError: Can't find variable: updateLineChart, WKJavaScriptExceptionColumnNumber=16, WKJavaScriptExceptionSourceURL=undefined, NSLocalizedDescription=A JavaScript exception occurred}
DominikPalo commented 3 months ago

Sorry for pinging again - are there any plans/ETA to fix the issue? It's still a problem for us, we are always on the latest version, but the Statistics doesn't work for most of our repos.

BTW, closing the empty "Statistics" window and reopening it helps most of the time - only the first attempt always fails. So, at least we have a workaround. Based on this behavior, I assume it shouldn't be a difficult fix.