search

NaturalGIS / naturalgis_geotag_and_import_photos

A QGIS plugin to tag and geo-tag photos (and import them as point layer)
GNU General Public License v2.0
9 stars 4 forks source link

#6: Remove print statements, replace with QgsMessageLog #14

Closed jdugge closed 6 years ago

jdugge commented 6 years ago

Removing print statements and replacing them with log messages where appropriate avoids an overflow when the console is not open.

gioman commented 6 years ago

@jdugge thanks for the patch? have you tested it > is it safe to commit it and push a new version of the plugin? I have no time at the moment to test the patch.

jdugge commented 6 years ago

I'll test it in more detail tonight and let you know.

jdugge commented 6 years ago

I tested the patch and it works well on my system (imports 300+ photos, error messages written to the log instead of printed on the console). I would say it's safe to merge.

Thanks for this very useful plugin!

gioman commented 6 years ago

I tested the patch and it works well on my system (imports 300+ photos, error messages written to the log instead of printed on the console). I would say it's safe to merge.

Thanks for this very useful plugin!

@jdugge thanks for the patch!