Replaced repository name as search criterion with repository id

Eddy Young

2015-05-22 4cb1a97509b9b42ca7efb4fa0f727b89b044202d

 src/main/java/com/gitblit/wicket/pages/MyTicketsPage.java

@@ -72,11 +72,10 @@
      final String[] statiiParam = (params == null) ? TicketsUI.openStatii : params.getStringArray(Lucene.status.name());
      final String assignedToParam = (params == null) ? "" : params.getString(Lucene.responsible.name(), null);
      final String milestoneParam = (params == null) ? "" : params.getString(Lucene.milestone.name(), null);
      // todo jeyoung add repository to this filter
      final String queryParam = (params == null || StringUtils.isEmpty(params.getString("q", null))) ? "watchedby:" + username : params.getString("q", null);
      final String searchParam = (params == null) ? "" : params.getString("s", null);
      final String sortBy = (params == null) ? "" : Lucene.fromString(params.getString("sort", Lucene.created.name())).name();
      final String repository = (params == null) ? "" : Lucene.fromString(params.getString("repository", Lucene.repository.name())).name();
      final String repositoryId = (params == null) ? "" : params.getString(Lucene.rid.name(), null);
      final boolean desc = (params == null) ? true : !"asc".equals(params.getString("direction", "desc"));

      // add the user title panel
