From 902e756dfbde25e79d4a4acf9c4a8106f8a47dc0 Mon Sep 17 00:00:00 2001
From: Christopher Creutzig <christopher@creutzig.de>
Date: Sun, 11 Sep 2011 15:23:23 +0200
Subject: [PATCH] download progress bar

---
 .../plus/ProgressDialogImplementation.java    | 47 +++++++------------
 .../activities/DownloadIndexActivity.java     | 22 +++++++--
 2 files changed, 36 insertions(+), 33 deletions(-)

diff --git a/OsmAnd/src/net/osmand/plus/ProgressDialogImplementation.java b/OsmAnd/src/net/osmand/plus/ProgressDialogImplementation.java
index fbab3a6a3d..7fa9f52b19 100644
--- a/OsmAnd/src/net/osmand/plus/ProgressDialogImplementation.java
+++ b/OsmAnd/src/net/osmand/plus/ProgressDialogImplementation.java
@@ -10,11 +10,9 @@ import android.os.Message;
 
 public class ProgressDialogImplementation implements IProgress {
 	
-	private static final float deltaToChange = 0.023f;
 	private String taskName;
 	private int progress;
 	private int work;
-	private int deltaWork;
 	private String message = ""; //$NON-NLS-1$
 	
 	private Handler mViewUpdateHandler;
@@ -35,6 +33,13 @@ public class ProgressDialogImplementation implements IProgress {
 				super.handleMessage(msg);
 				if(dialog != null){
 					dialog.setMessage(message);
+					if (isIndeterminate()) {
+						dialog.setIndeterminate(true);
+					} else {
+						dialog.setIndeterminate(false);
+						dialog.setMax(work);
+					}
+					dialog.show();
 				}
 			}
 		};	
@@ -80,35 +85,18 @@ public class ProgressDialogImplementation implements IProgress {
 
 	@Override
 	public void progress(int deltaWork) {
-		this.deltaWork += deltaWork;
-		if(change(progress + this.deltaWork)){
-			this.progress += this.deltaWork;
-			this.deltaWork = 0;
-			updateMessage();
+		this.progress += deltaWork;
+		if (!isIndeterminate() && dialog != null) {
+			dialog.setProgress(this.progress);
 		}
 	}
 	
-	private void updateMessage() {
-		message = taskName + String.format(" %.1f %%", this.progress * 100f / ((float) this.work)); //$NON-NLS-1$
-		mViewUpdateHandler.sendEmptyMessage(0);
-	}
-
-	public boolean change(int newProgress) {
-		if (newProgress < progress) {
-			return false;
-		}
-		if ((newProgress - progress) / ((float) work) < deltaToChange) {
-			return false;
-		}
-		return true;
-	}
 	@Override
 	public void remaining(int remainingWork) {
-		if(change(work - remainingWork)){
-			this.progress = work - remainingWork;
-			updateMessage();
+		this.progress = work - remainingWork;
+		if (!isIndeterminate() && dialog != null) {
+			dialog.setProgress(this.progress);
 		}
-		deltaWork = work - remainingWork - this.progress;
 	}
 	
 	public boolean isIndeterminate(){
@@ -121,19 +109,19 @@ public class ProgressDialogImplementation implements IProgress {
 			taskName = ""; //$NON-NLS-1$
 		}
 		message = taskName;
-		mViewUpdateHandler.sendEmptyMessage(0);
 		this.taskName = taskName;
 		startWork(work);
+		mViewUpdateHandler.sendEmptyMessage(0);
 	}
 
 	@Override
 	public void finishTask() {
+		work = -1;
+		progress = 0;
 		if (taskName != null) {
 			message = context.getResources().getString(R.string.finished_task) +" : "+ taskName; //$NON-NLS-1$
 			mViewUpdateHandler.sendEmptyMessage(0);
 		}
-		work = -1;
-		progress = 0;
 	}
 	@Override
 	public boolean isInterrupted() {
@@ -149,11 +137,10 @@ public class ProgressDialogImplementation implements IProgress {
 	@Override
 	public void startWork(int work) {
 		this.work = work;
-		if(this.work == 0){
+		if (this.work == 0) {
 			this.work = 1;
 		}
 		progress = 0;
-		deltaWork = 0;
 	}
 
 	@Override
diff --git a/OsmAnd/src/net/osmand/plus/activities/DownloadIndexActivity.java b/OsmAnd/src/net/osmand/plus/activities/DownloadIndexActivity.java
index 1529f5fdf6..b58a1c5186 100644
--- a/OsmAnd/src/net/osmand/plus/activities/DownloadIndexActivity.java
+++ b/OsmAnd/src/net/osmand/plus/activities/DownloadIndexActivity.java
@@ -479,10 +479,26 @@ private class DownloadIndexesAsyncTask extends  AsyncTask<String, Object, String
 		
 		@Override
 		protected void onPreExecute() {
-			progressFileDlg = ProgressDialog.show(DownloadIndexActivity.this, getString(R.string.downloading),
-					getString(R.string.downloading_file), true, true);
+			progressFileDlg = new ProgressDialog(DownloadIndexActivity.this);
+			progressFileDlg.setTitle(getString(R.string.downloading));
+			progressFileDlg.setMessage(getString(R.string.downloading_file));
+			progressFileDlg.setIndeterminate(false);
+			progressFileDlg.setCancelable(true);
+			// we'd prefer a plain progress bar without numbers,
+			// but that is only available starting from API level 11
+			try {
+				ProgressDialog.class
+					.getMethod("setProgressNumberFormat", new Class[] { String.class })
+					.invoke(progressFileDlg, (String)null);
+			} catch (NoSuchMethodException nsme) {
+				// failure, must be older device
+			} catch (IllegalAccessException nsme) {
+				// failure, must be older device
+			} catch (java.lang.reflect.InvocationTargetException nsme) {
+				// failure, must be older device
+			}
 			downloadFileHelper.setInterruptDownloading(false);
-			progressFileDlg.show();
+			progressFileDlg.setProgressStyle(ProgressDialog.STYLE_HORIZONTAL);
 			progress = new ProgressDialogImplementation(progressFileDlg, true);
 			progressFileDlg.setOnCancelListener(new DialogInterface.OnCancelListener() {
 				@Override