Compare commits

...

2 Commits

Author SHA1 Message Date
Kai S. K. Engelbart ae2f2e8a84
Merge pull request 'Fix ChangeManagerWrapper' (#8) from b/javafx-wrapper-properties into develop
zdm/undo-redo/pipeline/head This commit looks good Details
Reviewed-on: https://git.kske.dev/zdm/undo-redo/pulls/8
Reviewed-by: DieGurke <maxi@kske.dev>
Reviewed-by: delvh <leon@kske.dev>
2021-12-25 21:14:53 +01:00
Kai S. K. Engelbart 7101523584
Update properties on wrapper creation, fix NPE
zdm/undo-redo/pipeline/head This commit looks good Details
2021-12-25 21:57:25 +02:00
2 changed files with 2 additions and 1 deletions

View File

@ -47,6 +47,7 @@ public class ChangeManagerWrapper<C extends Change, M extends ChangeManager<C>>
*/
public ChangeManagerWrapper(M manager) {
this.manager = manager;
updateProperties();
}
@Override

View File

@ -21,7 +21,7 @@ public interface ObservableChangeManager<C extends Change> extends ChangeManager
@Override
default Optional<C> getLastChange() {
return Optional.of(lastChangeProperty().get());
return Optional.ofNullable(lastChangeProperty().get());
}
ReadOnlyBooleanProperty atMarkedChangeProperty();