Merge pull request #138 from CartoDB/mmorillo/ch79584/popup-error-wit…

…h-cartodb-id-column

Prevent adding the column carotbd_id as aggregation column

packages/viz/src/lib/layer/Layer.ts

@@ -16,6 +16,8 @@ import {
 } from './LayerInteractivity';
 import { LayerOptions } from './LayerOptions';
 
+const DEFAULT_ID_PROPERTY = 'cartodb_id';
+
 export class Layer extends WithEvents implements StyledLayer {
   private _source: Source;
   private _style: Style;
@@ -414,7 +416,8 @@ export class Layer extends WithEvents implements StyledLayer {
         {
           column: this._style.field,
           sample: true,
-          aggregation: true
+          // prevent aggregating by the id column
+          aggregation: this._style.field !== DEFAULT_ID_PROPERTY
         }
       ];
     }
@@ -427,7 +430,8 @@ export class Layer extends WithEvents implements StyledLayer {
         const field = {
           column,
           sample: false,
-          aggregation: true
+          // prevent aggregating by the id column
+          aggregation: column !== DEFAULT_ID_PROPERTY
         };
         this._fields.push(field);
       });