Skip to content

Commit 22d6af4

Browse files
committed
Rename AspectChangeContext to AspectChangeManager
1 parent 4f680da commit 22d6af4

File tree

7 files changed

+149
-160
lines changed

7 files changed

+149
-160
lines changed
Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
import java.util.ArrayDeque;
1717
import java.util.Deque;
1818
import java.util.HashMap;
19-
import java.util.List;
2019
import java.util.Map;
2120
import java.util.Optional;
2221
import java.util.stream.Stream;
@@ -30,21 +29,21 @@
3029
import org.slf4j.Logger;
3130
import org.slf4j.LoggerFactory;
3231

33-
public class AspectChangeContext implements ChangeContext {
34-
private static final Logger LOG = LoggerFactory.getLogger( AspectChangeContext.class );
32+
public class AspectChangeManager implements ChangeContext {
33+
private static final Logger LOG = LoggerFactory.getLogger( AspectChangeManager.class );
3534

3635
private final Deque<Change> undoStack = new ArrayDeque<>();
3736
private final Deque<Change> redoStack = new ArrayDeque<>();
3837
private final DefaultAspectModel aspectModel;
39-
private final AspectChangeContextConfig config;
38+
private final AspectChangeManagerConfig config;
4039
private final Map<AspectModelFile, FileState> fileState = new HashMap<>();
4140
private boolean isUndoOperation = false;
4241

4342
private enum FileState {
4443
CREATED, CHANGED, REMOVED
4544
}
4645

47-
public AspectChangeContext( final AspectChangeContextConfig config, final AspectModel aspectModel ) {
46+
public AspectChangeManager( final AspectChangeManagerConfig config, final AspectModel aspectModel ) {
4847
this.config = config;
4948
resetFileStates();
5049
if ( aspectModel instanceof final DefaultAspectModel defaultAspectModel ) {
@@ -54,8 +53,8 @@ public AspectChangeContext( final AspectChangeContextConfig config, final Aspect
5453
}
5554
}
5655

57-
public AspectChangeContext( final AspectModel aspectModel ) {
58-
this( AspectChangeContextConfigBuilder.builder().build(), aspectModel );
56+
public AspectChangeManager( final AspectModel aspectModel ) {
57+
this( AspectChangeManagerConfigBuilder.builder().build(), aspectModel );
5958
}
6059

6160
public synchronized ChangeReport applyChange( final Change change ) {
@@ -125,32 +124,29 @@ public Stream<AspectModelFile> aspectModelFiles() {
125124
}
126125

127126
@Override
128-
public AspectChangeContextConfig config() {
127+
public AspectChangeManagerConfig config() {
129128
return config;
130129
}
131130

132131
@Override
133-
public List<AspectModelFile> createdFiles() {
132+
public Stream<AspectModelFile> createdFiles() {
134133
return fileState.entrySet().stream()
135134
.filter( entry -> entry.getValue() == FileState.CREATED )
136-
.map( Map.Entry::getKey )
137-
.toList();
135+
.map( Map.Entry::getKey );
138136
}
139137

140138
@Override
141-
public List<AspectModelFile> modifiedFiles() {
139+
public Stream<AspectModelFile> modifiedFiles() {
142140
return fileState.entrySet().stream()
143141
.filter( entry -> entry.getValue() == FileState.CHANGED )
144-
.map( Map.Entry::getKey )
145-
.toList();
142+
.map( Map.Entry::getKey );
146143
}
147144

148145
@Override
149-
public List<AspectModelFile> removedFiles() {
146+
public Stream<AspectModelFile> removedFiles() {
150147
return fileState.entrySet().stream()
151148
.filter( entry -> entry.getValue() == FileState.REMOVED )
152-
.map( Map.Entry::getKey )
153-
.toList();
149+
.map( Map.Entry::getKey );
154150
}
155151

156152
@Override
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@
1818
import io.soabase.recordbuilder.core.RecordBuilder;
1919

2020
@RecordBuilder
21-
public record AspectChangeContextConfig(
21+
public record AspectChangeManagerConfig(
2222
List<String> defaultFileHeader,
2323
boolean detailedChangeReport
2424
) {
25-
public static AspectChangeContextConfig DEFAULT = AspectChangeContextConfigBuilder.builder().build();
25+
public static AspectChangeManagerConfig DEFAULT = AspectChangeManagerConfigBuilder.builder().build();
2626

27-
public AspectChangeContextConfig {
27+
public AspectChangeManagerConfig {
2828
if ( defaultFileHeader == null ) {
2929
defaultFileHeader = List.of();
3030
}

core/esmf-aspect-meta-model-java/src/main/java/org/eclipse/esmf/aspectmodel/edit/ChangeContext.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,21 +13,20 @@
1313

1414
package org.eclipse.esmf.aspectmodel.edit;
1515

16-
import java.util.List;
1716
import java.util.stream.Stream;
1817

1918
import org.eclipse.esmf.aspectmodel.AspectModelFile;
2019

2120
public interface ChangeContext {
2221
Stream<AspectModelFile> aspectModelFiles();
2322

24-
AspectChangeContextConfig config();
23+
AspectChangeManagerConfig config();
2524

26-
List<AspectModelFile> createdFiles();
25+
Stream<AspectModelFile> createdFiles();
2726

28-
List<AspectModelFile> modifiedFiles();
27+
Stream<AspectModelFile> modifiedFiles();
2928

30-
List<AspectModelFile> removedFiles();
29+
Stream<AspectModelFile> removedFiles();
3130

3231
void indicateFileIsAdded( AspectModelFile file );
3332

core/esmf-aspect-meta-model-java/src/main/java/org/eclipse/esmf/aspectmodel/edit/ChangeReportFormatter.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import org.apache.jena.rdf.model.Model;
2424
import org.apache.jena.rdf.model.ModelFactory;
2525

26-
public class ChangeReportFormatter implements BiFunction<ChangeReport, AspectChangeContextConfig, String> {
26+
public class ChangeReportFormatter implements BiFunction<ChangeReport, AspectChangeManagerConfig, String> {
2727
public static final ChangeReportFormatter INSTANCE = new ChangeReportFormatter();
2828

2929
private ChangeReportFormatter() {
@@ -37,7 +37,7 @@ private void handleSimpleEntry( final StringBuilder builder, final ChangeReport.
3737
}
3838

3939
private void handleMultipleEntries( final StringBuilder builder, final ChangeReport.MultipleEntries multipleEntries, final String indent,
40-
final int indentationLevel, final AspectChangeContextConfig config ) {
40+
final int indentationLevel, final AspectChangeManagerConfig config ) {
4141
if ( multipleEntries.summary() != null ) {
4242
builder.append( indent );
4343
builder.append( "- " );
@@ -58,7 +58,7 @@ private void handleMultipleEntries( final StringBuilder builder, final ChangeRep
5858
}
5959

6060
private void handleEntryWithDetails( final StringBuilder builder, final ChangeReport.EntryWithDetails entryWithDetails,
61-
final String indent, final AspectChangeContextConfig config ) {
61+
final String indent, final AspectChangeManagerConfig config ) {
6262
builder.append( indent );
6363
builder.append( "- " );
6464
builder.append( entryWithDetails.summary() );
@@ -99,7 +99,7 @@ private void handleEntryWithDetails( final StringBuilder builder, final ChangeRe
9999
}
100100
}
101101

102-
private void append( final StringBuilder builder, final ChangeReport report, final AspectChangeContextConfig config,
102+
private void append( final StringBuilder builder, final ChangeReport report, final AspectChangeManagerConfig config,
103103
final int indentationLevel ) {
104104
final String indent = " ".repeat( indentationLevel );
105105
if ( report instanceof final ChangeReport.SimpleEntry simpleEntry ) {
@@ -123,7 +123,7 @@ private String show( final Model model ) {
123123
}
124124

125125
@Override
126-
public String apply( final ChangeReport changeReport, final AspectChangeContextConfig config ) {
126+
public String apply( final ChangeReport changeReport, final AspectChangeManagerConfig config ) {
127127
final StringBuilder builder = new StringBuilder();
128128
append( builder, changeReport, config, 0 );
129129
return builder.toString();

0 commit comments

Comments
 (0)