Skip to content

Commit 0b1a272

Browse files
committed
Add forest to search results, remove POIs that has only wikidata.
1 parent e73799a commit 0b1a272

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

src/main/java/il/org/osm/israelhiking/PlanetSearchProfile.java

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -412,11 +412,14 @@ private boolean processOtherSourceFeature(SourceFeature feature, FeatureCollecto
412412

413413
setIconColorCategory(pointDocument, feature);
414414

415-
if (pointDocument.poiIcon == "icon-search" ||
416-
(pointDocument.poiIcon == "icon-home" && (!isInterestingPoint(pointDocument) || !feature.isPoint()))) {
415+
if (pointDocument.poiIcon == "icon-search") {
417416
return false;
418417
}
419418

419+
if (pointDocument.poiIcon == "icon-home" && !feature.isPoint()) {
420+
return true;
421+
}
422+
420423
insertPointToElasticsearch(pointDocument, docId);
421424

422425
if ((pointDocument.poiIcon == "icon-peak" || pointDocument.poiIcon == "icon-waterfall") && !isInterestingPoint(pointDocument)) {
@@ -458,6 +461,17 @@ private void addNonIconFeaturesToElasricseach(SourceFeature feature) throws Geom
458461
pointDocument.poiIconColor = "green";
459462
pointDocument.poiCategory = "Bicycle";
460463
}
464+
if (feature.hasTag("landuse", "forest")) {
465+
pointDocument.poiIcon = "icon-tree";
466+
pointDocument.poiIconColor = "#008000";
467+
pointDocument.poiCategory = "Other";
468+
}
469+
if ((feature.getString("wikidata") != null || feature.getString("wikipedia") != null)) {
470+
pointDocument.poiIconColor = "black";
471+
pointDocument.poiIcon = "icon-wikipedia-w";
472+
pointDocument.poiCategory = "Wikipedia";
473+
return;
474+
}
461475

462476
if (pointDocument.poiIcon == null) {
463477
return;
@@ -546,8 +560,7 @@ private Point getFirstPointOfLineRelation(MergedLinesHelper mergedLines) throws
546560
}
547561

548562
private boolean isInterestingPoint(PointDocument pointDocument) {
549-
return !pointDocument.description.isEmpty() ||
550-
pointDocument.wikidata != null ||
563+
return !pointDocument.description.isEmpty() ||
551564
pointDocument.image != null;
552565
}
553566

@@ -793,13 +806,6 @@ private void setIconColorCategory(PointDocument pointDocument, WithTags feature)
793806
return;
794807
}
795808

796-
if ((feature.getString("wikidata") != null || feature.getString("wikipedia") != null) &&
797-
feature.getString("highway") == null && feature.getString("boundary") == null) {
798-
pointDocument.poiIconColor = "black";
799-
pointDocument.poiIcon = "icon-wikipedia-w";
800-
pointDocument.poiCategory = "Wikipedia";
801-
return;
802-
}
803809

804810
pointDocument.poiIconColor = "black";
805811
pointDocument.poiIcon = "icon-search";

0 commit comments

Comments
 (0)