Skip to content

Commit 8332a5f

Browse files
committed
[INTERNAL] #399 Adds default methods to ActivityReceiver
Making AbstractActivityReceiver obsolete.
1 parent 9d4eae0 commit 8332a5f

File tree

7 files changed

+61
-141
lines changed

7 files changed

+61
-141
lines changed

core/src/saros/activities/AbstractActivityReceiver.java

Lines changed: 0 additions & 112 deletions
This file was deleted.

core/src/saros/activities/IActivityReceiver.java

Lines changed: 54 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -19,39 +19,75 @@ public interface IActivityReceiver {
1919
*/
2020

2121
/** */
22-
void receive(ChangeColorActivity changeColorActivity);
22+
default void receive(ChangeColorActivity changeColorActivity) {
23+
/*NOP*/
24+
}
2325

24-
void receive(ChecksumActivity checksumActivity);
26+
default void receive(ChecksumActivity checksumActivity) {
27+
/*NOP*/
28+
}
2529

26-
void receive(ChecksumErrorActivity checksumErrorActivity);
30+
default void receive(ChecksumErrorActivity checksumErrorActivity) {
31+
/*NOP*/
32+
}
2733

28-
void receive(EditorActivity editorActivity);
34+
default void receive(EditorActivity editorActivity) {
35+
/*NOP*/
36+
}
2937

30-
void receive(FileActivity fileActivity);
38+
default void receive(FileActivity fileActivity) {
39+
/*NOP*/
40+
}
3141

32-
void receive(FolderCreatedActivity folderCreatedActivity);
42+
default void receive(FolderCreatedActivity folderCreatedActivity) {
43+
/*NOP*/
44+
}
3345

34-
void receive(FolderDeletedActivity folderDeletedActivity);
46+
default void receive(FolderDeletedActivity folderDeletedActivity) {
47+
/*NOP*/
48+
}
3549

36-
void receive(FolderMovedActivity folderMovedActivity);
50+
default void receive(FolderMovedActivity folderMovedActivity) {
51+
/*NOP*/
52+
}
3753

38-
void receive(JupiterActivity jupiterActivity);
54+
default void receive(JupiterActivity jupiterActivity) {
55+
/*NOP*/
56+
}
3957

40-
void receive(NOPActivity nopActivity);
58+
default void receive(NOPActivity nopActivity) {
59+
/*NOP*/
60+
}
4161

42-
void receive(PermissionActivity permissionActivity);
62+
default void receive(PermissionActivity permissionActivity) {
63+
/*NOP*/
64+
}
4365

44-
void receive(ProgressActivity progressActivity);
66+
default void receive(ProgressActivity progressActivity) {
67+
/*NOP*/
68+
}
4569

46-
void receive(StartFollowingActivity startFollowingActivity);
70+
default void receive(StartFollowingActivity startFollowingActivity) {
71+
/*NOP*/
72+
}
4773

48-
void receive(StopActivity stopActivity);
74+
default void receive(StopActivity stopActivity) {
75+
/*NOP*/
76+
}
4977

50-
void receive(StopFollowingActivity stopFollowingActivity);
78+
default void receive(StopFollowingActivity stopFollowingActivity) {
79+
/*NOP*/
80+
}
5181

52-
void receive(TextEditActivity textEditActivity);
82+
default void receive(TextEditActivity textEditActivity) {
83+
/*NOP*/
84+
}
5385

54-
void receive(TextSelectionActivity textSelectionActivity);
86+
default void receive(TextSelectionActivity textSelectionActivity) {
87+
/*NOP*/
88+
}
5589

56-
void receive(ViewportActivity viewportActivity);
90+
default void receive(ViewportActivity viewportActivity) {
91+
/*NOP*/
92+
}
5793
}

core/src/saros/communication/extensions/ActivitiesExtension.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ public int getSequenceNumber() {
103103
}
104104

