Patch Email DeleteAll - Revision history

Lclarkjr: Fix bug in delete all causing an endless loop and include read all functionality.

2009-10-15T02:41:36Z

Fix bug in delete all causing an endless loop and include read all functionality.

Show changes

Lclarkjr at 01:59, 6 October 2009

2009-10-06T01:59:57Z

Lclarkjr: Add section headers

2009-10-06T01:59:14Z

Add section headers

Mamouton at 03:33, 21 September 2009

2009-09-21T03:33:20Z

Lclarkjr: Performance improvement. It now takes ~7mins to delete 100 emails vs ~20mins.

2009-09-19T20:56:38Z

Performance improvement. It now takes ~7mins to delete 100 emails vs ~20mins.

Hopspitfire: Moving comment to discussion page.

2009-08-14T21:13:02Z

Moving comment to discussion page.

Mamouton at 03:57, 14 August 2009

2009-08-14T03:57:02Z

Lclarkjr at 02:01, 6 August 2009

2009-08-06T02:01:35Z

Lclarkjr: New page: {{template:patch}} This will add a button at the bottom of the screen with a trashcan icon between the Compose and Refresh buttons. Selecting the button will delete all email items in the ...

2009-08-06T01:59:52Z

New page: {{template:patch}} This will add a button at the bottom of the screen with a trashcan icon between the Compose and Refresh buttons. Selecting the button will delete all email items in the ...

New page

{{template:patch}}
This will add a button at the bottom of the screen with a trashcan icon between the Compose and Refresh buttons. Selecting the button will delete all email items in the list after prompting the user for confirmation.

I am deliberately not using line numbers to minimize the need for updates as new WebOS releases come out.

Open:

<pre><nowiki>/usr/palm/applications/com.palm.app.email/app/controllers/list-assistant.js</nowiki></pre>

Find the setup function and modify the cmdMenuModel items, replace the

<pre><nowiki>{},</nowiki></pre>

with

<pre><nowiki>{label:$L('Delete All'), icon:'delete', command:'deleteall'},</nowiki></pre>

Find the handleCommand function and add the following case statement:

<pre><nowiki>
case 'deleteall':
this.handleDeleteAll();
break;

Now add the following three functions:

handleDeleteAllResponse: function (event) {
//check to see if there are more items to delete.
this.deleteAll();
},

deleteAll: function(){

var count = this.emailListElement.mojo.getLength();

var id;

if(count > 0)
{
var item = this.emailListElement.mojo.getNodeByIndex(0);

if(item !== undefined)
{
id = item.id;

if(id)
{
this.controller.serviceRequest(Email.identifier, {
method: 'setDeleted',
parameters: {'message':id, 'value': true },
onSuccess: this.handleDeleteAllResponse.bind(this),
onFailure: this.handleDeleteAllResponse.bind(this)
});
}//if(id)
}//if item !== undefined
else
{
//item was undefined probably because it is currently marked for delete
this.deleteAll();
}
}//count > 0
},

handleDeleteAll: function (event) {

var totalCount = 0;

totalCount = this.emailListElement.mojo.getLength();

this.controller.showAlertDialog({
onChoose: function(value) {
if(value == 'yes') {
//Delete all items in this folder
this.deleteAll();
}
},
title: '<b>' + $L('Delete All') + '</b>',
message: $L('Are you sure you want to delte all ') + "<b>" + totalCount + "</b>" + $L(' items in this folder?'),
choices: [
{label:$L('Yes'), value:'yes', type:'affirmative'},
{label:$L('No'), value:'no', type:'alert'}
]
});
},
</nowiki></pre>

← Older revision		Revision as of 21:13, 14 August 2009
Line 86:		Line 86:
	},		},
	</nowiki></pre>		</nowiki></pre>
−
−	~~Is there a way to select a certain number of messages and then delete those versus all those on the page?~~

← Older revision		Revision as of 03:57, 14 August 2009
Line 86:		Line 86:
	},		},
	</nowiki></pre>		</nowiki></pre>
		+
		+	Is there a way to select a certain number of messages and then delete those versus all those on the page?

@@ Line 1: / Line 1: @@
 {{template:patch}}
 ==Introduction==
 This will add a button at the bottom of the screen with a trashcan icon between the Compose and Refresh buttons. Selecting the button will delete all email items in the list after prompting the user for confirmation.

@@ Line 101: / Line 101: @@
 						},
 	                                	title: '<b>' + $L('Delete All') + '</b>',
-                       				message: $L('Are you sure you want to delte all ') + "<b>" + totalCount + "</b>" + $L(' items in this folder?'),
+                       				message: $L('Are you sure you want to delete all ') + "<b>" + totalCount + "</b>" + $L(' items in this folder?'),
                         				choices: [
                                 				{label:$L('Yes'), value:'yes', type:'affirmative'},

@@ Line 36: / Line 36: @@
    	var count = this.emailListElement.mojo.getLength();
-	var id;
+        var id;
-	if(count > 0)
+        if(count > 0)
-−
+        {
-		var item = this.emailListElement.mojo.getNodeByIndex(0);
+                var i;
-−
-		if(item !== undefined)
+                //Since the list is loaded dynamically we may have the count of all emails but not all of the data
+                //So start with count and work backward with error handling
-		   id = item.id;
+                //When reach 0 trigger at least one more pass after data has had a chance to refresh
+                for(i=count-1; i>=0; i--)
-		   if(id)
+                        var item = this.emailListElement.mojo.getNodeByIndex(i);
-			this.controller.serviceRequest(Email.identifier, {
-			        	method: 'setDeleted',
+                        if(item !== undefined)
-			                parameters: {'message':id, 'value': true },
-			                onSuccess: this.handleDeleteAllResponse.bind(this),
+                                id = item.id;
-			                onFailure: this.handleDeleteAllResponse.bind(this)
-	             			});
+                                if(id)
-	     	   }//if(id)
-		}//if item !== undefined
+                                        if(i==0)
-		else
+                                                this.controller.serviceRequest(Email.identifier, {
-			//item was undefined probably because it is currently marked for delete
+                                                        method: 'setDeleted',
-			this.deleteAll();
+                                                        parameters: {'message':id, 'value': true },
+                                                        onSuccess: this.handleDeleteAllResponse.bind(this),
-	}//count > 0
+                                                        onFailure: this.handleDeleteAllResponse.bind(this)
    },

@@ Line 22: / Line 22: @@
 	this.handleDeleteAll();
 	break;
 Now add the following three functions:
    handleDeleteAllResponse: function (event) {
          //check to see if there are more items to delete.