[Marble-bugs] [marble] [Bug 331467] New: Complete KmlPhotoOverlayWriter

[Dennis Nienhüser]

https://bugs.kde.org/show_bug.cgi?id=331467

            Bug ID: 331467
           Summary: Complete KmlPhotoOverlayWriter
    Classification: Unclassified
           Product: marble
           Version: 1.7 (KDE 4.12)
          Platform: unspecified
                OS: Linux
            Status: UNCONFIRMED
          Keywords: google-code-in, junior-jobs
          Severity: task
          Priority: NOR
         Component: general
          Assignee: marble-bugs at kde.org
          Reporter: earthwings at gentoo.org

Extend KmlPhotoOverlayWriter.cpp to also write the following elements (and
their children):
- rotation
- ViewVolume
- ImagePyramid
- shape

See https://developers.google.com/kml/documentation/kmlreference#photooverlay
for the correct syntax. All tags and their children can be written in
KmlPhotoOverlayWriter::writeMid(). There's no need to add new TagWriter classes
because these elements are unique to PhotoOverlay.

Make sure to compile Marble with the following cmake options enabled:
BUILD_MARBLE_TESTS=TRUE, BUILD_MARBLE_TOOLS=TRUE. To test your implementation,
move the inline KML file defined in TestPhotoOverlay::simpleParseTest() to a
new file tests/data/PhotoOverlayInFolder.kml. Add this file to
TestGeoDataWriter. Execute tests/TestGeoDataWriter from the build folder to
execute the unit test and make sure it finishes successfully. Use the tool
kml2kml in the build folder to find errors in your implementation if the test
fails. Once everything works, remove TestPhotoOverlay.

Add another .kml file PhotoOverlayAsRoot.kml where the PhotoOverlay tag is a
direct child of the kml tag (not inside a folder). Add it to TestGeoDataWriter
as well and make sure the test works as well.

Submit a review request at https://git.reviewboard.kde.org with the marble
groups as reviewers.

Please leave a comment here (e.g. "I want to work on this task.") if you start
working on this task to avoid multiple people working on it at the same time.

-- 
You are receiving this mail because:
You are the assignee for the bug.