Nemcio/SCM-Manager
1
0
Fork 0
mirror of https://github.com/scm-manager/scm-manager.git synced 2025-11-02 03:25:56 +01:00
Code Issues Packages Projects Releases Activity

handle response of HookChangesetProvider should be synchronized

Browse Source
This commit is contained in:
Sebastian Sdorra
2013-07-27 15:47:26 +02:00
parent a5bc2d2087
commit 07871b9cdb
3 changed files with 25 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/spi/GitHookChangesetProvider.java
View File

@@ -75,7 +75,8 @@ public class GitHookChangesetProvider implements HookChangesetProvider
* @return * @return
*/ */
@Override @Override
public HookChangesetResponse handleRequest(HookChangesetRequest request) public synchronized HookChangesetResponse handleRequest(
HookChangesetRequest request)
{ {
if (response == null) if (response == null)
{ {

2
scm-plugins/scm-hg-plugin/src/main/java/sonia/scm/repository/spi/HgHookChangesetProvider.java
View File

@@ -95,7 +95,7 @@ public class HgHookChangesetProvider implements HookChangesetProvider
* @return * @return
*/ */
@Override @Override
public HookChangesetResponse handleRequest(HookChangesetRequest request) public synchronized HookChangesetResponse handleRequest(HookChangesetRequest request)
{ {
if (response == null) if (response == null)
{ {

33
scm-plugins/scm-svn-plugin/src/main/java/sonia/scm/repository/spi/AbstractSvnHookChangesetProvider.java
View File

@@ -77,20 +77,31 @@ public abstract class AbstractSvnHookChangesetProvider
* @return * @return
*/ */
@Override @Override
public HookChangesetResponse handleRequest(HookChangesetRequest request) public synchronized HookChangesetResponse handleRequest(
HookChangesetRequest request)
{ {
Changeset c = fetchChangeset(); if (response == null)
Iterable<Changeset> iterable; {
Changeset c = fetchChangeset();
Iterable<Changeset> iterable;
if (c == null) if (c == null)
{ {
iterable = Collections.EMPTY_SET; iterable = Collections.EMPTY_SET;
} }
else else
{ {
iterable = ImmutableSet.of(c); iterable = ImmutableSet.of(c);
}
response = new HookChangesetResponse(iterable);
} }
return new HookChangesetResponse(iterable); return response;
} }
//~--- fields ---------------------------------------------------------------
/** Field description */
private HookChangesetResponse response;
} }