Browse Source

Rename interface and implementation for Deck integration

https://github.com/stefan-niedermann/nextcloud-deck/issues/208
Stefan Niedermann 5 years ago
parent
commit
53a1344e47

+ 2 - 2
src/main/java/com/nextcloud/client/integration/deck/DeckActionOverride.java → src/main/java/com/nextcloud/client/integration/deck/DeckApi.java

@@ -9,8 +9,8 @@ import com.owncloud.android.lib.resources.notifications.models.Notification;
 
 import androidx.annotation.NonNull;
 
-public interface DeckActionOverride {
+public interface DeckApi {
 
     @NonNull
-    Optional<PendingIntent> handleNotification(@NonNull final Notification notification, @NonNull final User user);
+    Optional<PendingIntent> createForwardToDeckActionIntent(@NonNull final Notification notification, @NonNull final User user);
 }

+ 4 - 5
src/main/java/com/nextcloud/client/integration/deck/DeckActionOverrideImpl.java → src/main/java/com/nextcloud/client/integration/deck/DeckApiImpl.java

@@ -23,7 +23,6 @@ package com.nextcloud.client.integration.deck;
 import android.app.PendingIntent;
 import android.content.Context;
 import android.content.Intent;
-import android.content.pm.PackageManager;
 import android.util.Log;
 
 import com.nextcloud.client.account.User;
@@ -32,9 +31,9 @@ import com.owncloud.android.lib.resources.notifications.models.Notification;
 
 import androidx.annotation.NonNull;
 
-public class DeckActionOverrideImpl implements DeckActionOverride {
+public class DeckApiImpl implements DeckApi {
 
-    private static final String TAG = DeckActionOverrideImpl.class.getSimpleName();
+    private static final String TAG = DeckApiImpl.class.getSimpleName();
 
     private static final String APP_NAME = "deck";
     private static final String DECK_APP_ID_BASE = "it.niedermann.nextcloud.deck";
@@ -43,13 +42,13 @@ public class DeckActionOverrideImpl implements DeckActionOverride {
 
     private final Context context;
 
-    public DeckActionOverrideImpl(@NonNull Context context) {
+    public DeckApiImpl(@NonNull Context context) {
         this.context = context;
     }
 
     @NonNull
     @Override
-    public Optional<PendingIntent> handleNotification(@NonNull Notification notification, @NonNull User user) {
+    public Optional<PendingIntent> createForwardToDeckActionIntent(@NonNull Notification notification, @NonNull User user) {
         if (!APP_NAME.equalsIgnoreCase(notification.app)) {
             return Optional.empty();
         }