From acaa2a2765c4864ef001ae1097285c4f5cf2987d Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Mon, 03 Mar 2014 21:37:54 -0500 Subject: [PATCH] Merged #1 "Ticket tracker with patchset contributions" --- src/main/java/com/gitblit/wicket/panels/ReflogPanel.java | 638 +++++++++++++++++++++++++++++---------------------------- 1 files changed, 325 insertions(+), 313 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/panels/ReflogPanel.java b/src/main/java/com/gitblit/wicket/panels/ReflogPanel.java index c1db726..baefc6b 100644 --- a/src/main/java/com/gitblit/wicket/panels/ReflogPanel.java +++ b/src/main/java/com/gitblit/wicket/panels/ReflogPanel.java @@ -1,313 +1,325 @@ -/* - * Copyright 2013 gitblit.com. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package com.gitblit.wicket.panels; - -import java.text.DateFormat; -import java.text.MessageFormat; -import java.text.SimpleDateFormat; -import java.util.ArrayList; -import java.util.Calendar; -import java.util.Date; -import java.util.List; -import java.util.TimeZone; - -import org.apache.wicket.markup.html.basic.Label; -import org.apache.wicket.markup.repeater.Item; -import org.apache.wicket.markup.repeater.data.DataView; -import org.apache.wicket.markup.repeater.data.ListDataProvider; -import org.apache.wicket.model.StringResourceModel; -import org.eclipse.jgit.lib.Repository; -import org.eclipse.jgit.transport.ReceiveCommand.Type; - -import com.gitblit.Constants; -import com.gitblit.Keys; -import com.gitblit.models.RefLogEntry; -import com.gitblit.models.RepositoryCommit; -import com.gitblit.models.RepositoryModel; -import com.gitblit.models.UserModel; -import com.gitblit.utils.RefLogUtils; -import com.gitblit.utils.StringUtils; -import com.gitblit.utils.TimeUtils; -import com.gitblit.wicket.WicketUtils; -import com.gitblit.wicket.pages.CommitPage; -import com.gitblit.wicket.pages.ComparePage; -import com.gitblit.wicket.pages.ReflogPage; -import com.gitblit.wicket.pages.TagPage; -import com.gitblit.wicket.pages.TreePage; -import com.gitblit.wicket.pages.UserPage; - -public class ReflogPanel extends BasePanel { - - private static final long serialVersionUID = 1L; - - private final boolean hasChanges; - - private boolean hasMore; - - public ReflogPanel(String wicketId, final RepositoryModel model, Repository r, int limit, int pageOffset) { - super(wicketId); - boolean pageResults = limit <= 0; - int changesPerPage = app().settings().getInteger(Keys.web.reflogChangesPerPage, 10); - if (changesPerPage <= 1) { - changesPerPage = 10; - } - - List<RefLogEntry> changes; - if (pageResults) { - changes = RefLogUtils.getLogByRef(model.name, r, pageOffset * changesPerPage, changesPerPage); - } else { - changes = RefLogUtils.getLogByRef(model.name, r, limit); - } - - // inaccurate way to determine if there are more commits. - // works unless commits.size() represents the exact end. - hasMore = changes.size() >= changesPerPage; - hasChanges = changes.size() > 0; - - setup(changes); - - // determine to show pager, more, or neither - if (limit <= 0) { - // no display limit - add(new Label("moreChanges").setVisible(false)); - } else { - if (pageResults) { - // paging - add(new Label("moreChanges").setVisible(false)); - } else { - // more - if (changes.size() == limit) { - // show more - add(new LinkPanel("moreChanges", "link", new StringResourceModel("gb.moreChanges", - this, null), ReflogPage.class, - WicketUtils.newRepositoryParameter(model.name))); - } else { - // no more - add(new Label("moreChanges").setVisible(false)); - } - } - } - } - - public ReflogPanel(String wicketId, List<RefLogEntry> changes) { - super(wicketId); - hasChanges = changes.size() > 0; - setup(changes); - add(new Label("moreChanges").setVisible(false)); - } - - protected void setup(List<RefLogEntry> changes) { - - ListDataProvider<RefLogEntry> dp = new ListDataProvider<RefLogEntry>(changes); - DataView<RefLogEntry> changeView = new DataView<RefLogEntry>("change", dp) { - private static final long serialVersionUID = 1L; - - @Override - public void populateItem(final Item<RefLogEntry> changeItem) { - final RefLogEntry change = changeItem.getModelObject(); - - String dateFormat = app().settings().getString(Keys.web.datetimestampLongFormat, "EEEE, MMMM d, yyyy HH:mm Z"); - TimeZone timezone = getTimeZone(); - DateFormat df = new SimpleDateFormat(dateFormat); - df.setTimeZone(timezone); - Calendar cal = Calendar.getInstance(timezone); - - String fullRefName = change.getChangedRefs().get(0); - String shortRefName = fullRefName; - boolean isTag = false; - if (shortRefName.startsWith(Constants.R_HEADS)) { - shortRefName = shortRefName.substring(Constants.R_HEADS.length()); - } else if (shortRefName.startsWith(Constants.R_TAGS)) { - shortRefName = shortRefName.substring(Constants.R_TAGS.length()); - isTag = true; - } - - String fuzzydate; - TimeUtils tu = getTimeUtils(); - Date changeDate = change.date; - if (TimeUtils.isToday(changeDate, timezone)) { - fuzzydate = tu.today(); - } else if (TimeUtils.isYesterday(changeDate, timezone)) { - fuzzydate = tu.yesterday(); - } else { - // calculate a fuzzy time ago date - cal.setTime(changeDate); - cal.set(Calendar.HOUR_OF_DAY, 0); - cal.set(Calendar.MINUTE, 0); - cal.set(Calendar.SECOND, 0); - cal.set(Calendar.MILLISECOND, 0); - Date date = cal.getTime(); - fuzzydate = getTimeUtils().timeAgo(date); - } - changeItem.add(new Label("whenChanged", fuzzydate + ", " + df.format(changeDate))); - - Label changeIcon = new Label("changeIcon"); - if (Type.DELETE.equals(change.getChangeType(fullRefName))) { - WicketUtils.setCssClass(changeIcon, "iconic-trash-stroke"); - } else if (isTag) { - WicketUtils.setCssClass(changeIcon, "iconic-tag"); - } else { - WicketUtils.setCssClass(changeIcon, "iconic-upload"); - } - changeItem.add(changeIcon); - - if (change.user.username.equals(change.user.emailAddress) && change.user.emailAddress.indexOf('@') > -1) { - // username is an email address - 1.2.1 push log bug - changeItem.add(new Label("whoChanged", change.user.getDisplayName())); - } else if (change.user.username.equals(UserModel.ANONYMOUS.username)) { - // anonymous change - changeItem.add(new Label("whoChanged", getString("gb.anonymousUser"))); - } else { - // link to user account page - changeItem.add(new LinkPanel("whoChanged", null, change.user.getDisplayName(), - UserPage.class, WicketUtils.newUsernameParameter(change.user.username))); - } - - boolean isDelete = false; - boolean isRewind = false; - String what; - String by = null; - switch(change.getChangeType(fullRefName)) { - case CREATE: - if (isTag) { - // new tag - what = getString("gb.pushedNewTag"); - } else { - // new branch - what = getString("gb.pushedNewBranch"); - } - break; - case DELETE: - isDelete = true; - if (isTag) { - what = getString("gb.deletedTag"); - } else { - what = getString("gb.deletedBranch"); - } - break; - case UPDATE_NONFASTFORWARD: - isRewind = true; - default: - what = MessageFormat.format(change.getCommitCount() > 1 ? getString("gb.pushedNCommitsTo") : getString("gb.pushedOneCommitTo") , change.getCommitCount()); - - if (change.getAuthorCount() == 1) { - by = MessageFormat.format(getString("gb.byOneAuthor"), change.getAuthorIdent().getName()); - } else { - by = MessageFormat.format(getString("gb.byNAuthors"), change.getAuthorCount()); - } - break; - } - changeItem.add(new Label("whatChanged", what)); - changeItem.add(new Label("byAuthors", by).setVisible(!StringUtils.isEmpty(by))); - - changeItem.add(new Label("refRewind", getString("gb.rewind")).setVisible(isRewind)); - - if (isDelete) { - // can't link to deleted ref - changeItem.add(new Label("refChanged", shortRefName)); - } else if (isTag) { - // link to tag - changeItem.add(new LinkPanel("refChanged", null, shortRefName, - TagPage.class, WicketUtils.newObjectParameter(change.repository, fullRefName))); - } else { - // link to tree - changeItem.add(new LinkPanel("refChanged", null, shortRefName, - TreePage.class, WicketUtils.newObjectParameter(change.repository, fullRefName))); - } - - int maxCommitCount = 5; - List<RepositoryCommit> commits = change.getCommits(); - if (commits.size() > maxCommitCount) { - commits = new ArrayList<RepositoryCommit>(commits.subList(0, maxCommitCount)); - } - - // compare link - String compareLinkText = null; - if ((change.getCommitCount() <= maxCommitCount) && (change.getCommitCount() > 1)) { - compareLinkText = MessageFormat.format(getString("gb.viewComparison"), commits.size()); - } else if (change.getCommitCount() > maxCommitCount) { - int diff = change.getCommitCount() - maxCommitCount; - compareLinkText = MessageFormat.format(diff > 1 ? getString("gb.nMoreCommits") : getString("gb.oneMoreCommit"), diff); - } - if (StringUtils.isEmpty(compareLinkText)) { - changeItem.add(new Label("compareLink").setVisible(false)); - } else { - String endRangeId = change.getNewId(fullRefName); - String startRangeId = change.getOldId(fullRefName); - changeItem.add(new LinkPanel("compareLink", null, compareLinkText, ComparePage.class, WicketUtils.newRangeParameter(change.repository, startRangeId, endRangeId))); - } - - ListDataProvider<RepositoryCommit> cdp = new ListDataProvider<RepositoryCommit>(commits); - DataView<RepositoryCommit> commitsView = new DataView<RepositoryCommit>("commit", cdp) { - private static final long serialVersionUID = 1L; - - @Override - public void populateItem(final Item<RepositoryCommit> commitItem) { - final RepositoryCommit commit = commitItem.getModelObject(); - - // author gravatar - commitItem.add(new GravatarImage("commitAuthor", commit.getAuthorIdent(), null, 16, false)); - - // merge icon - if (commit.getParentCount() > 1) { - commitItem.add(WicketUtils.newImage("commitIcon", "commit_merge_16x16.png")); - } else { - commitItem.add(WicketUtils.newBlankImage("commitIcon")); - } - - // short message - String shortMessage = commit.getShortMessage(); - String trimmedMessage = shortMessage; - if (commit.getRefs() != null && commit.getRefs().size() > 0) { - trimmedMessage = StringUtils.trimString(shortMessage, Constants.LEN_SHORTLOG_REFS); - } else { - trimmedMessage = StringUtils.trimString(shortMessage, Constants.LEN_SHORTLOG); - } - LinkPanel shortlog = new LinkPanel("commitShortMessage", "list", - trimmedMessage, CommitPage.class, WicketUtils.newObjectParameter( - change.repository, commit.getName())); - if (!shortMessage.equals(trimmedMessage)) { - WicketUtils.setHtmlTooltip(shortlog, shortMessage); - } - commitItem.add(shortlog); - - // commit hash link - int hashLen = app().settings().getInteger(Keys.web.shortCommitIdLength, 6); - LinkPanel commitHash = new LinkPanel("hashLink", null, commit.getName().substring(0, hashLen), - CommitPage.class, WicketUtils.newObjectParameter( - change.repository, commit.getName())); - WicketUtils.setCssClass(commitHash, "shortsha1"); - WicketUtils.setHtmlTooltip(commitHash, commit.getName()); - commitItem.add(commitHash); - } - }; - - changeItem.add(commitsView); - } - }; - - add(changeView); - } - - public boolean hasMore() { - return hasMore; - } - - public boolean hideIfEmpty() { - setVisible(hasChanges); - return hasChanges; - } -} +/* + * Copyright 2013 gitblit.com. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +package com.gitblit.wicket.panels; + +import java.text.DateFormat; +import java.text.MessageFormat; +import java.text.SimpleDateFormat; +import java.util.ArrayList; +import java.util.Calendar; +import java.util.Date; +import java.util.List; +import java.util.TimeZone; + +import org.apache.wicket.markup.html.basic.Label; +import org.apache.wicket.markup.repeater.Item; +import org.apache.wicket.markup.repeater.data.DataView; +import org.apache.wicket.markup.repeater.data.ListDataProvider; +import org.apache.wicket.model.StringResourceModel; +import org.eclipse.jgit.lib.Repository; +import org.eclipse.jgit.transport.ReceiveCommand.Type; + +import com.gitblit.Constants; +import com.gitblit.Keys; +import com.gitblit.models.RefLogEntry; +import com.gitblit.models.RepositoryCommit; +import com.gitblit.models.RepositoryModel; +import com.gitblit.models.UserModel; +import com.gitblit.utils.RefLogUtils; +import com.gitblit.utils.StringUtils; +import com.gitblit.utils.TimeUtils; +import com.gitblit.wicket.WicketUtils; +import com.gitblit.wicket.pages.CommitPage; +import com.gitblit.wicket.pages.ComparePage; +import com.gitblit.wicket.pages.ReflogPage; +import com.gitblit.wicket.pages.TagPage; +import com.gitblit.wicket.pages.TicketsPage; +import com.gitblit.wicket.pages.TreePage; +import com.gitblit.wicket.pages.UserPage; + +public class ReflogPanel extends BasePanel { + + private static final long serialVersionUID = 1L; + + private final boolean hasChanges; + + private boolean hasMore; + + public ReflogPanel(String wicketId, final RepositoryModel model, Repository r, int limit, int pageOffset) { + super(wicketId); + boolean pageResults = limit <= 0; + int changesPerPage = app().settings().getInteger(Keys.web.reflogChangesPerPage, 10); + if (changesPerPage <= 1) { + changesPerPage = 10; + } + + List<RefLogEntry> changes; + if (pageResults) { + changes = RefLogUtils.getLogByRef(model.name, r, pageOffset * changesPerPage, changesPerPage); + } else { + changes = RefLogUtils.getLogByRef(model.name, r, limit); + } + + // inaccurate way to determine if there are more commits. + // works unless commits.size() represents the exact end. + hasMore = changes.size() >= changesPerPage; + hasChanges = changes.size() > 0; + + setup(changes); + + // determine to show pager, more, or neither + if (limit <= 0) { + // no display limit + add(new Label("moreChanges").setVisible(false)); + } else { + if (pageResults) { + // paging + add(new Label("moreChanges").setVisible(false)); + } else { + // more + if (changes.size() == limit) { + // show more + add(new LinkPanel("moreChanges", "link", new StringResourceModel("gb.moreChanges", + this, null), ReflogPage.class, + WicketUtils.newRepositoryParameter(model.name))); + } else { + // no more + add(new Label("moreChanges").setVisible(false)); + } + } + } + } + + public ReflogPanel(String wicketId, List<RefLogEntry> changes) { + super(wicketId); + hasChanges = changes.size() > 0; + setup(changes); + add(new Label("moreChanges").setVisible(false)); + } + + protected void setup(List<RefLogEntry> changes) { + + ListDataProvider<RefLogEntry> dp = new ListDataProvider<RefLogEntry>(changes); + DataView<RefLogEntry> changeView = new DataView<RefLogEntry>("change", dp) { + private static final long serialVersionUID = 1L; + + @Override + public void populateItem(final Item<RefLogEntry> changeItem) { + final RefLogEntry change = changeItem.getModelObject(); + + String dateFormat = app().settings().getString(Keys.web.datetimestampLongFormat, "EEEE, MMMM d, yyyy HH:mm Z"); + TimeZone timezone = getTimeZone(); + DateFormat df = new SimpleDateFormat(dateFormat); + df.setTimeZone(timezone); + Calendar cal = Calendar.getInstance(timezone); + + String fullRefName = change.getChangedRefs().get(0); + String shortRefName = fullRefName; + String ticketId = null; + boolean isTag = false; + boolean isTicket = false; + if (shortRefName.startsWith(Constants.R_TICKET)) { + ticketId = fullRefName.substring(Constants.R_TICKET.length()); + shortRefName = MessageFormat.format(getString("gb.ticketN"), ticketId); + isTicket = true; + } else if (shortRefName.startsWith(Constants.R_HEADS)) { + shortRefName = shortRefName.substring(Constants.R_HEADS.length()); + } else if (shortRefName.startsWith(Constants.R_TAGS)) { + shortRefName = shortRefName.substring(Constants.R_TAGS.length()); + isTag = true; + } + + String fuzzydate; + TimeUtils tu = getTimeUtils(); + Date changeDate = change.date; + if (TimeUtils.isToday(changeDate, timezone)) { + fuzzydate = tu.today(); + } else if (TimeUtils.isYesterday(changeDate, timezone)) { + fuzzydate = tu.yesterday(); + } else { + // calculate a fuzzy time ago date + cal.setTime(changeDate); + cal.set(Calendar.HOUR_OF_DAY, 0); + cal.set(Calendar.MINUTE, 0); + cal.set(Calendar.SECOND, 0); + cal.set(Calendar.MILLISECOND, 0); + Date date = cal.getTime(); + fuzzydate = getTimeUtils().timeAgo(date); + } + changeItem.add(new Label("whenChanged", fuzzydate + ", " + df.format(changeDate))); + + Label changeIcon = new Label("changeIcon"); + if (Type.DELETE.equals(change.getChangeType(fullRefName))) { + WicketUtils.setCssClass(changeIcon, "iconic-trash-stroke"); + } else if (isTag) { + WicketUtils.setCssClass(changeIcon, "iconic-tag"); + } else if (isTicket) { + WicketUtils.setCssClass(changeIcon, "fa fa-ticket"); + } else { + WicketUtils.setCssClass(changeIcon, "iconic-upload"); + } + changeItem.add(changeIcon); + + if (change.user.username.equals(change.user.emailAddress) && change.user.emailAddress.indexOf('@') > -1) { + // username is an email address - 1.2.1 push log bug + changeItem.add(new Label("whoChanged", change.user.getDisplayName())); + } else if (change.user.username.equals(UserModel.ANONYMOUS.username)) { + // anonymous change + changeItem.add(new Label("whoChanged", getString("gb.anonymousUser"))); + } else { + // link to user account page + changeItem.add(new LinkPanel("whoChanged", null, change.user.getDisplayName(), + UserPage.class, WicketUtils.newUsernameParameter(change.user.username))); + } + + boolean isDelete = false; + boolean isRewind = false; + String what; + String by = null; + switch(change.getChangeType(fullRefName)) { + case CREATE: + if (isTag) { + // new tag + what = getString("gb.pushedNewTag"); + } else { + // new branch + what = getString("gb.pushedNewBranch"); + } + break; + case DELETE: + isDelete = true; + if (isTag) { + what = getString("gb.deletedTag"); + } else { + what = getString("gb.deletedBranch"); + } + break; + case UPDATE_NONFASTFORWARD: + isRewind = true; + default: + what = MessageFormat.format(change.getCommitCount() > 1 ? getString("gb.pushedNCommitsTo") : getString("gb.pushedOneCommitTo"), change.getCommitCount()); + + if (change.getAuthorCount() == 1) { + by = MessageFormat.format(getString("gb.byOneAuthor"), change.getAuthorIdent().getName()); + } else { + by = MessageFormat.format(getString("gb.byNAuthors"), change.getAuthorCount()); + } + break; + } + changeItem.add(new Label("whatChanged", what)); + changeItem.add(new Label("byAuthors", by).setVisible(!StringUtils.isEmpty(by))); + changeItem.add(new Label("refRewind", getString("gb.rewind")).setVisible(isRewind)); + + if (isDelete) { + // can't link to deleted ref + changeItem.add(new Label("refChanged", shortRefName)); + } else if (isTag) { + // link to tag + changeItem.add(new LinkPanel("refChanged", null, shortRefName, + TagPage.class, WicketUtils.newObjectParameter(change.repository, fullRefName))); + } else if (isTicket) { + // link to ticket + changeItem.add(new LinkPanel("refChanged", null, shortRefName, + TicketsPage.class, WicketUtils.newObjectParameter(change.repository, ticketId))); + } else { + // link to tree + changeItem.add(new LinkPanel("refChanged", null, shortRefName, + TreePage.class, WicketUtils.newObjectParameter(change.repository, fullRefName))); + } + + int maxCommitCount = 5; + List<RepositoryCommit> commits = change.getCommits(); + if (commits.size() > maxCommitCount) { + commits = new ArrayList<RepositoryCommit>(commits.subList(0, maxCommitCount)); + } + + // compare link + String compareLinkText = null; + if ((change.getCommitCount() <= maxCommitCount) && (change.getCommitCount() > 1)) { + compareLinkText = MessageFormat.format(getString("gb.viewComparison"), commits.size()); + } else if (change.getCommitCount() > maxCommitCount) { + int diff = change.getCommitCount() - maxCommitCount; + compareLinkText = MessageFormat.format(diff > 1 ? getString("gb.nMoreCommits") : getString("gb.oneMoreCommit"), diff); + } + if (StringUtils.isEmpty(compareLinkText)) { + changeItem.add(new Label("compareLink").setVisible(false)); + } else { + String endRangeId = change.getNewId(fullRefName); + String startRangeId = change.getOldId(fullRefName); + changeItem.add(new LinkPanel("compareLink", null, compareLinkText, ComparePage.class, WicketUtils.newRangeParameter(change.repository, startRangeId, endRangeId))); + } + + ListDataProvider<RepositoryCommit> cdp = new ListDataProvider<RepositoryCommit>(commits); + DataView<RepositoryCommit> commitsView = new DataView<RepositoryCommit>("commit", cdp) { + private static final long serialVersionUID = 1L; + + @Override + public void populateItem(final Item<RepositoryCommit> commitItem) { + final RepositoryCommit commit = commitItem.getModelObject(); + + // author gravatar + commitItem.add(new GravatarImage("commitAuthor", commit.getAuthorIdent(), null, 16, false)); + + // merge icon + if (commit.getParentCount() > 1) { + commitItem.add(WicketUtils.newImage("commitIcon", "commit_merge_16x16.png")); + } else { + commitItem.add(WicketUtils.newBlankImage("commitIcon")); + } + + // short message + String shortMessage = commit.getShortMessage(); + String trimmedMessage = shortMessage; + if (commit.getRefs() != null && commit.getRefs().size() > 0) { + trimmedMessage = StringUtils.trimString(shortMessage, Constants.LEN_SHORTLOG_REFS); + } else { + trimmedMessage = StringUtils.trimString(shortMessage, Constants.LEN_SHORTLOG); + } + LinkPanel shortlog = new LinkPanel("commitShortMessage", "list", + trimmedMessage, CommitPage.class, WicketUtils.newObjectParameter( + change.repository, commit.getName())); + if (!shortMessage.equals(trimmedMessage)) { + WicketUtils.setHtmlTooltip(shortlog, shortMessage); + } + commitItem.add(shortlog); + + // commit hash link + int hashLen = app().settings().getInteger(Keys.web.shortCommitIdLength, 6); + LinkPanel commitHash = new LinkPanel("hashLink", null, commit.getName().substring(0, hashLen), + CommitPage.class, WicketUtils.newObjectParameter( + change.repository, commit.getName())); + WicketUtils.setCssClass(commitHash, "shortsha1"); + WicketUtils.setHtmlTooltip(commitHash, commit.getName()); + commitItem.add(commitHash); + } + }; + + changeItem.add(commitsView); + } + }; + + add(changeView); + } + + public boolean hasMore() { + return hasMore; + } + + public boolean hideIfEmpty() { + setVisible(hasChanges); + return hasChanges; + } +} -- Gitblit v1.9.1