-
Notifications
You must be signed in to change notification settings - Fork 482
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixing Source List Displaying Empty Items #810
Fixing Source List Displaying Empty Items #810
Conversation
normalSafeApiCall { | ||
// Extra normalSafeApiCall since .toSet() iterates. | ||
_currentLinks.postValue(currentLinks.toSet()) | ||
if (it.first?.url?.isNotEmpty() == true) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not account for ExtractorUri (local files), in that case it.first would be null
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I will try other fix methods
Perhaps this would be better implemented in mainapi and we can do warnings and filtering there instead without adding complexity to the already complex link loading. Better to fix it at the source, since links are used by other stuff like the downloader, copy links and external players. |
This should instead be in RepoLinkGenerator |
This reverts commit 8ba4879.
… feasible to handle it in the RepoLinkGenerator)
|
🙏 Nice, this should fix all the empty links |
No description provided.