Aggregation is disabled in charting eclipse-birt#856

Enabled aggregation. Seems to have been a negation wrong. Also expanded
the default IChartUIHelper.useDataSetRow() implementation to a simple
startsWith("data") check previously used in the enableAddEditor()
method-

chart/org.eclipse.birt.chart.ui.extension/src/org/eclipse/birt/chart/ui/swt/wizard/data/BaseDataDefinitionComponent.java

@@ -876,7 +876,7 @@ private void enableAggEditor(String expression) {
 				ec.decode(expression);
 				expression = ec.convertJSExpression(false);
 
-				boolean enabled = this.context.getUIFactory().createUIHelper().useDataSetRow(context.getExtendedItem(),
+				boolean enabled = !this.context.getUIFactory().createUIHelper().useDataSetRow(context.getExtendedItem(),
 						expression);
 				fAggEditorComposite.setEnabled(enabled);
 			} catch (BirtException e) {

chart/org.eclipse.birt.chart.ui/src/org/eclipse/birt/chart/ui/integrate/ChartUIHelperBase.java

@@ -48,7 +48,12 @@ public boolean canCombine(IChartType type, ChartWizardContext context) {
 
 	@Override
 	public boolean useDataSetRow(Object reportItem, String expression) throws BirtException {
-		// Default implementation, do nothing
-		return false;
+		/*
+		 * Default implementation is a bit simple, this behavior is copied from
+		 * org.eclipse.birt.chart.ui.swt.wizard.data.BaseDataDefinitionComponent.
+		 * enableAggEditor( String expression ) prior to the refactoring that yielded
+		 * this method. According to the javadoc it should be more comprehensive though.
+		 */
+		return expression.startsWith("data"); //$NON-NLS-1$
 	}
 }