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/wizards/BirtWizardUtil.java
{
		if ( map == null )
			map = new HashMap( );
		
		IConfigurationElement[] elements = findConfigurationElementsByExtension( BIRT_RESOURCES_EXTENSION_POINT );
		if ( elements == null || elements.length <= 0 )
			return map;
		for ( int i = 0; i < elements.length; i++ )
		{
			if ( !EXT_WEBAPP.equalsIgnoreCase( elements[i].getName( ) ) )
				continue;
			IConfigurationElement[] contextParams = elements[i]
					.getChildren( EXT_CONTEXT_PARAM );
			IConfigurationElement[] listeners = elements[i]
					.getChildren( EXT_LISTENER );
			IConfigurationElement[] servlets = elements[i]
					.getChildren( EXT_SERVLET );
			IConfigurationElement[] servletMappings = elements[i]
					.getChildren( EXT_SERVLET_MAPPING );
			IConfigurationElement[] taglibs = elements[i]
					.getChildren( EXT_TAGLIB );
			
			if ( contextParams != null )
			{
				Map son = (Map) map.get( EXT_CONTEXT_PARAM );
				if ( son == null )
					son = new HashMap( );
				for ( int j = 0; j < contextParams.length; j++ )
				{
					String name = contextParams[j].getAttribute( "name" );