Skip to content
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

🐛 [BUG] Certain Items Returned from Search Results with Negative ID #498

Open
1 task done
hao-lh opened this issue Jun 16, 2023 · 6 comments
Open
1 task done
Labels

Comments

@hao-lh
Copy link

hao-lh commented Jun 16, 2023

Your Environment

  • mas version: 1.8.6
  • macOS version (system_profiler SPSoftwareDataType -detailLevel mini):
    System Version: macOS 10.15.7 (19H2)
    Kernel Version: Darwin 19.6.0
    Time since boot: 1:30

mas Install Method

  • brew install mas (homebrew-core)
  • [] mas-cli/tap
  • [] .pkg installer from releases
  • [] Built from source
    • Fork/branch: ? (e.g. mas-cli/main)
    • Xcode version: 10.?

Describe the Bug

When use keyword with mas to search apps, some items returned with a negative MAS ID.

To Reproduce

Steps to reproduce the behavior:

  1. mas search xcode
  2. some items from search results returned a negative ID
  3. already highlighted details from below screenshot
  4. should be type overflow

Screenshots, Terminal Output

IMG_2023-06-16_12-09-04
@bjesuiter
Copy link

I have the same problem with "Lockera Widgets".

mas search "Lockera Widgets" returns -2123114745 Lockera Widgets (1.6)

and trying to install this with mas "Lockera Widgets", id: -2123114745 fails.

@voodooGQ
Copy link

voodooGQ commented Feb 3, 2024

Having the same issue with "iBar" showing up as -2146090692 iBar-Menubar icon control tool (1.2.4)

@tvlieg
Copy link

tvlieg commented Feb 6, 2024

I have the same with "Noteshelf 3":
-2131199389 Noteshelf 3 (1.4.8)

@jandamm
Copy link

jandamm commented Feb 28, 2024

Same here for StopTheMadness Pro and Keymapp.

Seems like the InstallOptions need an Int instead of UInt64:

let appIds: [UInt64]

@deed02392
Copy link

deed02392 commented Jul 12, 2024

I think it's the other way around @jandamm – we want UInt64 throughout Search:
https://github.com/mas-cli/mas/blob/main/Sources/MasKit/Controllers/MasStoreSearch.swift#L15

edit: it's mainly an output/formatting bug, fixed in: #479UInt64 would still be a more proper type though

@rgoldberg
Copy link
Contributor

@hao-lh Can we mark this as a duplicate of #478?

There's an initial PR (#479) to fix #478, and there should be follow up PRs to clean up app IDs throughout the code later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants