Uploaded image for project: 'appNG Manager'
  1. appNG Manager
  2. MGR-108

Repository packages do not show installed version

    Details

    • Type: Bug
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: 1.16.0, 1.17.0, 1.17.1
    • Fix Version/s: 1.18.0

      Description

      The applicationlist in Repositories>Packages does not show the installed version for some applications. This affects the column "installed version" as well as the red/green indicator icon.

      When you select an application, the table for "package version" is correct and indicates the installed version with the green icon correctly.

      This bug only happens, when the number of applications in the repository is larger than the pagesize of the datasource.

      The affected applications can change depending on the pagesize and sort order.

        Attachments

          Activity

          Hide
          c.brueckner Christian Brückner added a comment - - edited

          I have tracked this issue to ManagerService#searchInstallablePackages

          line 395 applicationRepository.search(fp.getPageable()); does not retrieve the correct applications with respect to the pageable (size and sort).

          Iinstalled version are only retrieved for this list (line 401 repository.getInstallablePackages(identifiers))

          But in the end, a different list is handed to the client (line 414 data.setPage(packages, fp.getPageable()))

          Show
          c.brueckner Christian Brückner added a comment - - edited I have tracked this issue to ManagerService#searchInstallablePackages line 395 applicationRepository.search(fp.getPageable()); does not retrieve the correct applications with respect to the pageable (size and sort). Iinstalled version are only retrieved for this list (line 401 repository.getInstallablePackages(identifiers)) But in the end, a different list is handed to the client (line 414 data.setPage(packages, fp.getPageable()) )
          Hide
          mueller.matthias Matthias Müller added a comment - - edited

          Hi Christian,

          thanks for finding this!
          As you correctly analyzed, the solution is, instead of doing applicationRepository.search(fp.getPageable()), just retrieve all applications with applicationRepository.findAll().

          Regards
          Matthias

          Show
          mueller.matthias Matthias Müller added a comment - - edited Hi Christian, thanks for finding this! As you correctly analyzed, the solution is, instead of doing applicationRepository.search(fp.getPageable()) , just retrieve all applications with applicationRepository.findAll() . Regards Matthias

            People

            • Assignee:
              mueller.matthias Matthias Müller
              Reporter:
              c.brueckner Christian Brückner
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: