--- android/TrainInfo/src/dk/thoerup/traininfo/TimetableList.java	2009/09/30 09:14:27	365
+++ android/TrainInfo/src/dk/thoerup/traininfo/TimetableList.java	2010/09/08 06:03:45	1027
@@ -1,6 +1,12 @@
 package dk.thoerup.traininfo;
 
 
+import static dk.thoerup.traininfo.R.string.generic_cancel;
+import static dk.thoerup.traininfo.R.string.generic_retry;
+import static dk.thoerup.traininfo.R.string.timetablelist_fetchdata;
+import static dk.thoerup.traininfo.R.string.timetablelist_fetcherror;
+import static dk.thoerup.traininfo.R.string.timetablelist_nodata;
+
 import java.util.ArrayList;
 import java.util.List;
 
@@ -13,6 +19,8 @@
 import android.os.AsyncTask;
 import android.os.Bundle;
 import android.util.Log;
+import android.view.View;
+import android.widget.ListView;
 import android.widget.TextView;
 import dk.thoerup.traininfo.provider.ProviderFactory;
 import dk.thoerup.traininfo.provider.TimetableProvider;
@@ -21,10 +29,11 @@
 public class TimetableList extends ListActivity {
 	
 	private static final int DLG_PROGRESS = 8000;
-	DepartureBean departure;
+	DepartureEntry departure;
 	TimetableListAdapter adapter;
 	TimetableFetcher fetcher;
 	List<TimetableBean> timetables;
+	int commFailCounter = 0;
 	
 	TimetableProvider provider; 
 
@@ -36,17 +45,20 @@
 		
 		provider = ProviderFactory.getTimetableProvider();
 		
-		adapter = new TimetableListAdapter(this);
+		adapter = new TimetableListAdapter(this);		
 		setListAdapter(adapter);
+
+
 		
 		Intent launchedBy = getIntent();
-		departure = (DepartureBean) launchedBy.getSerializableExtra("departure");
+		departure = (DepartureEntry) launchedBy.getSerializableExtra("departure");
 		
 		((TextView)findViewById(R.id.Train)).setText(departure.getTrainNumber());
 		((TextView)findViewById(R.id.Status)).setText(departure.getStatus());
+		((TextView)findViewById(R.id.Location)).setText(departure.getLocation());
 		((TextView)findViewById(R.id.Note)).setText(departure.getNote());
-		((TextView)findViewById(R.id.Updated)).setText(departure.getLastUpdateString());
-		
+		((TextView)findViewById(R.id.Updated)).setText(departure.getLastUpdateString(this));
+						
 		
 		if (savedInstanceState == null) {
 			startTimetableFetcher();
@@ -56,27 +68,47 @@
 		}
 	}
 	
-    @Override 
+	@Override
+	protected void onStart() {
+		super.onStart();
+		ProviderFactory.purgeOldEntries();
+	}
+
+	@Override
+	protected void onDestroy() {
+		super.onDestroy();
+		
+		if (fetcher != null) {
+			fetcher.cancel(true);
+		}
+	}
+	
+    @Override
+	protected void onListItemClick(ListView l, View v, int position, long id) {
+		super.onListItemClick(l, v, position, id);
+
+		TimetableBean tt = timetables.get(position);
+		
+		StationBean station = new StationBean();
+		station.setName( tt.getStation() );
+		station.setId( tt.getStationId() );
+		station.setRegional(true);
+		
+		Intent intent = new Intent(this, DepartureList.class);
+		intent.putExtra("stationbean", station);
+		startActivity(intent);
+		
+	}
+    
+
+
+	@Override 
     public void onSaveInstanceState(Bundle outState) 
     {        	
     	dismissDialog(DLG_PROGRESS);
     	outState.putSerializable("timetables", (ArrayList<TimetableBean>) timetables);
     }
 
