We have an interface offering notify. OpenRewrite changes the calls to notifyAll. However, this method does not exist.
diff --git a/jabgui/src/main/java/org/jabref/gui/frame/SendAsEMailAction.java b/jabgui/src/main/java/org/jabref/gui/frame/SendAsEMailAction.java
index f459878e40..db8b796206 100644
--- a/jabgui/src/main/java/org/jabref/gui/frame/SendAsEMailAction.java
+++ b/jabgui/src/main/java/org/jabref/gui/frame/SendAsEMailAction.java
@@ -60,7 +60,7 @@ public abstract class SendAsEMailAction extends SimpleCommand {
.onFailure(e -> {
String message = Localization.lang("Error creating email");
LOGGER.warn(message, e);
- dialogService.notify(message);
+ dialogService.notifyAll(message);
})
.executeWith(taskExecutor);
}
Our interface: https://github.com/JabRef/jabref/blob/main/jablib/src/main/java/org/jabref/logic/util/NotificationService.java
Expected: No change by org.openrewrite.staticanalysis.UseObjectNotifyAll if notify method is not from java.lang.Object.
We have an interface offering
notify. OpenRewrite changes the calls tonotifyAll. However, this method does not exist.Our interface: https://github.com/JabRef/jabref/blob/main/jablib/src/main/java/org/jabref/logic/util/NotificationService.java
Expected: No change by
org.openrewrite.staticanalysis.UseObjectNotifyAllifnotifymethod is not fromjava.lang.Object.