marticliment / UniGetUI

UniGetUI: The Graphical Interface for your package managers. Could be terribly described as a package manager manager to manage your package managers
https://www.marticliment.com/unigetui/
MIT License
13.1k stars 450 forks source link

[BUG] Opening a bundle filters all non installed packages once, but shows after any refresh #2635

Open eusousu opened 2 months ago

eusousu commented 2 months ago

Please confirm these before moving forward

UniGetUI Version

3.1.1

Windows version, edition, and architecture

Windows 11 Pro 22631.3958

Describe your issue

As long as it seems to me if the package is not installed it does not show by default when opening a bundle .json file. When opening you will be met with a blank screen and the view subtitle will say X packages were found, 0 of which match the specified filters where X is the number of packages on the bundle.json.

I don't think this is expected. But the failure doesn't seem constant, sometimes it opens normally. Sometimes some packages show and others don't. I can test further if given new approaches.

Steps to reproduce the issue

Open a bundle with uninstalled package. See that no package is shown.

Try clear source selection and select all again, package shows.

Loading the bundle.json once again also make the packages appear.

I think, by reading the logs, that it seems related to the order of operations, maybe as it creates the options after when the package does not exist locally, it can't render when it loads and only after a refresh or reload it shows.

UniGetUI Log

[14/08/2024 13:42:45] Adding native package with id=Google.Chrome to bundle as a VALID package...
[14/08/2024 13:42:45] Creating new instance of InstallationOptions for package UniGetUI.PackageEngine.PackageClasses.ImportedPackage, as no instance was found in cache

Package Managers Logs

Not related

Relevant information

No response

Screenshots and videos

clearing and selecting sources renders the package: https://github.com/user-attachments/assets/566941d9-fc38-46a7-874c-93dccf870b27

marticliment commented 2 months ago

I will investigate

eusousu commented 2 months ago

Is this fix on beta? I was testing the other issue and it is still happening :(

Here is a video showing it happening

https://github.com/user-attachments/assets/4839f62a-046b-4fdb-a133-fafc9b78ab69

marticliment commented 2 months ago

Yes, this issue should have been fixed, and, at least to what I could reproduced it got fixed

eusousu commented 2 months ago

It's also happening in a place I (at least that I remember) didn't notice before, in the "Discover Packages" when searching. Sometimes I need to search twice (and now clearing and selecting again do not always work) as the same issue that was happening with bundles occur, I search and the page subtitle becomes "X packages were found, 0 of which match the specific filters" and no package is shown

marticliment commented 2 months ago

On the discover packages page you may need to change the "Search Mode" to "Show similar packages"

marticliment commented 2 months ago

If the issue is happening only on the Discover PAckages page then it was fixed

eusousu commented 2 months ago

No no, I send a video with it happening on bundles here

Is this fix on beta? I was testing the other issue and it is still happening :(

Here is a video showing it happening

At least when selecting to add a package to a bundle It still happens

marticliment commented 2 months ago

Does it happen always? I am not able to reproduce the issue

eusousu commented 2 months ago

I tested and could record it not happening and happening. So I think It's not always. But it seems to be more consistent when you first open unigetui.

(I sped up because of the file size cap) https://github.com/user-attachments/assets/99f5f7ec-2cb1-4a63-91b8-c5cadf3e095b

ninbura commented 2 months ago

The problem isn't exclusive to non-installed packages, nor is it exclusive to the bundles tab. Frequently applications will not list on any tab of the application, and you have to check and uncheck winget to make them show.

I've been able to replicate this on 8 systems across 5 different chipsets.

https://github.com/user-attachments/assets/1d642b89-8727-441d-a5cd-60b7cbbbd7b7

ninbura commented 2 months ago

Apologies, I missed the context that it was working in the latest beta version. It appears to be fixed on my end when running v3.1.2-beta0.