Review Request 112950: Insert the set shortcuts into the internal hash of actions and shortcuts in KGlobalAccel

Commit Hook null at kde.org
Wed Oct 9 04:35:24 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112950/#review41418
-----------------------------------------------------------


This review has been submitted with commit 63866d5cebef2ccceaa642e45dfd624ba95446f5 by Kevin Ottens to branch frameworks.

- Commit Hook


On Sept. 26, 2013, 8:16 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112950/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2013, 8:16 p.m.)
> 
> 
> Review request for KDE Frameworks and Valentin Rusu.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> When calling setShortcut or setDefaultShortcut the new list of shortcuts is not added to the action(Default)Shortcuts map and so the getter methods return an empty shortcut for the given action. This can also be seen when executing the KGlobalShortcutTest::testSetShortcut() testcase against the kde4 kglobalacceld (on the CI it's just skipping this test!). With this patch the testcase succeeds.
> 
> 
> Diffs
> -----
> 
>   staging/xmlgui/src/kglobalaccel.cpp f5bcfca 
> 
> Diff: http://git.reviewboard.kde.org/r/112950/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131009/2614f142/attachment.html>


More information about the Kde-frameworks-devel mailing list