@@ -94,7 +93,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("bugsQuery", MyTicketsPage.class,
@@ -105,7 +104,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("enhancementsQuery", MyTicketsPage.class,
@@ -116,7 +115,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("tasksQuery", MyTicketsPage.class,
@@ -127,7 +126,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("questionsQuery", MyTicketsPage.class,
@@ -138,7 +137,18 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));
		
      add(new BookmarkablePageLink<Void>("maintenanceQuery", MyTicketsPage.class,
            queryParameters(
                  Lucene.type.matches(TicketModel.Type.Maintenance.name()),
                  milestoneParam,
                  statiiParam,
                  assignedToParam,
                  sortBy,
                  desc,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("resetQuery", MyTicketsPage.class,
@@ -161,7 +171,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      add(new BookmarkablePageLink<Void>("watchedQuery", MyTicketsPage.class,
@@ -172,7 +182,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));
      add(new BookmarkablePageLink<Void>("mentionsQuery", MyTicketsPage.class,
            queryParameters(
@@ -182,7 +192,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));
      add(new BookmarkablePageLink<Void>("responsibleQuery", MyTicketsPage.class,
            queryParameters(
@@ -192,7 +202,7 @@
                  assignedToParam,
                  sortBy,
                  desc,
                        repository,
                        repositoryId,
                  1)));

      // states
@@ -201,9 +211,9 @@
      } else {
         add(new Label("selectedStatii", StringUtils.flattenStrings(Arrays.asList(statiiParam), ",")));
      }
      add(new BookmarkablePageLink<Void>("openTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, TicketsUI.openStatii, assignedToParam, sortBy, desc, repository, 1)));
      add(new BookmarkablePageLink<Void>("closedTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, TicketsUI.closedStatii, assignedToParam, sortBy, desc, repository, 1)));
      add(new BookmarkablePageLink<Void>("allTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, null, assignedToParam, sortBy, desc, repository, 1)));
      add(new BookmarkablePageLink<Void>("openTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, TicketsUI.openStatii, assignedToParam, sortBy, desc, repositoryId, 1)));
      add(new BookmarkablePageLink<Void>("closedTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, TicketsUI.closedStatii, assignedToParam, sortBy, desc, repositoryId, 1)));
      add(new BookmarkablePageLink<Void>("allTickets", MyTicketsPage.class, queryParameters(queryParam, milestoneParam, null, assignedToParam, sortBy, desc, repositoryId, 1)));

      // by status
      List<Status> statii = new ArrayList<Status>(Arrays.asList(Status.values()));
@@ -215,7 +225,7 @@
         @Override
         public void populateItem(final Item<Status> item) {
            final Status status = item.getModelObject();
            PageParameters p = queryParameters(queryParam, milestoneParam, new String [] { status.name().toLowerCase() }, assignedToParam, sortBy, desc, repository, 1);
            PageParameters p = queryParameters(queryParam, milestoneParam, new String [] { status.name().toLowerCase() }, assignedToParam, sortBy, desc, repositoryId, 1);
            String css = TicketsUI.getStatusClass(status);
            item.add(new LinkPanel("statusLink", css, status.toString(), MyTicketsPage.class, p).setRenderBodyOnly(true));
         }
@@ -234,6 +244,10 @@
      sortChoices.add(new TicketSort(getString("gb.sortLeastPatchsetRevisions"), Lucene.patchsets.name(), false));
      sortChoices.add(new TicketSort(getString("gb.sortMostVotes"), Lucene.votes.name(), true));
      sortChoices.add(new TicketSort(getString("gb.sortLeastVotes"), Lucene.votes.name(), false));
      sortChoices.add(new TicketSort(getString("gb.sortHighestPriority"), Lucene.priority.name(), true));
      sortChoices.add(new TicketSort(getString("gb.sortLowestPriority"), Lucene.priority.name(), false));
      sortChoices.add(new TicketSort(getString("gb.sortHighestSeverity"), Lucene.severity.name(), true));
      sortChoices.add(new TicketSort(getString("gb.sortLowestSeverity"), Lucene.severity.name(), false));

      TicketSort currentSort = sortChoices.get(0);
      for (TicketSort ts : sortChoices) {
@@ -251,7 +265,7 @@
         @Override
         public void populateItem(final Item<TicketSort> item) {
            final TicketSort ts = item.getModelObject();
            PageParameters params = queryParameters(queryParam, milestoneParam, statiiParam, assignedToParam, ts.sortBy, ts.desc, repository, 1);
            PageParameters params = queryParameters(queryParam, milestoneParam, statiiParam, assignedToParam, ts.sortBy, ts.desc, repositoryId, 1);
            item.add(new LinkPanel("sortLink", null, ts.name, MyTicketsPage.class, params).setRenderBodyOnly(true));
         }
      };
@@ -259,17 +273,18 @@

        // by repository
      List<RepositoryModel> repositoryChoices = getRepositoryModels();
        RepositoryModel noneChoice = new RepositoryModel();
        noneChoice.name = getString("gb.all");
        repositoryChoices.add(0, noneChoice);
      RepositoryModel currentRepository = repositoryChoices.size() > 0 ? repositoryChoices.get(0) : null;
      for (RepositoryModel r : repositoryChoices) {
         if (r.name.equals(repository)) {
         if (r.getRID().equals(repositoryId)) {
            currentRepository = r;
            break;
         }
      }
      add(new Label("currentRepository", currentRepository == null ? "none" : currentRepository.name));
      add(new Label("currentRepository", currentRepository == null ? "none" : currentRepository.toString()));

        // todo insert a "none" option that will show tickets for all
        // repositories
      ListDataProvider<RepositoryModel> repositoryChoicesDp = new ListDataProvider<RepositoryModel>(repositoryChoices);
      DataView<RepositoryModel> repositoryMenu = new DataView<RepositoryModel>("repository", repositoryChoicesDp) {
         private static final long serialVersionUID = 1L;
@@ -277,9 +292,8 @@
         @Override
         public void populateItem(final Item<RepositoryModel> item) {
            final RepositoryModel r = item.getModelObject();
                // todo add repository parameter
            PageParameters params = queryParameters(queryParam, milestoneParam, statiiParam, assignedToParam, sortBy, desc, r.name, 1);
            item.add(new LinkPanel("repositoryLink", null, r.name, MyTicketsPage.class, params).setRenderBodyOnly(true));
            PageParameters params = queryParameters(queryParam, milestoneParam, statiiParam, assignedToParam, sortBy, desc, r.getRID(), 1);
            item.add(new LinkPanel("repositoryLink", null, r.toString(), MyTicketsPage.class, params).setRenderBodyOnly(true));
         }
      };
      add(repositoryMenu);
@@ -305,7 +319,11 @@
         }
      }

      // todo jeyoung specify repository name, similar to above
        if (noneChoice != currentRepository && !qb.containsField(Lucene.rid.name())) {
            QueryBuilder q1 = new QueryBuilder();
            q1.and(Lucene.rid.matches(repositoryId));
            qb.and(q1.toSubquery().toString());
        }

      final String luceneQuery;
      if (qb.containsField(Lucene.createdby.name())
@@ -335,13 +353,12 @@
      }

      int totalResults = results.size() == 0 ? 0 : results.get(0).totalResults;
      buildPager(queryParam, milestoneParam, statiiParam, assignedToParam, sortBy, desc, repository, page, pageSize, results.size(), totalResults);
      buildPager(queryParam, milestoneParam, statiiParam, assignedToParam, sortBy, desc, repositoryId, page, pageSize, results.size(), totalResults);

      final boolean showSwatch = app().settings().getBoolean(Keys.web.repositoryListSwatches, true);
      add(new TicketListPanel("ticketList", results, showSwatch, true));
   }

    // todo add repository name here
   protected PageParameters queryParameters(
         String query,
         String milestone,
@@ -349,7 +366,7 @@
         String assignedTo,
         String sort,
         boolean descending,
         String repository,
         String repositoryId,
         int page) {

      PageParameters params = WicketUtils.newRepositoryParameter("");
@@ -373,8 +390,8 @@
      if (!descending) {
         params.add("direction", "asc");
      }
      if (!StringUtils.isEmpty(repository)) {
         params.add(Lucene.repository.name(), repository);
      if (!StringUtils.isEmpty(repositoryId)) {
         params.add(Lucene.rid.name(), repositoryId);
      }
      if (page > 1) {
         params.add("pg", "" + page);
@@ -389,7 +406,7 @@
         final String assignedTo,
         final String sort,
         final boolean desc,
         final String repository,
         final String repositoryId,
         final int page,
         int pageSize,
         int count,
@@ -398,8 +415,8 @@
      boolean showNav = total > (2 * pageSize);
      boolean allowPrev = page > 1;
      boolean allowNext = (pageSize * (page - 1) + count) < total;
      add(new BookmarkablePageLink<Void>("prevLink", MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repository, page - 1)).setEnabled(allowPrev).setVisible(showNav));
      add(new BookmarkablePageLink<Void>("nextLink", MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repository, page + 1)).setEnabled(allowNext).setVisible(showNav));
      add(new BookmarkablePageLink<Void>("prevLink", MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repositoryId, page - 1)).setEnabled(allowPrev).setVisible(showNav));
      add(new BookmarkablePageLink<Void>("nextLink", MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repositoryId, page + 1)).setEnabled(allowNext).setVisible(showNav));

      if (total <= pageSize) {
         add(new Label("pageLink").setVisible(false));
@@ -428,7 +445,7 @@
         @Override
         public void populateItem(final Item<Integer> item) {
            final Integer i = item.getModelObject();
            LinkPanel link = new LinkPanel("page", null, "" + i, MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repository, i));
            LinkPanel link = new LinkPanel("page", null, "" + i, MyTicketsPage.class, queryParameters(query, milestone, states, assignedTo, sort, desc, repositoryId, i));
            link.setRenderBodyOnly(true);
            if (i == page) {
               WicketUtils.setCssClass(item, "active");