diff --git a/OsmAnd-java/src/main/java/net/osmand/PlatformUtil.java b/OsmAnd-java/src/main/java/net/osmand/PlatformUtil.java index ffc9bf05b0..fd3fa2e2eb 100644 --- a/OsmAnd-java/src/main/java/net/osmand/PlatformUtil.java +++ b/OsmAnd-java/src/main/java/net/osmand/PlatformUtil.java @@ -23,7 +23,9 @@ public class PlatformUtil { } public static XmlPullParser newXMLPullParser() throws XmlPullParserException{ - return new org.kxml2.io.KXmlParser(); + org.kxml2.io.KXmlParser xmlParser = new org.kxml2.io.KXmlParser(); + xmlParser.setFeature(XmlPullParser.FEATURE_PROCESS_NAMESPACES, true); + return xmlParser; } public static XmlSerializer newSerializer() { diff --git a/OsmAnd/src/net/osmand/plus/NavigationService.java b/OsmAnd/src/net/osmand/plus/NavigationService.java index 16f1afddcc..cb7d249d10 100644 --- a/OsmAnd/src/net/osmand/plus/NavigationService.java +++ b/OsmAnd/src/net/osmand/plus/NavigationService.java @@ -4,16 +4,13 @@ import android.app.Notification; import android.app.Service; import android.content.Context; import android.content.Intent; -import android.location.LocationManager; import android.os.Binder; import android.os.IBinder; -import android.util.Log; import android.widget.Toast; import androidx.annotation.NonNull; import net.osmand.Location; -import net.osmand.PlatformUtil; import net.osmand.plus.helpers.LocationServiceHelper; import net.osmand.plus.helpers.LocationServiceHelper.LocationCallback; import net.osmand.plus.notifications.OsmandNotification; @@ -95,12 +92,6 @@ public class NavigationService extends Service { @Override public void onLocationAvailability(boolean locationAvailable) { - if (!locationAvailable) { - OsmandApplication app = (OsmandApplication) getApplication(); - if (app != null) { - app.showToastMessage(getString(R.string.off_router_service_no_gps_available)); - } - } } }); } catch (SecurityException e) { @@ -137,10 +128,12 @@ public class NavigationService extends Service { app.setNavigationService(null); usedBy = 0; // remove updates - try { - locationServiceHelper.removeLocationUpdates(); - } catch (SecurityException e) { - // Location service permission not granted + if (locationServiceHelper != null) { + try { + locationServiceHelper.removeLocationUpdates(); + } catch (SecurityException e) { + // Location service permission not granted + } } // remove notification stopForeground(Boolean.TRUE); diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/UploadPhotosAsyncTask.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/UploadPhotosAsyncTask.java index e552cb1373..2cfc70f14e 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/UploadPhotosAsyncTask.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/UploadPhotosAsyncTask.java @@ -4,7 +4,6 @@ import android.content.DialogInterface; import android.content.DialogInterface.OnDismissListener; import android.graphics.Bitmap; import android.graphics.BitmapFactory; -import android.graphics.Matrix; import android.net.Uri; import android.os.AsyncTask; @@ -102,10 +101,11 @@ public class UploadPhotosAsyncTask extends AsyncTask { private boolean handleSelectedImage(final Uri uri) { boolean success = false; InputStream inputStream = null; + int[] imageDimensions = null; try { inputStream = app.getContentResolver().openInputStream(uri); if (inputStream != null) { - success = uploadImageToPlace(inputStream); + imageDimensions = calcImageDimensions(inputStream); } } catch (Exception e) { LOG.error(e); @@ -113,12 +113,27 @@ public class UploadPhotosAsyncTask extends AsyncTask { } finally { Algorithms.closeStream(inputStream); } + if (imageDimensions != null && imageDimensions.length == 2) { + try { + inputStream = app.getContentResolver().openInputStream(uri); + if (inputStream != null) { + int width = imageDimensions[0]; + int height = imageDimensions[1]; + success = uploadImageToPlace(inputStream, width, height); + } + } catch (Exception e) { + LOG.error(e); + app.showToastMessage(R.string.cannot_upload_image); + } finally { + Algorithms.closeStream(inputStream); + } + } return success; } - private boolean uploadImageToPlace(InputStream image) { + private boolean uploadImageToPlace(InputStream image, int width, int height) { boolean success = false; - InputStream serverData = new ByteArrayInputStream(compressImageToJpeg(image)); + InputStream serverData = new ByteArrayInputStream(compressImageToJpeg(image, width, height)); String baseUrl = OPRConstants.getBaseUrl(app); // all these should be constant String url = baseUrl + "api/ipfs/image"; @@ -178,28 +193,35 @@ public class UploadPhotosAsyncTask extends AsyncTask { } } - private byte[] compressImageToJpeg(InputStream image) { + private int[] calcImageDimensions(InputStream image) { BufferedInputStream bufferedInputStream = new BufferedInputStream(image); BitmapFactory.Options opts = new BitmapFactory.Options(); - opts.inSampleSize = 4; - Bitmap bmp = BitmapFactory.decodeStream(bufferedInputStream, null, opts); - ByteArrayOutputStream os = new ByteArrayOutputStream(); - int h = bmp.getHeight(); - int w = bmp.getWidth(); + opts.inJustDecodeBounds = true; + BitmapFactory.decodeStream(bufferedInputStream, null, opts); + return new int[] { opts.outWidth, opts.outHeight }; + } + + private byte[] compressImageToJpeg(InputStream image, int width, int height) { + BufferedInputStream bufferedInputStream = new BufferedInputStream(image); + int w = width; + int h = height; boolean scale = false; + int divider = 1; while (w > MAX_IMAGE_LENGTH || h > MAX_IMAGE_LENGTH) { - w = w / 2; - h = h / 2; + w /= 2; + h /= 2; + divider *= 2; scale = true; } + Bitmap bmp; if (scale) { - Matrix matrix = new Matrix(); - matrix.postScale(w, h); - Bitmap resizedBitmap = Bitmap.createBitmap( - bmp, 0, 0, w, h, matrix, false); - bmp.recycle(); - bmp = resizedBitmap; + BitmapFactory.Options opts = new BitmapFactory.Options(); + opts.inSampleSize = divider; + bmp = BitmapFactory.decodeStream(bufferedInputStream, null, opts); + } else { + bmp = BitmapFactory.decodeStream(bufferedInputStream); } + ByteArrayOutputStream os = new ByteArrayOutputStream(); bmp.compress(Bitmap.CompressFormat.JPEG, 90, os); return os.toByteArray(); } diff --git a/OsmAnd/src/net/osmand/plus/voice/JSMediaCommandPlayerImpl.java b/OsmAnd/src/net/osmand/plus/voice/JSMediaCommandPlayerImpl.java index 80fd5381ab..fd9caafe2c 100644 --- a/OsmAnd/src/net/osmand/plus/voice/JSMediaCommandPlayerImpl.java +++ b/OsmAnd/src/net/osmand/plus/voice/JSMediaCommandPlayerImpl.java @@ -90,9 +90,12 @@ public class JSMediaCommandPlayerImpl extends MediaCommandPlayerImpl { if (voiceDir.getName().contains("tts")) { return false; } - for (File f : voiceDir.listFiles()) { - if (f.getName().endsWith(IndexConstants.TTSVOICE_INDEX_EXT_JS)) { - return true; + File[] files = voiceDir.listFiles(); + if (files != null) { + for (File f : files) { + if (f.getName().endsWith(IndexConstants.TTSVOICE_INDEX_EXT_JS)) { + return true; + } } } return false;