fixes issues that are assigned no version and no component are listed under all projects

Sat, 31 Oct 2020 09:50:15 +0100

author
Mike Becker <universe@uap-core.de>
date
Sat, 31 Oct 2020 09:50:15 +0100
changeset 154
3d10f2a390a1
parent 153
e914fbf4decc
child 155
3ace475224f7

fixes issues that are assigned no version and no component are listed under all projects

src/main/java/de/uapcore/lightpit/dao/postgres/PGIssueDao.java file | annotate | diff | comparison | revisions
     1.1 --- a/src/main/java/de/uapcore/lightpit/dao/postgres/PGIssueDao.java	Sat Oct 31 09:27:36 2020 +0100
     1.2 +++ b/src/main/java/de/uapcore/lightpit/dao/postgres/PGIssueDao.java	Sat Oct 31 09:50:15 2020 +0100
     1.3 @@ -67,7 +67,8 @@
     1.4                          "select issueid, versionid from lpit_issue_resolved_version) "+
     1.5                          query +
     1.6                          "left join issue_version using (issueid) "+
     1.7 -                        "where coalesce(versionid,-1) = ? and coalesce(component, -1) = coalesce(?, component, -1)"
     1.8 +                        "where i.project = ? "+
     1.9 +                        "and coalesce(versionid,-1) = ? and coalesce(component, -1) = coalesce(?, component, -1)"
    1.10          );
    1.11  
    1.12          find = connection.prepareStatement(query + "where issueid = ? ");
    1.13 @@ -220,15 +221,17 @@
    1.14  
    1.15      @Override
    1.16      public List<Issue> list(Project project, Component component, Version version) throws SQLException {
    1.17 -        listForVersion.setInt(1, Optional.ofNullable(version).map(Version::getId).orElse(-1));
    1.18 -        listForVersion.setInt(2, Optional.ofNullable(component).map(Component::getId).orElse(-1));
    1.19 +        listForVersion.setInt(1, project.getId());
    1.20 +        listForVersion.setInt(2, Optional.ofNullable(version).map(Version::getId).orElse(-1));
    1.21 +        listForVersion.setInt(3, Optional.ofNullable(component).map(Component::getId).orElse(-1));
    1.22          return executeQuery(listForVersion);
    1.23      }
    1.24  
    1.25      @Override
    1.26      public List<Issue> list(Project project, Version version) throws SQLException {
    1.27 -        listForVersion.setInt(1, Optional.ofNullable(version).map(Version::getId).orElse(-1));
    1.28 -        listForVersion.setNull(2, Types.INTEGER);
    1.29 +        listForVersion.setInt(1, project.getId());
    1.30 +        listForVersion.setInt(2, Optional.ofNullable(version).map(Version::getId).orElse(-1));
    1.31 +        listForVersion.setNull(3, Types.INTEGER);
    1.32          return executeQuery(listForVersion);
    1.33      }
    1.34  

mercurial