Merge pull request #4751 from bencostamagna/delete_warning

Warning on multiple deletions: added some details.
This commit is contained in:
Bryan Housel
2018-01-29 08:58:51 -05:00
committed by GitHub
3 changed files with 13 additions and 4 deletions
+11 -2
View File
@@ -4,12 +4,21 @@ import { t } from '../util/locale';
export function validationManyDeletions() {
var threshold = 100;
var validation = function(changes) {
var validation = function(changes, graph) {
var warnings = [];
var nodes=0, ways=0, areas=0, relations=0;
changes.deleted.forEach(function(c) {
if (c.type === 'node') {nodes++;}
else if (c.type === 'way' && c.geometry(graph) === 'line') {ways++;}
else if (c.type === 'way' && c.geometry(graph) === 'area') {areas++;}
else if (c.type === 'relation') {relations++;}
});
if (changes.deleted.length > threshold) {
warnings.push({
id: 'many_deletions',
message: t('validations.many_deletions', { n: changes.deleted.length })
message: t('validations.many_deletions',
{ n: changes.deleted.length, p: nodes, l: ways, a:areas, r: relations })
});
}