-/* 		case DLG_DETAILS:
-			DepartureBean currentDeparture = departures.get(selectedItemId);
-			((TextView)dialog.findViewById(R.id.Time)).setText(currentDeparture.getTime());
-			((TextView)dialog.findViewById(R.id.Train)).setText(currentDeparture.getTrainNumber());
-			((TextView)dialog.findViewById(R.id.Destination)).setText( currentDeparture.getDestination());
-			((TextView)dialog.findViewById(R.id.Origin)).setText(currentDeparture.getOrigin());
-			((TextView)dialog.findViewById(R.id.Location)).setText(currentDeparture.getLocation());
-			((TextView)dialog.findViewById(R.id.Updated)).setText(currentDeparture.getLastUpdateString());
-			((TextView)dialog.findViewById(R.id.Status)).setText(currentDeparture.getStatus());
-			((TextView)dialog.findViewById(R.id.Note)).setText(currentDeparture.getNote());
-			detailsDialog = dialog;
-			break;
-
- */
 	
 	@Override
 	protected void onPrepareDialog(int id, Dialog dialog) {
@@ -93,8 +125,8 @@
 	protected Dialog onCreateDialog(int id) {
 		switch (id) {
 		case DLG_PROGRESS:
-			ProgressDialog dlg = new ProgressDialog(this);
-			dlg.setMessage("Fetch timetable data");
+			ProgressDialog dlg = new ProgressDialog(this);			
+			dlg.setMessage( getString(timetablelist_fetchdata) );
 			dlg.setCancelable(true);
 			return dlg;			
 		default:
@@ -105,12 +137,11 @@
 	void startTimetableFetcher() {
 		showDialog(DLG_PROGRESS);
 		fetcher = new TimetableFetcher(); 
-		fetcher.execute(departure.getTrainNumber());
+		fetcher.execute(departure.getType(), departure.getTrainNumber());
 	}
 	
 	class TimetableFetcher extends AsyncTask<String,Void,Void> {
 		
-		boolean success;
 		
 		@Override
 		protected void onPostExecute(Void result) {
@@ -118,37 +149,49 @@
 			dismissDialog(DLG_PROGRESS);
 	
 			
-			if (success) {
+			if (timetables != null) {
+				commFailCounter = 0;
+				TimetableList.this.getListView().invalidateViews();
 				adapter.setTimetable(timetables);
 				if (timetables.size() == 0) {
-					MessageBox.showMessage(TimetableList.this, "No timetable found");
+					MessageBox.showMessage(TimetableList.this, getString(timetablelist_nodata), true);
 				}
 			} else { // communication or parse error
+				commFailCounter++;
 				AlertDialog.Builder builder = new AlertDialog.Builder(TimetableList.this);						
-				builder.setMessage("Error finding departures");
+				builder.setMessage(getString(timetablelist_fetcherror));
 				builder.setCancelable(true);
-				builder.setPositiveButton("Retry", new DialogInterface.OnClickListener() {
-					public void onClick(DialogInterface dialog, int id) {
-						dialog.dismiss();
-						startTimetableFetcher();
-						
-					}
-				});
-				builder.setNegativeButton("Cancel", new DialogInterface.OnClickListener() {
+				if (commFailCounter < 3) {
+					builder.setPositiveButton(getString(generic_retry), new DialogInterface.OnClickListener() {
+						public void onClick(DialogInterface dialog, int id) {
+							dialog.dismiss();
+							startTimetableFetcher();
+							
+						}
+					});
+				}
+				builder.setNegativeButton(getString(generic_cancel), new DialogInterface.OnClickListener() {
 					public void onClick(DialogInterface dialog, int id) {
 						dialog.dismiss();
+						TimetableList.this.finish();
 					}							
-				});												
-				builder.show();				
+				});
+				
+				try {
+					builder.show();
+				} catch (android.view.WindowManager.BadTokenException e) {					
+					Log.i("TimetableList", "BadTokenException"); // this can happen if the user switched away from this activity, while doInBackground was running
+				}
+								
 			}
 			
 		}
 		
 		@Override
 		protected Void doInBackground(String... arg0) {
-			String trainID = arg0[0];
-			success = provider.lookupTimetable(trainID);
-			timetables = provider.getTimetable();
+			String type = arg0[0];
+			String trainID = arg0[1];
+			timetables = provider.lookupTimetable(type, trainID);
 			
 			return null;
 		}