Bug 340030

Summary: Plugins can not be deactivated on non-English locales [patch]
Product: [Applications] digikam Reporter: Michael G. Hansen <mike>
Component: Setup-PluginsAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: amair.sob, caulier.gilles, kmm
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: http://commits.kde.org/libkipi/195bc2f26864cb79852a20cc6d1346ba236cff63 Version Fixed In: 4.5.0
Sentry Crash Report:
Attachments: Patch for libkipi from Maik Qualmann

Description Michael G. Hansen 2014-10-16 19:07:24 UTC 
I received the attached patch from Maik Qualmann and submit it to the bugtracker because I do not have a system at hand to test the patch myself.

Description of the bug: Deactivated plugins are registered to digikamrc under the translated name, but are searched for under the original name, thus the deactivation setting is never applied.


Reproducible: Always
Comment 1 Michael G. Hansen 2014-10-16 19:08:20 UTC 
Created attachment 89165 [details]
Patch for libkipi from Maik Qualmann
Comment 2 Michael G. Hansen 2014-10-16 19:09:19 UTC 
Gilles,

can you test and apply the patch?

Best,

Michael
Comment 3 caulier.gilles 2014-10-16 21:28:54 UTC 
Git commit 195bc2f26864cb79852a20cc6d1346ba236cff63 by Gilles Caulier.
Committed on 16/10/2014 at 21:27.
Pushed by cgilles into branch 'master'.

Apply patch #89165 to be able to turn on/off plugin from kipi host when application language is not English
Bump ABI/API id as library fingerprint as changed
FIXED-IN: 4.5.0

M  +14   -14   CMakeLists.txt
M  +2    -2    libkipi/configwidget.cpp
M  +1    -1    libkipi/configwidget.h
M  +1    -1    libkipi/configwidget_p.cpp
M  +6    -1    libkipi/pluginloader.cpp
M  +2    -1    libkipi/pluginloader.h

http://commits.kde.org/libkipi/195bc2f26864cb79852a20cc6d1346ba236cff63
Comment 4 km 2014-11-28 22:22:18 UTC 
still doesn't work for me in 4.5.
Comment 5 caulier.gilles 2014-11-28 22:27:46 UTC 
You need to update libkipi through KDE update, not digiKam

Gilles Caulier
Comment 6 Andreas Mair 2014-12-12 14:12:30 UTC 
(In reply to Gilles Caulier from comment #5)
> You need to update libkipi through KDE update, not digiKam
> 
> Gilles Caulier

Hi,

I've updated to KDE 4.14.3 and digikam 4.5 now but it still doesn't work.
Which KDE version do I need?

Best regards,
Andreas
Comment 7 Andreas Mair 2014-12-15 11:49:17 UTC 
OK, I'll answer myself:

I've taken a look at the sources and Gilles' commit didn't make it into KDE 4.14.3. Let's hope it will be in 4.14.4...

Best regards,
Andreas
Comment 8 Andreas Mair 2014-12-16 08:21:59 UTC 
It's me again...

I've now looked into libkipi's GIT sources and the patch isn't included in KDE/4.14 branch.
I don't know if KDE 4.14.4 is planned at all, but in case it is, I'd suggest the patch has to be included in the KDE/4.14 branch.

Best regards,
Andreas
Comment 9 caulier.gilles 2014-12-16 08:35:55 UTC 
As the patch add a new method and break ABI, this cannot be backport as well.

Patch still in git/master.

Gilles Caulier