Fix: Gallery crash when clicking null images in replies

[graycreate]

Summary

• Fixes crash that occurs when clicking on images with null data in reply lists
• Only affected release builds due to ProGuard obfuscation issues
• Adds validation and null safety checks to prevent crashes

Problem

The app crashed with a NullPointerException when users clicked on images in reply lists that had null or invalid image data. This only occurred in release builds because ProGuard was obfuscating the ImagesInfo classes, breaking the Fruit HTML parsing library's reflection mechanism.

Solution

1. Added validation in GalleryActivity.open(): Checks if images are valid before opening gallery
2. Added null safety checks in GalleryAdapter: Handles null image objects gracefully
3. Added ProGuard rules: Prevents ImagesInfo classes from being obfuscated
4. Shows user-friendly message: Displays "图片不存在" toast instead of crashing

Testing

• Tested with debug build - no crash
• Tested with release build - no crash
• Clicking valid images opens gallery normally
• Clicking invalid/null images shows toast message

Related Issues

Fixes crash reported when clicking images in reply lists

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR fixes a crash that occurs when clicking on images with null or invalid data in reply lists, specifically affecting release builds due to ProGuard obfuscation issues with the Fruit HTML parsing library.

• Adds comprehensive null safety validation before opening the gallery
• Implements defensive programming in gallery adapter and activity methods
• Adds ProGuard rules to prevent obfuscation of ImagesInfo classes

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
GalleryActivity.java	Adds validation in open() method and null checks in image handling methods
GalleryAdapter.java	Adds bounds checking and null safety in getItem() and instantiateItem() methods
proguard-rules.pro	Adds keep rules for ImagesInfo classes to prevent obfuscation

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Parameter name has a typo: 'postion' should be 'position'.

Suggested change

	private ImagesInfo.Images.Image getItem(int postion) {
	if (mImagesInfo == null || mImagesInfo.getImages() == null ||
	postion < 0 || postion >= mImagesInfo.getImages().size()) {
	return null;
	}
	return mImagesInfo.getImages().get(postion);
	private ImagesInfo.Images.Image getItem(int position) {
	if (mImagesInfo == null || mImagesInfo.getImages() == null ||
	position < 0 || position >= mImagesInfo.getImages().size()) {
	return null;
	}
	return mImagesInfo.getImages().get(position);

[Copilot]

Variable name has a typo: 'postion' should be 'position' (appears twice in this condition).

Suggested change

	private ImagesInfo.Images.Image getItem(int postion) {
	if (mImagesInfo == null || mImagesInfo.getImages() == null ||
	postion < 0 || postion >= mImagesInfo.getImages().size()) {
	return null;
	}
	return mImagesInfo.getImages().get(postion);
	private ImagesInfo.Images.Image getItem(int position) {
	if (mImagesInfo == null || mImagesInfo.getImages() == null ||
	position < 0 || position >= mImagesInfo.getImages().size()) {
	return null;
	}
	return mImagesInfo.getImages().get(position);