105105
/**
106-
* @JTourBusStop 5, Creating a new Activity type, XStream registration:
106+
* @JTourBusStop 4, Creating a new Activity type, XStream registration:
107107
*
108108
* <p>We use the XStream library to convert handy Java objects to easy-to-send XML string and vice
109109
* versa. To beautify this XML output, we make use of annotations (such as XStreamAlias or

core/src/saros/concurrent/management/ConcurrentDocumentClient.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import java.util.ArrayList;
44
import java.util.List;
55
import org.apache.log4j.Logger;
6-
import saros.activities.AbstractActivityReceiver;
76
import saros.activities.ChecksumActivity;
87
import saros.activities.FileActivity;
98
import saros.activities.IActivity;
@@ -126,7 +125,7 @@ private IActivity receiveChecksum(ChecksumActivity activity) {
126125

127126
/** Used to remove JupiterClientDocuments for deleted files */
128127
private final IActivityReceiver clientReceiver =
129-
new AbstractActivityReceiver() {
128+
new IActivityReceiver() {
130129
@Override
131130
public void receive(FileActivity fileActivity) {
132131
if (fileActivity.getType() == FileActivity.Type.REMOVED) {

core/src/saros/concurrent/management/ConcurrentDocumentServer.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import java.util.Map;
66
import java.util.Map.Entry;
77
import org.apache.log4j.Logger;
8-
import saros.activities.AbstractActivityReceiver;
98
import saros.activities.ChecksumActivity;
109
import saros.activities.FileActivity;
1110
import saros.activities.IActivity;
@@ -77,7 +76,7 @@ public void checkFileDeleted(final IActivity activity) {
7776
}
7877

7978
private final IActivityReceiver hostReceiver =
80-
new AbstractActivityReceiver() {
79+
new IActivityReceiver() {
8180
@Override
8281
public void receive(final FileActivity activity) {
8382
if (activity.getType() == FileActivity.Type.REMOVED) {

core/src/saros/session/AbstractActivityConsumer.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package saros.session;
22

3-
import saros.activities.AbstractActivityReceiver;
43
import saros.activities.IActivity;
54
import saros.activities.IActivityReceiver;
65
import saros.activities.TextSelectionActivity;
@@ -54,8 +53,7 @@
5453
* };
5554
* </pre>
5655
*/
57-
public abstract class AbstractActivityConsumer extends AbstractActivityReceiver
58-
implements IActivityConsumer {
56+
public abstract class AbstractActivityConsumer implements IActivityReceiver, IActivityConsumer {
5957

6058
/**
6159
* @JTourBusStop 11, Activity sending, The second dispatch:

core/src/saros/session/internal/ChangeColorManager.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public class ChangeColorManager extends AbstractActivityProducer implements Star
5252
private final Map<Integer, Integer> usedColorIDs = new HashMap<Integer, Integer>();
5353

5454
/**
55-
* @JTourBusStop 7, Creating a new Activity type, Waiting for incoming activities:
55+
* @JTourBusStop 6, Creating a new Activity type, Waiting for incoming activities:
5656
*
5757
* <p>All you have to do on the receiver's side, is to create a new IActivityReceiver (or amend an
5858
* existing one), provide it with an receive() method of your newly created flavor, and react on
@@ -144,7 +144,7 @@ public synchronized void start() {
144144
}
145145
}
146146
/**
147-
* @JTourBusStop 8, Creating a new Activity type, Arming your consumer:
147+
* @JTourBusStop 7, Creating a new Activity type, Arming your consumer:
148148
*
149149
* <p>To ensure your newly created consumer actually receives incoming activities, you need to
150150
* register it on the session. That's it :)
@@ -179,7 +179,7 @@ public synchronized Set<Integer> getUsedColorIDs() {
179179
public void changeColorID(int colorID) {
180180

181181
/**
182-
* @JTourBusStop 6, Creating a new Activity type, Create activity instances of your new type:
182+
* @JTourBusStop 5, Creating a new Activity type, Create activity instances of your new type:
183183
*
184184
* <p>Now you are prepared to make use of your new activity type: Find a place in the business
185185
* logic where to react on the events you want to send as an Activity to the other session

0 commit comments

Comments
 (0)