Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

[java-firestore] CustomClassMapper.convertBigDecimal calls .abs() on Double, unintentionally flipping negative values to positive

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#14,533 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
85/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
java
Lĩnh vực
databases

Hướng nghiên cứu

Bắt đầu trong com.google.cloud.firestore.encoding.CustomClassMapper.java, tại convertBigDecimal, và lần theo cách DocumentSnapshot.toObject(Class) đi tới đó. Tái hiện việc giải tuần tự một giá trị số thực âm của Firestore vào một trường BigDecimal. Được xem là hoàn tất khi dấu âm được giữ nguyên và các chuyển đổi hiện có của số nguyên, BigDecimal và String vẫn tiếp tục hoạt động.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Environment
  • Artifact: com.google.cloud:google-cloud-firestore (also used via com.google.firebase:firebase-admin)
  • Affected Versions: 3.21.1 through 3.48.0 (latest)
  • File: com.google.cloud.firestore.encoding.CustomClassMapper.java
Describe the Bug

When deserializing a Firestore document containing a negative floating point number into a Java POJO field of type java.math.BigDecimal using DocumentSnapshot.toObject(Class<T>), the negative value is converted to a positive number.

Offending Code

In com.google.cloud.firestore.encoding.CustomClassMapper.java :

private static BigDecimal convertBigDecimal(Object o, DeserializeContext.ErrorPath errorPath) {
    if (o instanceof Integer) {
        return BigDecimal.valueOf(((Integer) o).intValue());
    } else if (o instanceof Long) {
        return BigDecimal.valueOf(((Long) o).longValue());
    } else if (o instanceof Double) {
        return BigDecimal.valueOf(((Double) o).doubleValue()).abs(); // <--- Calls .abs()
    } else if (o instanceof BigDecimal) {
        return (BigDecimal) o;
    } else if (o instanceof String) {
        return new BigDecimal((String) o);
    } else {
        throw errorPath.deserializeError(
            "Failed to convert a value of type " + o.getClass().getName() + " to BigDecimal");
    }
}
Ngôn ngữ chính
Java
Star
2.1k
Fork
1.2k
Merge trung bình
2 ngày 15 giờ
Pull request đã merge (30 ngày)
168

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của googleapis/google-cloud-java

Tất cả issue của googleapis/google-cloud-java

Issue tương tự

Thêm issue về Java

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.