cartodb/Windshaft-cartodb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #1048 from CartoDB/fix-uncaught-error

Browse Source 
Prevent from uncaught exception
This commit is contained in:
Daniel G. Aubert 2018-10-23 11:54:55 +02:00 committed by GitHub
commit 2ebb1728ee
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 23 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
NEWS.md
View File

@ -12,6 +12,9 @@ New features
- `pg-mvt`: Accept trailing semicolon in input queries. - `pg-mvt`: Accept trailing semicolon in input queries.
- `Renderer Cache Entry`: Do not throw errors for integrity checks. - `Renderer Cache Entry`: Do not throw errors for integrity checks.
Bug Fixes:
- Prevent from uncaught exception: Range filter Error from camshaft when getting analysis query.
## 6.4.0 ## 6.4.0
Released 2018-09-24 Released 2018-09-24

18
lib/cartodb/models/mapconfig/adapter/analysis-mapconfig-adapter.js
View File

@ -107,14 +107,19 @@ AnalysisMapConfigAdapter.prototype.getMapConfig = function(user, requestMapConfi
return sourceId2Query; return sourceId2Query;
}, {}); }, {});
var missingNodesErrors = []; var analysesErrors = [];
requestMapConfig.layers = requestMapConfig.layers.map(function(layer, layerIndex) { requestMapConfig.layers = requestMapConfig.layers.map(function(layer, layerIndex) {
if (getLayerSourceId(layer)) { if (getLayerSourceId(layer)) {
var layerSourceId = getLayerSourceId(layer); var layerSourceId = getLayerSourceId(layer);
var layerNode = sourceId2Node[layerSourceId]; var layerNode = sourceId2Node[layerSourceId];
if (layerNode) { if (layerNode) {
var analysisSql = layerQuery(layerNode); try {
var analysisSql;
// it might throw error: Range filter from camshaft, for instance.
analysisSql = layerQuery(layerNode);
var sqlQueryWrap = layer.options.sql_wrap; var sqlQueryWrap = layer.options.sql_wrap;
if (sqlQueryWrap) { if (sqlQueryWrap) {
layer.options.sql_raw = analysisSql; layer.options.sql_raw = analysisSql;
@ -123,8 +128,11 @@ AnalysisMapConfigAdapter.prototype.getMapConfig = function(user, requestMapConfi
layer.options.sql = analysisSql; layer.options.sql = analysisSql;
layer.options.columns = getDataviewsColumns(getLayerDataviews(layer, dataviews)); layer.options.columns = getDataviewsColumns(getLayerDataviews(layer, dataviews));
layer.options.affected_tables = getAllAffectedTablesFromSourceNodes(layerNode); layer.options.affected_tables = getAllAffectedTablesFromSourceNodes(layerNode);
} catch (error) {
analysesErrors.push(error);
}
} else { } else {
missingNodesErrors.push( analysesErrors.push(
new Error('Missing analysis node.id="' + layerSourceId +'" for layer='+layerIndex) new Error('Missing analysis node.id="' + layerSourceId +'" for layer='+layerIndex)
); );
} }
@ -133,8 +141,8 @@ AnalysisMapConfigAdapter.prototype.getMapConfig = function(user, requestMapConfi
}); });
var missingDataviewsNodesErrors = getMissingDataviewsSourceIds(dataviews, sourceId2Node); var missingDataviewsNodesErrors = getMissingDataviewsSourceIds(dataviews, sourceId2Node);
if (missingNodesErrors.length > 0 || missingDataviewsNodesErrors.length > 0) { if (analysesErrors.length > 0 || missingDataviewsNodesErrors.length > 0) {
return callback(missingNodesErrors.concat(missingDataviewsNodesErrors)); return callback(analysesErrors.concat(missingDataviewsNodesErrors));
} }
// Augment dataviews with sql from analyses // Augment dataviews with sql from analyses