Skip to content

Fix Mapepire NotInstalled detection when no remote jars are found#3325

Merged
sebjulliand merged 1 commit into
codefori:masterfrom
bobcozzi:fix/mapepire-remoteVersions-empty-check
Jun 24, 2026
Merged

Fix Mapepire NotInstalled detection when no remote jars are found#3325
sebjulliand merged 1 commit into
codefori:masterfrom
bobcozzi:fix/mapepire-remoteVersions-empty-check

Conversation

@bobcozzi

Copy link
Copy Markdown
Contributor

Summary

Fixes Mapepire remote state detection when no remote server JARs exist.

Problem

remoteVersions is always an array. The previous check used if (!remoteVersions), which never evaluates to true, so the no-results case could not be classified as NotInstalled.

Change

  • In src/api/components/mapepire/index.ts, changed:
    • if (!remoteVersions)
    • to if (remoteVersions.length === 0)

Impact

  • Correctly reports NotInstalled when no matching remote Mapepire JARs are present.
  • No behavior changes for NeedsUpdate or Installed paths.

Validation

  • TypeScript diagnostics for src/api/components/mapepire/index.ts: no errors.

@bobcozzi bobcozzi temporarily deployed to testing_environment June 23, 2026 20:53 — with GitHub Actions Inactive

@sebjulliand sebjulliand left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks ok @bobcozzi , but it seems you created this branch on top of the one that exports the system version thing.

You may want to start from the main branch instead.

@bobcozzi bobcozzi force-pushed the fix/mapepire-remoteVersions-empty-check branch from d6471d9 to bf5e971 Compare June 24, 2026 12:55
@bobcozzi bobcozzi temporarily deployed to testing_environment June 24, 2026 12:55 — with GitHub Actions Inactive
@bobcozzi

Copy link
Copy Markdown
Contributor Author

It looks ok @bobcozzi , but it seems you created this branch on top of the one that exports the system version thing.

You may want to start from the main branch instead.

I rebased the branch onto master and pushed the updated PR

@sebjulliand sebjulliand left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

@sebjulliand sebjulliand merged commit eb64755 into codefori:master Jun 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants