githubFork/gitblit.git - Solinfo Gitblit

			@@ -30,7 +30,6 @@
			import java.util.regex.Matcher;
			import java.util.regex.Pattern;

			import org.eclipse.jgit.lib.AnyObjectId;
			import org.eclipse.jgit.lib.BatchRefUpdate;
			import org.eclipse.jgit.lib.NullProgressMonitor;
			import org.eclipse.jgit.lib.ObjectId;
			@@ -60,6 +59,8 @@
			import com.gitblit.models.TicketModel.Patchset;
			import com.gitblit.models.TicketModel.PatchsetType;
			import com.gitblit.models.TicketModel.Status;
			import com.gitblit.models.TicketModel.TicketAction;
			import com.gitblit.models.TicketModel.TicketLink;
			import com.gitblit.models.UserModel;
			import com.gitblit.tickets.BranchTicketService;
			import com.gitblit.tickets.ITicketService;
			@@ -485,9 +486,27 @@
			switch (cmd.getType()) {
			case CREATE:
			case UPDATE:
			if (cmd.getRefName().startsWith(Constants.R_HEADS)) {
			Collection<TicketModel> tickets = processReferencedTickets(cmd);
			ticketsProcessed += tickets.size();
			for (TicketModel ticket : tickets) {
			ticketNotifier.queueMailing(ticket);
			}
			}
			break;

			case UPDATE_NONFASTFORWARD:
			if (cmd.getRefName().startsWith(Constants.R_HEADS)) {
			Collection<TicketModel> tickets = processMergedTickets(cmd);
			String base = JGitUtils.getMergeBase(getRepository(), cmd.getOldId(), cmd.getNewId());
			List<TicketLink> deletedRefs = JGitUtils.identifyTicketsBetweenCommits(getRepository(), settings, base, cmd.getOldId().name());
			for (TicketLink link : deletedRefs) {
			link.isDelete = true;
			}
			Change deletion = new Change(user.username);
			deletion.pendingLinks = deletedRefs;
			ticketService.updateTicket(repository, 0, deletion);

			Collection<TicketModel> tickets = processReferencedTickets(cmd);
			ticketsProcessed += tickets.size();
			for (TicketModel ticket : tickets) {
			ticketNotifier.queueMailing(ticket);
			@@ -606,12 +625,14 @@
			}

			// check to see if this commit is already linked to a ticket
			long id = identifyTicket(tipCommit, false);
			if (id > 0) {
			sendError("{0} has already been pushed to ticket {1,number,0}.", shortTipId, id);
			if (ticket != null &&
			JGitUtils.getTicketNumberFromCommitBranch(getRepository(), tipCommit) == ticket.number) {
			sendError("{0} has already been pushed to ticket {1,number,0}.", shortTipId, ticket.number);
			sendRejection(cmd, "everything up-to-date");
			return null;
			}

			List<TicketLink> ticketLinks = JGitUtils.identifyTicketsFromCommitMessage(getRepository(), settings, tipCommit);

			PatchsetCommand psCmd;
			if (ticket == null) {
			@@ -802,6 +823,10 @@
			}
			break;
			}

			Change change = psCmd.getChange();
			change.pendingLinks = ticketLinks;

			return psCmd;
			}

			@@ -890,11 +915,11 @@

			/**
			* Automatically closes open tickets that have been merged to their integration
			* branch by a client.
			* branch by a client and adds references to tickets if made in the commit message.
			*
			* @param cmd
			*/
			private Collection<TicketModel> processMergedTickets(ReceiveCommand cmd) {
			private Collection<TicketModel> processReferencedTickets(ReceiveCommand cmd) {
			Map<Long, TicketModel> mergedTickets = new LinkedHashMap<Long, TicketModel>();
			final RevWalk rw = getRevWalk();
			try {
			@@ -907,12 +932,18 @@
			RevCommit c;
			while ((c = rw.next()) != null) {
			rw.parseBody(c);
			long ticketNumber = identifyTicket(c, true);
			if (ticketNumber == 0L \|\| mergedTickets.containsKey(ticketNumber)) {
			List<TicketLink> ticketLinks = JGitUtils.identifyTicketsFromCommitMessage(getRepository(), settings, c);
			if (ticketLinks == null) {
			continue;
			}

			TicketModel ticket = ticketService.getTicket(repository, ticketNumber);
			for (TicketLink link : ticketLinks) {

			if (mergedTickets.containsKey(link.targetTicketId)) {
			continue;
			}

			TicketModel ticket = ticketService.getTicket(repository, link.targetTicketId);
			if (ticket == null) {
			continue;
			}
			@@ -925,6 +956,17 @@
			integrationBranch = Constants.R_HEADS + ticket.mergeTo;
			}

			Change change;
			Patchset patchset = null;
			String mergeSha = c.getName();
			String mergeTo = Repository.shortenRefName(cmd.getRefName());

			if (link.action == TicketAction.Commit) {
			//A commit can reference a ticket in any branch even if the ticket is closed.
			//This allows developers to identify and communicate related issues
			change = new Change(user.username);
			change.referenceCommit(mergeSha);
			} else {
			// ticket must be open and, if specified, the ref must match the integration branch
			if (ticket.isClosed() \|\| (integrationBranch != null && !integrationBranch.equals(cmd.getRefName()))) {
			continue;
			@@ -942,13 +984,8 @@
			}
			}

			String mergeSha = c.getName();
			String mergeTo = Repository.shortenRefName(cmd.getRefName());
			Change change;
			Patchset patchset;
			if (knownPatchset) {
			// identify merged patchset by the patchset tip
			patchset = null;
			for (Patchset ps : ticket.getPatchsets()) {
			if (ps.tip.equals(mergeSha)) {
			patchset = ps;
			@@ -990,22 +1027,56 @@
			// unassigned tickets are assigned to the closer
			change.setField(Field.responsible, user.username);
			}
			}

			ticket = ticketService.updateTicket(repository, ticket.number, change);

			if (ticket != null) {
			sendInfo("");
			sendHeader("#{0,number,0}: {1}", ticket.number, StringUtils.trimString(ticket.title, Constants.LEN_SHORTLOG));

			switch (link.action) {
			case Commit: {
			sendInfo("referenced by push of {0} to {1}", c.getName(), mergeTo);
			}
			break;

			case Close: {
			sendInfo("closed by push of {0} to {1}", patchset, mergeTo);
			mergedTickets.put(ticket.number, ticket);
			}
			break;

			default: {

			}
			}

			sendInfo(ticketService.getTicketUrl(ticket));
			sendInfo("");
			mergedTickets.put(ticket.number, ticket);

			} else {
			String shortid = mergeSha.substring(0, settings.getInteger(Keys.web.shortCommitIdLength, 6));
			sendError("FAILED to close ticket {0,number,0} by push of {1}", ticketNumber, shortid);

			switch (link.action) {
			case Commit: {
			sendError("FAILED to reference ticket {0,number,0} by push of {1}", link.targetTicketId, shortid);
			}
			break;
			case Close: {
			sendError("FAILED to close ticket {0,number,0} by push of {1}", link.targetTicketId, shortid);
			} break;

			default: {

			}
			}
			}
			}
			}

			} catch (IOException e) {
			LOGGER.error("Can't scan for changes to close", e);
			LOGGER.error("Can't scan for changes to reference or close", e);
			} finally {
			rw.reset();
			}
			@@ -1013,75 +1084,9 @@
			return mergedTickets.values();
			}

			/**
			* Try to identify a ticket id from the commit.
			*
			* @param commit
			* @param parseMessage
			* @return a ticket id or 0
			*/
			private long identifyTicket(RevCommit commit, boolean parseMessage) {
			// try lookup by change ref
			Map<AnyObjectId, Set<Ref>> map = getRepository().getAllRefsByPeeledObjectId();
			Set<Ref> refs = map.get(commit.getId());
			if (!ArrayUtils.isEmpty(refs)) {
			for (Ref ref : refs) {
			long number = PatchsetCommand.getTicketNumber(ref.getName());
			if (number > 0) {
			return number;
			}
			}
			}

			if (parseMessage) {
			// parse commit message looking for fixes/closes #n
			String dx = "(?:fixes\|closes)[\\s-]+#?(\\d+)";
			String x = settings.getString(Keys.tickets.closeOnPushCommitMessageRegex, dx);
			if (StringUtils.isEmpty(x)) {
			x = dx;
			}
			try {
			Pattern p = Pattern.compile(x, Pattern.CASE_INSENSITIVE);
			Matcher m = p.matcher(commit.getFullMessage());
			while (m.find()) {
			String val = m.group(1);
			return Long.parseLong(val);
			}
			} catch (Exception e) {
			LOGGER.error(String.format("Failed to parse \"%s\" in commit %s", x, commit.getName()), e);
			}
			}
			return 0L;
			}

			private int countCommits(String baseId, String tipId) {
			int count = 0;
			RevWalk walk = getRevWalk();
			walk.reset();
			walk.sort(RevSort.TOPO);
			walk.sort(RevSort.REVERSE, true);
			try {
			RevCommit tip = walk.parseCommit(getRepository().resolve(tipId));
			RevCommit base = walk.parseCommit(getRepository().resolve(baseId));
			walk.markStart(tip);
			walk.markUninteresting(base);
			for (;;) {
			RevCommit c = walk.next();
			if (c == null) {
			break;
			}
			count++;
			}
			} catch (IOException e) {
			// Should never happen, the core receive process would have
			// identified the missing object earlier before we got control.
			LOGGER.error("failed to get commit count", e);
			return 0;
			} finally {
			walk.close();
			}
			return count;
			}


			/**
			* Creates a new patchset with metadata.
			@@ -1091,7 +1096,7 @@
			* @param tip
			*/
			private Patchset newPatchset(TicketModel ticket, String mergeBase, String tip) {
			int totalCommits = countCommits(mergeBase, tip);
			int totalCommits = JGitUtils.countCommits(getRepository(), getRevWalk(), mergeBase, tip);

			Patchset newPatchset = new Patchset();
			newPatchset.tip = tip;