Document styles are not inherited

fixes #312

src/GUI/waypointitem.cpp

@@ -99,18 +99,26 @@ void WaypointItem::updateCache()
 	QPainterPath p;
 	qreal pointSize = _font.bold() ? HS(_size) : _size;
 
-	if (_showLabel) {
+	if (_showLabel && !_waypoint.name().isEmpty()) {
 		QFontMetrics fm(_font);
 		_labelBB = fm.tightBoundingRect(_waypoint.name());
 
 		if (_showIcon && _icon) {
 			if (_font.bold())
-				p.addRect(-_icon->width() * 0.625, -_icon->height() * 1.25,
+				p.addRect(-_icon->width() * 0.625, _waypoint.icon().isNull()
+				  ? -_icon->height() * 1.25 : -_icon->height() * 0.625,
 				  _icon->width() * 1.25, _icon->height() * 1.25);
 			else
-				p.addRect(-_icon->width()/2.0, -_icon->height(), _icon->width(),
+				p.addRect(-_icon->width()/2.0, _waypoint.icon().isNull()
+				  ? -_icon->height() : -_icon->height()/2, _icon->width(),
 				  _icon->height());
-			p.addRect(0, 0, _labelBB.width(), _labelBB.height() + fm.descent());
+
+			if (_waypoint.icon().isNull())
+				p.addRect(0, 0, _labelBB.width(), _labelBB.height()
+				  + fm.descent());
+			else
+				p.addRect(_icon->width()/2, _icon->height()/2, _labelBB.width(),
+				  _labelBB.height() + fm.descent());
 		} else {
 			p.addRect(-pointSize/2, -pointSize/2, pointSize, pointSize);
 			p.addRect(pointSize/2, pointSize/2, _labelBB.width(),
@@ -119,10 +127,12 @@ void WaypointItem::updateCache()
 	} else {
 		if (_showIcon && _icon) {
 			if (_font.bold())
-				p.addRect(-_icon->width() * 0.625, -_icon->height() * 1.25,
+				p.addRect(-_icon->width() * 0.625, _waypoint.icon().isNull()
+				  ? -_icon->height() * 1.25 : -_icon->height() * 0.625,
 				  _icon->width() * 1.25, _icon->height() * 1.25);
 			else
-				p.addRect(-_icon->width()/2, -_icon->height(), _icon->width(),
+				p.addRect(-_icon->width()/2, _waypoint.icon().isNull()
+				  ? -_icon->height() : -_icon->height()/2, _icon->width(),
 				  _icon->height());
 		} else
 			p.addRect(-pointSize/2, -pointSize/2, pointSize, pointSize);
@@ -140,12 +150,16 @@ void WaypointItem::paint(QPainter *painter,
 
 	painter->setPen(_color);
 
-	if (_showLabel) {
+	if (_showLabel && !_waypoint.name().isEmpty()) {
 		painter->setFont(_font);
-		if (_showIcon && _icon)
-			painter->drawText(-qMax(_labelBB.x(), 0), _labelBB.height(),
-			  _waypoint.name());
-		else
+		if (_showIcon && _icon) {
+			if (_waypoint.icon().isNull())
+				painter->drawText(-qMax(_labelBB.x(), 0), _labelBB.height(),
+				  _waypoint.name());
+			else
+				painter->drawText(_icon->width()/2 - qMax(_labelBB.x(), 0),
+				  _icon->height()/2 + _labelBB.height(), _waypoint.name());
+		} else
 			painter->drawText(pointSize/2 - qMax(_labelBB.x(), 0), pointSize/2
 			  + _labelBB.height(), _waypoint.name());
 	}
@@ -153,11 +167,13 @@ void WaypointItem::paint(QPainter *painter,
 	painter->setBrush(QBrush(_color, Qt::SolidPattern));
 	if (_showIcon && _icon) {
 		if (_font.bold())
-			painter->drawPixmap(-_icon->width() * 0.625, -_icon->height() * 1.25,
+			painter->drawPixmap(-_icon->width() * 0.625, _waypoint.icon().isNull()
+			  ? -_icon->height() * 1.25 : -_icon->height() * 0.625,
 			  _icon->scaled(_icon->width() * 1.25, _icon->height() * 1.25,
 			  Qt::IgnoreAspectRatio, Qt::SmoothTransformation));
 		else
-			painter->drawPixmap(-_icon->width()/2.0, -_icon->height(), *_icon);
+			painter->drawPixmap(-_icon->width()/2.0, _waypoint.icon().isNull()
+			  ? -_icon->height() : -_icon->height()/2, *_icon);
 	} else
 		painter->drawEllipse(-pointSize/2, -pointSize/2, pointSize, pointSize);
 

src/data/data.cpp

@@ -53,6 +53,7 @@ static QMultiMap<QString, Parser*> parsers()
 	map.insert("gpx", &gpx);
 	map.insert("tcx", &tcx);
 	map.insert("kml", &kml);
+	map.insert("kmz", &kml);
 	map.insert("fit", &fit);
 	map.insert("csv", &csv);
 	map.insert("igc", &igc);
@@ -162,7 +163,7 @@ QString Data::formats()
 	  + qApp->translate("Data", "IGC files") + " (*.igc);;"
 	  + qApp->translate("Data", "ITN files") + " (*.itn);;"
 	  + qApp->translate("Data", "JPEG images") + " (*.jpg *.jpeg);;"
-	  + qApp->translate("Data", "KML files") + " (*.kml);;"
+	  + qApp->translate("Data", "KML files") + " (*.kml *.kmz);;"
 	  + qApp->translate("Data", "LOC files") + " (*.loc);;"
 	  + qApp->translate("Data", "NMEA files") + " (*.nmea);;"
 	  + qApp->translate("Data", "ONmove files") + " (*.omd *.ghp);;"

src/data/kmlparser.cpp

@@ -1,5 +1,28 @@
+/*
+	WARNING: This code uses internal Qt API - the QZipReader class for reading
+	ZIP files - and things may break if Qt changes the API. For Qt5 this is not
+	a problem as we can "see the future" now and there are no changes in all
+	the supported Qt5 versions up to the last one (5.15). In Qt6 the class
+	might change or even disappear in the future, but this is very unlikely
+	as there were no changes for several years and The Qt Company's policy
+	is: "do not invest any resources into any desktop related stuff unless
+	absolutely necessary". There is an issue (QTBUG-3897) since the year 2009 to
+	include the ZIP reader into the public API, which aptly illustrates the
+	effort The Qt Company is willing to make about anything desktop related...
+*/
+
+#include <QFileInfo>
+#include <QDir>
+#include <QTemporaryDir>
+#include <private/qzipreader_p.h>
 #include "kmlparser.h"
 
+#define ZIP_MAGIC 0x04034b50
+
+static inline uint readUInt(const uchar *data)
+{
+	return (data[0]) + (data[1]<<8) + (data[2]<<16) + (data[3]<<24);
+}
 
 qreal KMLParser::number()
 {
@@ -490,9 +513,9 @@ void KMLParser::multiGeometry(QList<TrackData> &tracks, QList<Area> &areas,
 }
 
 void KMLParser::placemark(QList<TrackData> &tracks, QList<Area> &areas,
-  QVector<Waypoint> &waypoints)
+  QVector<Waypoint> &waypoints, const QMap<QString, QPixmap> &icons)
 {
-	QString name, desc, phone, address;
+	QString name, desc, phone, address, id;
 	QDateTime timestamp;
 
 	while (_reader.readNextStartElement()) {
@@ -516,6 +539,8 @@ void KMLParser::placemark(QList<TrackData> &tracks, QList<Area> &areas,
 			w.setTimestamp(timestamp);
 			w.setAddress(address);
 			w.setPhone(phone);
+			if (!id.isNull())
+				w.setIcon(icons.value(id));
 			point(w);
 		} else if (_reader.name() == QLatin1String("LineString")
 		  || _reader.name() == QLatin1String("LinearRing")) {
@@ -544,49 +569,98 @@ void KMLParser::placemark(QList<TrackData> &tracks, QList<Area> &areas,
 			a.setName(name);
 			a.setDescription(desc);
 			polygon(a);
+		} else if (_reader.name() == QLatin1String("styleUrl")) {
+			id = _reader.readElementText();
+			id = (id.at(0) == '#') ? id.right(id.size() - 1) : QString();
 		} else
 			_reader.skipCurrentElement();
 	}
 }
 
-void KMLParser::folder(QList<TrackData> &tracks, QList<Area> &areas,
-  QVector<Waypoint> &waypoints)
+void KMLParser::icon(const QDir &dir, const QString &id,
+  QMap<QString, QPixmap> &icons)
+{
+	QString path;
+
+	while (_reader.readNextStartElement()) {
+		if (_reader.name() == QLatin1String("href"))
+			path = _reader.readElementText();
+		else
+			_reader.skipCurrentElement();
+	}
+
+	if (!path.isNull())
+		icons.insert(id, QPixmap(dir.absoluteFilePath(path)));
+}
+
+void KMLParser::iconStyle(const QDir &dir, const QString &id,
+  QMap<QString, QPixmap> &icons)
 {
 	while (_reader.readNextStartElement()) {
-		if (_reader.name() == QLatin1String("Placemark"))
-			placemark(tracks, areas, waypoints);
-		else if (_reader.name() == QLatin1String("Folder"))
-			folder(tracks, areas, waypoints);
+		if (_reader.name() == QLatin1String("Icon"))
+			icon(dir, id, icons);
 		else
 			_reader.skipCurrentElement();
 	}
 }
 
-void KMLParser::document(QList<TrackData> &tracks, QList<Area> &areas,
-  QVector<Waypoint> &waypoints)
+void KMLParser::style(const QDir &dir, QMap<QString, QPixmap> &icons)
 {
+	QString id = _reader.attributes().value("id").toString();
+
 	while (_reader.readNextStartElement()) {
-		if (_reader.name() == QLatin1String("Document"))
-			document(tracks, areas, waypoints);
-		else if (_reader.name() == QLatin1String("Placemark"))
-			placemark(tracks, areas, waypoints);
-		else if (_reader.name() == QLatin1String("Folder"))
-			folder(tracks, areas, waypoints);
+		if (_reader.name() == QLatin1String("IconStyle"))
+			iconStyle(dir, id, icons);
 		else
 			_reader.skipCurrentElement();
 	}
 }
 
-void KMLParser::kml(QList<TrackData> &tracks, QList<Area> &areas,
-  QVector<Waypoint> &waypoints)
+void KMLParser::folder(const QDir &dir, QList<TrackData> &tracks,
+  QList<Area> &areas, QVector<Waypoint> &waypoints,
+  const QMap<QString, QPixmap> &icons)
 {
 	while (_reader.readNextStartElement()) {
 		if (_reader.name() == QLatin1String("Document"))
-			document(tracks, areas, waypoints);
+			document(dir, tracks, areas, waypoints);
 		else if (_reader.name() == QLatin1String("Placemark"))
-			placemark(tracks, areas, waypoints);
+			placemark(tracks, areas, waypoints, icons);
 		else if (_reader.name() == QLatin1String("Folder"))
-			folder(tracks, areas, waypoints);
+			folder(dir, tracks, areas, waypoints, icons);
+		else
+			_reader.skipCurrentElement();
+	}
+}
+
+void KMLParser::document(const QDir &dir, QList<TrackData> &tracks,
+  QList<Area> &areas, QVector<Waypoint> &waypoints)
+{
+	QMap<QString, QPixmap> icons;
+
+	while (_reader.readNextStartElement()) {
+		if (_reader.name() == QLatin1String("Document"))
+			document(dir, tracks, areas, waypoints);
+		else if (_reader.name() == QLatin1String("Placemark"))
+			placemark(tracks, areas, waypoints, icons);
+		else if (_reader.name() == QLatin1String("Folder"))
+			folder(dir, tracks, areas, waypoints, icons);
+		else if (_reader.name() == QLatin1String("Style"))
+			style(dir, icons);
+		else
+			_reader.skipCurrentElement();
+	}
+}
+
+void KMLParser::kml(const QDir &dir, QList<TrackData> &tracks,
+  QList<Area> &areas, QVector<Waypoint> &waypoints)
+{
+	while (_reader.readNextStartElement()) {
+		if (_reader.name() == QLatin1String("Document"))
+			document(dir, tracks, areas, waypoints);
+		else if (_reader.name() == QLatin1String("Placemark"))
+			placemark(tracks, areas, waypoints, QMap<QString, QPixmap>());
+		else if (_reader.name() == QLatin1String("Folder"))
+			folder(dir, tracks, areas, waypoints, QMap<QString, QPixmap>());
 		else
 			_reader.skipCurrentElement();
 	}
@@ -596,15 +670,49 @@ bool KMLParser::parse(QFile *file, QList<TrackData> &tracks,
   QList<RouteData> &routes, QList<Area> &areas, QVector<Waypoint> &waypoints)
 {
 	Q_UNUSED(routes);
+	uchar magic[4];
+	QFileInfo fi(*file);
 
 	_reader.clear();
-	_reader.setDevice(file);
 
-	if (_reader.readNextStartElement()) {
-		if (_reader.name() == QLatin1String("kml"))
-			kml(tracks, areas, waypoints);
-		else
-			_reader.raiseError("Not a KML file");
+	if (file->read((char *)magic, 4) == 4 && readUInt(magic) == ZIP_MAGIC) {
+		QZipReader zip(fi.absoluteFilePath(), QIODevice::ReadOnly);
+		QTemporaryDir tempDir;
+		if (!zip.extractAll(tempDir.path()))
+			_reader.raiseError("Error extracting ZIP file");
+		else {
+			QDir zipDir(tempDir.path());
+			QFileInfoList files(zipDir.entryInfoList(QStringList("*.kml"),
+			  QDir::Files));
+
+			if (files.isEmpty())
+				_reader.raiseError("No KML file found in ZIP file");
+			else {
+				QFile kmlFile(files.first().absoluteFilePath());
+				if (!kmlFile.open(QIODevice::ReadOnly))
+					_reader.raiseError("Error opening KML file");
+				else {
+					_reader.setDevice(&kmlFile);
+
+					if (_reader.readNextStartElement()) {
+						if (_reader.name() == QLatin1String("kml"))
+							kml(zipDir, tracks, areas, waypoints);
+						else
+							_reader.raiseError("Not a KML file");
+					}
+				}
+			}
+		}
+	} else {
+		file->reset();
+		_reader.setDevice(file);
+
+		if (_reader.readNextStartElement()) {
+			if (_reader.name() == QLatin1String("kml"))
+				kml(fi.absoluteDir(), tracks, areas, waypoints);
+			else
+				_reader.raiseError("Not a KML file");
+		}
 	}
 
 	return !_reader.error();

src/data/kmlparser.h

@@ -5,6 +5,8 @@
 #include <QDateTime>
 #include "parser.h"
 
+class QDir;
+
 class KMLParser : public Parser
 {
 public:
@@ -14,14 +16,14 @@ public:
 	int errorLine() const {return _reader.lineNumber();}
 
 private:
-	void kml(QList<TrackData> &tracks, QList<Area> &areas,
+	void kml(const QDir &dir, QList<TrackData> &tracks, QList<Area> &areas,
 	  QVector<Waypoint> &waypoints);
-	void document(QList<TrackData> &tracks, QList<Area> &areas,
-	  QVector<Waypoint> &waypoints);
-	void folder(QList<TrackData> &tracks, QList<Area> &areas,
+	void document(const QDir &dir, QList<TrackData> &tracks, QList<Area> &areas,
 	  QVector<Waypoint> &waypoints);
+	void folder(const QDir &dir, QList<TrackData> &tracks, QList<Area> &areas,
+	  QVector<Waypoint> &waypoints, const QMap<QString, QPixmap> &icons);
 	void placemark(QList<TrackData> &tracks, QList<Area> &areas,
-	  QVector<Waypoint> &waypoints);
+	  QVector<Waypoint> &waypoints, const QMap<QString, QPixmap> &icons);
 	void multiGeometry(QList<TrackData> &tracks, QList<Area> &areas,
 	  QVector<Waypoint> &waypoints, const QString &name, const QString &desc,
 	  const QDateTime &timestamp);
@@ -45,6 +47,10 @@ private:
 	QDateTime timeStamp();
 	qreal number();
 	QDateTime time();
+	void style(const QDir &dir, QMap<QString, QPixmap> &icons);
+	void iconStyle(const QDir &dir, const QString &id,
+	  QMap<QString, QPixmap> &icons);
+	void icon(const QDir &dir, const QString &id, QMap<QString, QPixmap> &icons);
 
 	QXmlStreamReader _reader;
 };