Fix: add resourceId existence checking method

Previously, the existence of a resourceId was determined based on whether it was a positive number. However, since resourceId can also be negative, I improved this by checking the flag in the `fileds` variable to determine whether resourceId exists.
bumptech · Sep 27, 2024 · 5f23a12 · 5f23a12
1 parent a7351b0
commit 5f23a12
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/library/src/main/java/com/bumptech/glide/request/BaseRequestOptions.java b/library/src/main/java/com/bumptech/glide/request/BaseRequestOptions.java
@@ -1363,11 +1363,21 @@ public final int getErrorId() {
     return errorId;
   }
 
+  @SuppressWarnings("WeakerAccess")
+  public final boolean hasErrorId() {
+    return isSet(ERROR_ID);
+  }
+
   @SuppressWarnings("WeakerAccess")
   public final int getPlaceholderId() {
     return placeholderId;
   }
 
+  @SuppressWarnings("WeakerAccess")
+  public final boolean hasPlaceholderId() {
+    return isSet(PLACEHOLDER_ID);
+  }
+
   @SuppressWarnings("WeakerAccess")
   @Nullable
   public final Drawable getPlaceholderDrawable() {
@@ -1379,6 +1389,11 @@ public final int getFallbackId() {
     return fallbackId;
   }
 
+  @SuppressWarnings("WeakerAccess")
+  public final boolean hasFallbackId() {
+    return isSet(FALLBACK_ID);
+  }
+
   @SuppressWarnings("WeakerAccess")
   @Nullable
   public final Drawable getFallbackDrawable() {

diff --git a/library/src/main/java/com/bumptech/glide/request/SingleRequest.java b/library/src/main/java/com/bumptech/glide/request/SingleRequest.java
@@ -387,7 +387,7 @@ public boolean isAnyResourceSet() {
   private Drawable getErrorDrawable() {
     if (errorDrawable == null) {
       errorDrawable = requestOptions.getErrorPlaceholder();
-      if (errorDrawable == null && requestOptions.getErrorId() > 0) {
+      if (errorDrawable == null && requestOptions.hasErrorId()) {
         errorDrawable = loadDrawable(requestOptions.getErrorId());
       }
     }
@@ -398,7 +398,7 @@ private Drawable getErrorDrawable() {
   private Drawable getPlaceholderDrawable() {
     if (placeholderDrawable == null) {
       placeholderDrawable = requestOptions.getPlaceholderDrawable();
-      if (placeholderDrawable == null && requestOptions.getPlaceholderId() > 0) {
+      if (placeholderDrawable == null && requestOptions.hasPlaceholderId()) {
         placeholderDrawable = loadDrawable(requestOptions.getPlaceholderId());
       }
     }
@@ -409,7 +409,7 @@ private Drawable getPlaceholderDrawable() {
   private Drawable getFallbackDrawable() {
     if (fallbackDrawable == null) {
       fallbackDrawable = requestOptions.getFallbackDrawable();
-      if (fallbackDrawable == null && requestOptions.getFallbackId() > 0) {
+      if (fallbackDrawable == null && requestOptions.hasFallbackId()) {
         fallbackDrawable = loadDrawable(requestOptions.getFallbackId());
       }
     }