Opened 5 years ago
Last modified 5 years ago
#18021 new enhancement
Merge photo_geotagging and photoadjust
Reported by: | overflorian | Owned by: | holgermappt |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Plugin photoadjust | Version: | |
Keywords: | Cc: |
Description
Hi,
Every time I edit photos, I need both the geotagging and the photoadjust pluggin. I never used one without the other. These 2 pluggins seems to me to be really complementary.
Why not merge them into a single one? I suggest to keep photoadjust name and embed the geotagging features into it. After all, to geotag is on of the ways to adjust a picture so it makes perfect sense.
Thanks in advance guys
Attachments (0)
Change History (3)
comment:1 by , 5 years ago
Priority: | minor → normal |
---|
follow-up: 3 comment:2 by , 5 years ago
comment:3 by , 5 years ago
Replying to holgermappt:
Is there a procedure to retire a plugin that was absorbed by other components?
Yes:
1) delete the jar file of the old plugin in plugins/dist
2) svn mv the old plugins source directory from plugins/<oldplugin> to oldplugins/<oldplugin>
3) update PluginHandler.DEPRECATED_PLUGINS
: add
new DeprecatedPlugin("<oldplugin>", tr("no longer required"))
at the end
I tried to contact Paul Hartmann, the maintainer of the photo_geotagging plugin, but got no reply. We had the discussion whether it should be one or two plugins when I started the photoadjust pluggin. Paul was fine with both options, I decided to put my stuff in a separate plugin.
My plan is to work on the open photo_geotagging tickets first and do the merge of the two plugins as a second step. I have no preference which name to keep, so I might just flip a coin.
Is there a procedure to retire a plugin that was absorbed by other components?