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
/**
	 * Prepare file list from zip file
	 * 
	 * @param structure
	 * @param entry
	 * @param list
	 * @return
	 */
	private static List prepareFileList( ZipFileStructureProvider structure,
			ZipEntry entry, List list )
	{
		if ( structure == null || entry == null )
			return null;
		if ( list == null )
		{
			list = new ArrayList( );
		}
		
		List son = structure.getChildren( entry );
		if ( son == null )
			return list;
		
		Iterator it = son.iterator( );
		while ( it.hasNext( ) )
		{
			ZipEntry temp = (ZipEntry) it.next( );
			if ( temp.isDirectory( ) )
			{
				prepareFileList( structure, temp, list );
			}