Bug 331701

Summary:	multiple toobar buttons for the same tag
Product:	[Applications] kontact	Reporter:	luisfe <luisfe>
Component:	mail	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	montel
Priority:	NOR
Version:	4.13 Pre
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:	http://commits.kde.org/kdepim/1966d2fadd2267ba7b62111d917325ebd5a0f33f	Version Fixed In:	4.13
Sentry Crash Report:
Attachments:	kontact with mail component, fresh session kontact after switching to korganizer and back again to mail component

Description luisfe 2014-03-03 14:18:20 UTC

If I add a tag to kmail and specify to have toolbar button, then the toolbar button multiplies itself while changing the kontact component.

Reproducible: Always

Steps to Reproduce:
1. Add a tag "XXX" to kmail, tick on enable toolbar button
2. Open kontact with mail and calendar component
3. Switch from/to mail and calendar.
Actual Results:  
Each time you switch components, new, repeated, tag buttons appear in the toolbar.

Expected Results:  
Have just one button per configured tag.

Just in case it is relevant, I have set custom pictures for the tags.

Comment 1 Laurent Montel 2014-03-03 16:30:45 UTC

screenshot please

Comment 2 luisfe 2014-03-03 18:28:55 UTC

Created attachment 85400 [details]
kontact with mail component, fresh session

Comment 3 luisfe 2014-03-03 18:31:24 UTC

Created attachment 85401 [details]
kontact after switching to korganizer and back again to mail component

There where two toolbar buttons associated to tags, one with the letters AC and the EU flag. Now each button appears three times.

Comment 4 Laurent Montel 2014-03-03 20:42:52 UTC

I confirm it.
I will investigate it.

Comment 5 Laurent Montel 2014-03-03 20:57:52 UTC

Git commit 1966d2fadd2267ba7b62111d917325ebd5a0f33f by Montel Laurent.
Committed on 03/03/2014 at 20:57.
Pushed by mlaurent into branch 'KDE/4.13'.

Fix Bug 331701 - multiple toobar buttons for the same tag

FIXED-IN: 4.13

M  +3    -2    kmail/tag/tagactionmanager.cpp

http://commits.kde.org/kdepim/1966d2fadd2267ba7b62111d917325ebd5a0f33f