thedevstack/PiratX-NoCleanFork
2
0
Fork 0
forked from mirror/monocles_chat
Code Issues 5 Pull requests Wiki Activity

do not compress images with alpha channels

Browse source 
(cherry picked from commit 2155a50875)
This commit is contained in:
Daniel Gultsch 2021-01-29 21:25:00 +01:00 • committed by Christian Schneppe
parent 8cf631a9cf
commit 4a454b74ac
3 changed files with 23 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
src/main/java/eu/siacs/conversations/persistance/FileBackend.java
View file

@ -525,8 +525,11 @@ public class FileBackend {
return pos > 0 ? filename.substring(pos + 1) : null;
}
private void copyImageToPrivateStorage(File file, Uri image, int sampleSize) throws FileCopyException, NotAnImageFileException {
file.getParentFile().mkdirs();
private void copyImageToPrivateStorage(File file, Uri image, int sampleSize) throws FileCopyException, ImageCompressionException {
final File parent = file.getParentFile();
if (parent.mkdirs()) {
Log.d(Config.LOGTAG,"created parent directory");
}
InputStream is = null;
OutputStream os = null;
try {
@ -545,7 +548,11 @@ public class FileBackend {
originalBitmap = BitmapFactory.decodeStream(is, null, options);
is.close();
if (originalBitmap == null) {
throw new NotAnImageFileException();
throw new ImageCompressionException("Source file was not an image");
}
if (hasAlpha(originalBitmap)) {
originalBitmap.recycle();
throw new ImageCompressionException("Source file had alpha channel");
}
int size;
if (mXmppConnectionService.getCompressImageResolutionPreference() == 0) {
@ -591,12 +598,12 @@ public class FileBackend {
}
}
public void copyImageToPrivateStorage(File file, Uri image) throws FileCopyException, NotAnImageFileException {
public void copyImageToPrivateStorage(File file, Uri image) throws FileCopyException, ImageCompressionException {
Log.d(Config.LOGTAG, "copy image (" + image.toString() + ") to private storage " + file.getAbsolutePath());
copyImageToPrivateStorage(file, image, 0);
}
public void copyImageToPrivateStorage(Message message, Uri image) throws FileCopyException, NotAnImageFileException {
public void copyImageToPrivateStorage(Message message, Uri image) throws FileCopyException, ImageCompressionException {
String filename = "Sent/" + fileDateFormat.format(new Date(message.getTimeSent())) + "_" + message.getUuid().substring(0, 4);
switch (Config.IMAGE_FORMAT) {
case JPEG:
@ -664,11 +671,11 @@ public class FileBackend {
} else if (mime.startsWith("video/")) {
thumbnail = getVideoPreview(file, size);
} else if (mime.startsWith("image/")) {
Bitmap fullsize = getFullsizeImagePreview(file, size);
if (fullsize == null) {
final Bitmap fullSize = getFullsizeImagePreview(file, size);
if (fullSize == null) {
throw new FileNotFoundException();
}
thumbnail = resize(fullsize, size);
thumbnail = resize(fullSize, size);
thumbnail = rotate(thumbnail, getRotation(file));
if (mime.equals("image/gif")) {
Bitmap withGifOverlay = thumbnail.copy(Bitmap.Config.ARGB_8888, true);
@ -1534,10 +1541,14 @@ public class FileBackend {
}
}
public static class NotAnImageFileException extends Exception {
public static class ImageCompressionException extends Exception {
ImageCompressionException(String message) {
super(message);
}
}
public static class FileCopyException extends Exception {
private int resId;

3
src/main/java/eu/siacs/conversations/services/XmppConnectionService.java
View file

@ -603,7 +603,8 @@ public class XmppConnectionService extends Service {
mFileAddingExecutor.execute(() -> {
try {
getFileBackend().copyImageToPrivateStorage(message, uri);
} catch (FileBackend.NotAnImageFileException e) {
} catch (FileBackend.ImageCompressionException e) {
Log.d(Config.LOGTAG, "unable to compress image. fall back to file transfer", e);
attachFileToConversation(conversation, uri, mimeType, callback);
return;
} catch (final FileBackend.FileCopyException e) {

1
src/main/java/eu/siacs/conversations/ui/PublishProfilePictureActivity.java
View file

@ -133,6 +133,7 @@ public class PublishProfilePictureActivity extends XmppActivity implements XmppC
@Override
public void onActivityResult(int requestCode, int resultCode, Intent data) {
super.onActivityResult(requestCode, resultCode, data);
if (requestCode == CropImage.CROP_IMAGE_ACTIVITY_REQUEST_CODE) {
CropImage.ActivityResult result = CropImage.getActivityResult(data);
if (resultCode == RESULT_OK) {