Skip to content

Commit b576ecd

Browse files
committed
FDL getDynamicLink(Intent) method to accept null intent to be in sync with the documentation and since the intent is not a must parameter to derive the pending FDL.
1 parent 086eded commit b576ecd

File tree

3 files changed

+21
-8
lines changed

3 files changed

+21
-8
lines changed

firebase-dynamic-links/src/main/java/com/google/firebase/dynamiclinks/FirebaseDynamicLinks.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
import android.content.Intent;
1919
import android.net.Uri;
2020
import androidx.annotation.NonNull;
21+
import androidx.annotation.Nullable;
2122
import com.google.android.gms.tasks.OnCompleteListener;
2223
import com.google.android.gms.tasks.OnFailureListener;
2324
import com.google.android.gms.tasks.OnSuccessListener;
@@ -99,7 +100,7 @@ public static synchronized FirebaseDynamicLinks getInstance(@NonNull FirebaseApp
99100
* <p>If a dynamic link, the call will also send FirebaseAnalytics dynamic link event.
100101
*/
101102
@NonNull
102-
public abstract Task<PendingDynamicLinkData> getDynamicLink(@NonNull Intent intent);
103+
public abstract Task<PendingDynamicLinkData> getDynamicLink(@Nullable Intent intent);
103104

104105
/**
105106
* Determine if the app has a pending dynamic link and provide access to the dynamic link

firebase-dynamic-links/src/main/java/com/google/firebase/dynamiclinks/internal/FirebaseDynamicLinksImpl.java

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -92,15 +92,20 @@ public PendingDynamicLinkData getPendingDynamicLinkData(@NonNull Intent intent)
9292
}
9393

9494
@Override
95-
public Task<PendingDynamicLinkData> getDynamicLink(@NonNull final Intent intent) {
95+
public Task<PendingDynamicLinkData> getDynamicLink(@Nullable final Intent intent) {
96+
String dynamicLinkDataString = intent != null ? intent.getDataString() : null;
9697
Task<PendingDynamicLinkData> result =
97-
googleApi.doWrite(new GetDynamicLinkImpl(analytics, intent.getDataString()));
98-
PendingDynamicLinkData pendingDynamicLinkData = getPendingDynamicLinkData(intent);
99-
if (pendingDynamicLinkData != null) {
100-
// DynamicLinkData included in the Intent, return it immediately and allow the Task to run in
101-
// the background to do logging and mark the FDL as returned.
102-
result = Tasks.forResult(pendingDynamicLinkData);
98+
googleApi.doWrite(new GetDynamicLinkImpl(analytics, dynamicLinkDataString));
99+
100+
if (intent != null) {
101+
PendingDynamicLinkData pendingDynamicLinkData = getPendingDynamicLinkData(intent);
102+
if (pendingDynamicLinkData != null) {
103+
// DynamicLinkData included in the Intent, return it immediately and allow the Task to run
104+
// in the background to do logging and mark the FDL as returned.
105+
result = Tasks.forResult(pendingDynamicLinkData);
106+
}
103107
}
108+
104109
return result;
105110
}
106111

firebase-dynamic-links/src/test/java/com/google/firebase/dynamiclinks/internal/FirebaseDynamicLinksImplTest.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,13 @@ public void testGetDynamicLink_Intent() {
153153
.doWrite(ArgumentMatchers.<TaskApiCall<DynamicLinksClient, PendingDynamicLinkData>>any());
154154
}
155155

156+
@Test
157+
public void testGetDynamicLink_NullIntent() {
158+
api.getDynamicLink((Intent) null);
159+
verify(mockGoogleApi)
160+
.doWrite(ArgumentMatchers.<TaskApiCall<DynamicLinksClient, PendingDynamicLinkData>>any());
161+
}
162+
156163
@Test
157164
public void testGetDynamicLink_IntentWithDynamicLinkData() {
158165
Bundle extensions = new Bundle();

0 commit comments

Comments
 (0)