TimeAdjust has a dependency on KIPI feature "ImagesHasTime" although it doesn't have to. This prevents it from being shown in other applications than Digikam, for example in Gwenview. It would perfectly make sense for TimeAdjust to be enabled in Gwenview, too; because it is not dependent on application-provided timestamps only. It can use and/or update any other timestamps such as selected metadata, filesystem etc. Instead of declaring a dependency on ImagesHasTime, it is better to check if the current application supports it and if not, only the application timestamp related controls should be disabled (or made invisible). This means, the application timestamp should not be usable as the source and it shoud not be updateable. I created a patch that makes exactly this change and the plugin just works in Digikam, as well as in Gwenview. Reproducible: Always Steps to Reproduce: 1. Start Gwenview 2. List plugins 3. Go to Images > Actual Results: Adjust Time & Date plugin is not shown Expected Results: The plugin should be shown, but with invisible options to use/update the nonexistent "Gwenview timestamp"
Created attachment 77377 [details] Patch enabling TimeAdjust in Gwenview
Smit, this patch is for you... Gilles
Git commit 7ce28b03dd7e579e58fa59f2b9ba0f56995283b2 by Smit Mehta. Committed on 22/02/2013 at 10:53. Pushed by smitmehta into branch 'master'. Applied patch for making TimeAdjust plugin work in Gwenview as well. Credits for patch : Karol Slanina M +0 -1 timeadjust/kipiplugin_timeadjust.desktop.cmake M +1 -0 timeadjust/plugin_timeadjust.cpp M +12 -3 timeadjust/settingswidget.cpp M +1 -0 timeadjust/settingswidget.h M +5 -0 timeadjust/timeadjustdialog.cpp M +1 -0 timeadjust/timeadjustdialog.h http://commits.kde.org/kipi-plugins/7ce28b03dd7e579e58fa59f2b9ba0f56995283b2