Attachment 36365 Details for Bug 132029 – update ui patch for download-size and install-size

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] update ui patch for download-size and install-size

org.eclipse.update.ui_132029.patch (text/plain), 1.87 KB, created by Doug MacDonald

on 2006-03-15 16:02:52 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Doug MacDonald

Created: 2006-03-15 16:02:52 EST

Size: 1.87 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.update.ui
>Index: src/org/eclipse/update/internal/ui/wizards/TargetPage.java
>===================================================================
>RCS file: /home/eclipse/org.eclipse.update.ui/src/org/eclipse/update/internal/ui/wizards/TargetPage.java,v
>retrieving revision 1.74
>diff -u -r1.74 TargetPage.java
>--- src/org/eclipse/update/internal/ui/wizards/TargetPage.java	21 Nov 2005 21:39:24 -0000	1.74
>+++ src/org/eclipse/update/internal/ui/wizards/TargetPage.java	15 Mar 2006 21:03:16 -0000
>@@ -250,6 +250,8 @@
> 		File file = new File(site.getSite().getURL().getFile());
> 		long available = LocalSystemInfo.getFreeSpace(file);
> 		long required = computeRequiredSizeFor(site);
>+		//add the download size to space required to do operation since all plugins and nonplugin data will be downloaded first
>+		required += computeDownloadSizeFor(site); 
> 		if (required <= 0)
> 			requiredSpaceLabel.setText(UpdateUIMessages.InstallWizard_TargetPage_unknownSize); 
> 		else
>@@ -261,6 +263,11 @@
> 		else
> 			availableSpaceLabel.setText(
> 				NLS.bind(UpdateUIMessages.InstallWizard_TargetPage_size, "" + available)); //$NON-NLS-1$
>+		if(required > available){
>+			this.setPageComplete(false);
>+			//TODO: set error message: "...not enough space..."
>+		}else
>+			this.setPageComplete(true);
> 	}
> 
> 	private long computeRequiredSizeFor(IConfiguredSite site) {
>@@ -275,6 +282,19 @@
> 		}
> 		return totalSize;
> 	}
>+	
>+	private long computeDownloadSizeFor(IConfiguredSite site) {
>+		long totalSize = 0;
>+		for (int i = 0; i < jobs.length; i++) {
>+			if (site.equals(jobs[i].getTargetSite())) {
>+				long jobSize = site.getSite().getDownloadSizeFor(jobs[i].getFeature());
>+				if (jobSize == -1)
>+					return -1;
>+				totalSize += jobSize;
>+			}
>+		}
>+		return totalSize;
>+	}
> 
> 	private void pageChanged() {
> 		boolean empty = false;

Actions: View | Diff

Attachments on bug 132029: 36364 | 36365 | 38848 | 39636 | 39706