Modify

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 Don-vip, 5 years ago

Priority: minornormal

comment:2 by holgermappt, 5 years ago

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?

in reply to:  2 comment:3 by Don-vip, 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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain holgermappt.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from holgermappt to the specified user.
Next status will be 'needinfo'. The owner will be changed from holgermappt to overflorian.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from holgermappt to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.