--- android/Side9/src/dk/thoerup/side9/Side9WidgetProvider.java	2010/03/23 14:56:52	633
+++ android/Side9/src/dk/thoerup/side9/Side9WidgetProvider.java	2010/03/24 08:57:30	635
@@ -3,17 +3,22 @@
 import java.net.URL;
 import java.net.URLConnection;
 
+import android.app.PendingIntent;
 import android.appwidget.AppWidgetManager;
 import android.appwidget.AppWidgetProvider;
-import android.appwidget.AppWidgetProviderInfo;
 import android.content.Context;
+import android.content.Intent;
 import android.graphics.Bitmap;
 import android.graphics.BitmapFactory;
+import android.net.Uri;
 import android.util.Log;
 import android.widget.RemoteViews;
 
 public class Side9WidgetProvider extends AppWidgetProvider  {
 	
+	
+	//The data needs to be static, since BroadcastReceivers (which WidgetProviders extends) are only valid during onReceive()
+	private static Side9Data usedData;
 		
 	@Override
 	public void onUpdate(Context context, AppWidgetManager appWidgetManager, int[] appWidgetIds) {
@@ -21,39 +26,57 @@
 		
 		final int N = appWidgetIds.length; //
 		
-		Log.e("onUpdate", "onUpdate:" + N);
+		Log.e("Side9Pigen", "onUpdate:" + N);
 		
 		
         try {
-        	Side9Xml dataSource = new Side9Xml();
-        	Side9Data data = dataSource.loadXml();
+        	Side9Data newData = Side9Xml.loadXml();
         	
+        	if (! newData.equals(usedData)) {
+        		
+        		
+        		Log.e("Side9Pigen", "(Re)loading image:" + newData.url);
+	        		        	
+		        URL imgUrl = new URL( newData.url );	        
+		        
+		        URLConnection conn = imgUrl.openConnection();
+		        Bitmap image = BitmapFactory.decodeStream( conn.getInputStream() );
+		        
+		        
+		        usedData = newData; // if we made it to here without exceptions, save the new data
+		        
+		
+		        // Perform this loop procedure for each App Widget that belongs to this provider
+		        for (int i=0; i<N; i++) {
+		            int appWidgetId = appWidgetIds[i];
+		            		            
+		            RemoteViews views = new RemoteViews(context.getPackageName(), R.layout.side9widget);
+		            		            
+		            views.setImageViewBitmap(R.id.side9picture, image);
+		            
+		            // Launch a browser when user clicks on the image
+		            Intent viewIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(Side9Xml.BASEURL));
+		            viewIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
+		            PendingIntent pending = PendingIntent.getActivity(context, 0, viewIntent, Intent.FLAG_ACTIVITY_NEW_TASK);		            
+		            views.setOnClickPendingIntent(R.id.side9picture, pending);		            
+		            
+		            // Tell the AppWidgetManager to perform an update on the current App Widget
+		            appWidgetManager.updateAppWidget(appWidgetId, views);
+		        }	
+	        } //END if (data.equals)
         	
-	        URL imgUrl = new URL( data.url );
-	        
-	        URLConnection conn = imgUrl.openConnection();
-	        Bitmap image = BitmapFactory.decodeStream( conn.getInputStream() );
-	        
-	
-	        // Perform this loop procedure for each App Widget that belongs to this provider
-	        for (int i=0; i<N; i++) {
-	            int appWidgetId = appWidgetIds[i];
-	            
-	            
-	            RemoteViews views = new RemoteViews(context.getPackageName(), R.layout.side9widget);
-	            
-	            
-	            views.setImageViewBitmap(R.id.side9picture, image);
-	            
-	            
-	            // Tell the AppWidgetManager to perform an update on the current App Widget
-	            appWidgetManager.updateAppWidget(appWidgetId, views);
-	            
-	
-	        }
         } catch (Exception e) {
-        	Log.e("UPDATE", "UPDATE", e);
+        	Log.e("Side9Pigen", "update failed", e);
         }
 	}
 
+	
+	//Called when the last widget is removed/disabled
+	@Override
+	public void onDisabled(Context context) {
+		super.onDisabled(context);
+		
+		usedData = null; //free memory
+	}
+
 }