Bug 176875 BIRT viewer - WTP integration - web.xml file created has blank values
When you create a BIRT web project, the web.xml file has lot of blank default values, which is very confusing to understand. Some users commented about it during the eclipse conference. Suggestions 1) BIRT_VIEWER_WORKING_FOLDER - set the default value 2) BIRT_VIEWER_MAX_ROWS - Write a comment above, saying by default all rows are shown, also explain the user of this variable. 3) BIRT_RESOURCE_PATH - This should specify the default value or atleast put a comment indicating where is it picked from. This is very confusing. 4) General It would be good to put a onew line comment on all.
viewer/org.eclipse.birt.integration.wtp.ui/src/org/eclipse/birt/integration/wtp/ui/internal/actions/ImportBirtRuntimeAction.java
/**
	 * Invoke selectionChanged event
	 * 
	 * @see org.eclipse.ui.IActionDelegate#selectionChanged(org.eclipse.jface.action.IAction,
	 *      org.eclipse.jface.viewers.ISelection)
	 */
	public void selectionChanged( IAction action, ISelection selection )
	{
		boolean bEnable = false;
		if ( selection instanceof IStructuredSelection )
		{
			fSelection = (IStructuredSelection) selection;
			bEnable = validateSelected( fSelection );
		}
		( (Action) action ).setEnabled( bEnable );
	}
	/**
	 * check if it is a J2EE Dynamic Web Project
	 */
	protected boolean isValidProject( IProject fProject )
	{
		return J2EEProjectUtilities.isDynamicWebProject( fProject );
	}
	/**
	 * Invoke selectionChanged method to check selected project.
	 */
	protected boolean validateSelected( ISelection selection )
	{
		if ( !( selection instanceof IStructuredSelection ) )
			return false;