Summary: | "Send by email" tries to start kmail | ||
---|---|---|---|
Product: | [Plasma] Active | Reporter: | Thomas Pfeiffer <thomas.pfeiffer> |
Component: | Share Like Connect | Assignee: | Marco Martin <notmart> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | active, fania.bremmer, javier, lamarque, sebas |
Priority: | NOR | ||
Version: | PA 2 | ||
Target Milestone: | unscheduled | ||
Platform: | Unlisted Binaries | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Thomas Pfeiffer
2011-10-05 21:14:23 UTC
Confirmed. Issue reproducible on 2011-10-27-21-59-basyskom-plasma-active-testing-meego-usb-live.iso *** Bug 287992 has been marked as a duplicate of this bug. *** I traced the problem until kdelibs/kdecore/kernel/ktoolinvocation_x11.cpp. KInvocationTool starts kmail by default if there is no mail client configured in ~/.kde/share/config/emaildefaults, which is our case. I set kmail-mobile as mail client but it does not recognise the parameters KInvocationTool passes to it. I will fix today. The fix for this will take more time than I have thought. Kmail-mobile does not support opening the composer from command line, which is how KInvocationTool launches the regular kmail. Kmail-mobile does support opening the composer using dbus, but it does not support passing an file name to be attached. I will implement attachment parameter support in kmail-mobile's dbus interface, it does not seem too difficult. We will have to drop KInvocationTool in share-like-connect's dataengine and launch kmail-mobile using dbus, which I think is a good thing. Usually that kind of change should not the commited close to a release. I will create the patch for kdepim and share-like-connect packages and try my best to not to screw up anything :-) Patch that fix this problem: https://git.reviewboard.kde.org/r/103316/ We still need to change ~/.kde/share/config/emaildefaults to configure kmail-mobile as the default mail client (kmail is the default). We need to change the line EmailClient= to EmailClient=kmail-mobile -t %t -s %s -c %c -b %b -B %B -A %A awesome. i added an emaildefaults in the config folder of plasma-mobile, should be put in the plasma-contour-config package Already fixed in 2011-12-06-18-24-basyskom-plasma-active-devel-meego-usb-live.iso Git commit 22ed26ff43c16abf6e9baff9eabdae65343dde96 by Lamarque V. Souza. Committed on 16/12/2011 at 12:57. Pushed by lvsouza into branch 'master'. Add command line parameters support to kmail-mobile. REVIEW: 8985 BUG: 283420 M +2 -2 mobile/lib/kdeclarativeapplication.cpp M +1 -1 mobile/lib/kdeclarativeapplication.h A +38 -0 mobile/mail/kmailmobileoptions.h [License: LGPL (v2.1+)] M +31 -2 mobile/mail/main.cpp M +107 -0 mobile/mail/mainview.cpp M +19 -0 mobile/mail/mainview.h http://commits.kde.org/kdepim/22ed26ff43c16abf6e9baff9eabdae65343dde96 This bug has now reappeared (tested on the current MeeGo devel image) Git commit f206fda6bcc9da9806b3701215fe000bd05c8c6f by Lamarque V. Souza. Committed on 25/05/2012 at 22:27. Pushed by lvsouza into branch 'Active/2.1'. Fix kmail-mobile configuration. (cherry picked from commit e8c270894bd2d15cc7f6afb0e7572410b477b1d5) M +3 -0 config/emaildefaults http://commits.kde.org/plasma-mobile/f206fda6bcc9da9806b3701215fe000bd05c8c6f Git commit e8c270894bd2d15cc7f6afb0e7572410b477b1d5 by Lamarque V. Souza. Committed on 25/05/2012 at 22:27. Pushed by lvsouza into branch 'master'. Fix kmail-mobile configuration. M +3 -0 config/emaildefaults http://commits.kde.org/plasma-mobile/e8c270894bd2d15cc7f6afb0e7572410b477b1d5 This commit also needs to go into the plasma-mobile-config repo, no? (And the duplication should probably be removed.) I pushed it plasma-mobile-config repo too, for some reason the git hooks did not added a comment to this bug entry. OKay, cool. Thanks! Git commit 76d6bda20bce82f5ca8127b70eda8d2454b51d9b by Aaron Seigo. Committed on 30/05/2012 at 13:48. Pushed by aseigo into branch 'master'. commit the fix in the correct repository M +3 -0 appconfig/emaildefaults http://commits.kde.org/plasma-mobile-config/76d6bda20bce82f5ca8127b70eda8d2454b51d9b (In reply to comment #15) > Git commit 76d6bda20bce82f5ca8127b70eda8d2454b51d9b by Aaron Seigo. > Committed on 30/05/2012 at 13:48. > Pushed by aseigo into branch 'master'. > > commit the fix in the correct repository > > M +3 -0 appconfig/emaildefaults > > http://commits.kde.org/plasma-mobile-config/ > 76d6bda20bce82f5ca8127b70eda8d2454b51d9b Ok, that explains why nothing showed up here when I pushed the commit to the scratch repo :) Bug is fixed again. |