Closed
Bug 749739
Opened 12 years ago
Closed 12 years ago
New email sound notification file cannot contain spaces (Linux - Thunderbird 12)
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
RESOLVED
FIXED
mozilla15
Tracking | Status | |
---|---|---|
firefox14 | --- | fixed |
People
(Reporter: pligdas, Assigned: wolfiR)
References
Details
Attachments
(1 file, 1 obsolete file)
1.27 KB,
patch
|
karlt
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux i686; rv:11.0) Gecko/20100101 Firefox/11.0 Build ID: 20120313130328 Steps to reproduce: After update from v. 11 to 12.0 I cannot hear sound notification (.wav) If their filenames contain spaces. The "problem" is fixed if I rename the sound filo it doesn't contain spaces (My Sound.wav -> MySound.wav). I am using the Mailbox Alert 0.16.1 addon.
Comment 1•12 years ago
|
||
(In reply to Pascalis Ligdas from comment #0) You have to report this to the add-on developer, this is not a bug in Thunderbird.
Reporter | ||
Comment 2•12 years ago
|
||
Reproduce in Thunderbird: Edit > Preferences > General then check "Play a sound" and "Use the following sound file" and enter any wav file with spaces. Note 1: Sound files with spaces play just fine in v.11 Note 2: This does not depend on the addon as I thought originally.
Comment 3•12 years ago
|
||
(In reply to Pascalis Ligdas from comment #2) Confirmed here. Just after the upgrade, sounds don't play upon receiving messages. This is shown in the Error console when playing sounds from preferences: Error: uncaught exception: [Exception... "Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsISound.play]" nsresult: "0x80040111 (NS_ERROR_NOT_AVAILABLE)" location: "JS frame :: chrome://messenger/content/preferences/general.js :: <TOP_LEVEL> :: line 94" data: no] User Agent: Mozilla/5.0 (X11; Linux i686; rv:12.0) Gecko/20120420 Thunderbird/12.0 Application Build ID: 20120420153403
Comment 4•12 years ago
|
||
(In reply to Pascalis Ligdas from comment #2) Bug 635918 was filed to fix inability to play sounds. Recent Thunderbird update contains fix for it. Previously I used the SoundPlus add-on but when upgrade to V.12 happened, the add-on did not work (the add-on needs to be updated).
Assignee | ||
Comment 5•12 years ago
|
||
I can confirm that issue that a wav file with whitespace is not played. I do not see the error in the error console though. I do not know if that is a regression caused by the changes in nsSound though as the reporter used an addon before apparently. As nsSound is core I'll move the report.
Status: UNCONFIRMED → NEW
Component: General → Widget: Gtk
Ever confirmed: true
Product: Thunderbird → Core
QA Contact: general → gtk
Version: 12 → Trunk
Assignee | ||
Comment 6•12 years ago
|
||
Chris, I guess canberra does not like %20 as a whitespace?
Comment 7•12 years ago
|
||
Sounds like QueryInterface to nsIFileURL or perhaps g_filename_from_uri would be useful.
Blocks: 635918
Assignee | ||
Comment 9•12 years ago
|
||
use g_filename_from_uri() to get full path from spec
Attachment #622167 -
Flags: review?(karlt)
Comment 10•12 years ago
|
||
Comment on attachment 622167 [details] [diff] [review] patch g_filename_from_uri returns a newly allocated string that needs to be freed. It can return NULL on error so I think that should be checked. Can you rename path to spec, please?
Attachment #622167 -
Flags: review?(karlt) → review-
Assignee | ||
Comment 11•12 years ago
|
||
null check and free path I wasn't sure about the error ID (NS_ERROR_FILE_UNRECOGNIZED_PATH). Probably that choice doesn't match exactly. Should I just use a generic (NS_ERROR_FAILURE)?
Attachment #622167 -
Attachment is obsolete: true
Attachment #622292 -
Flags: review?(karlt)
Comment 12•12 years ago
|
||
Comment on attachment 622292 [details] [diff] [review] patch #2 > I wasn't sure about the error ID (NS_ERROR_FILE_UNRECOGNIZED_PATH). These error codes don't seem well defined, so I think this is fine as is.
Attachment #622292 -
Flags: review?(karlt) → review+
Assignee | ||
Comment 13•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/7944b9f57213
Target Milestone: --- → mozilla15
Comment 14•12 years ago
|
||
Comment on attachment 622292 [details] [diff] [review] patch #2 [Approval Request Comment] Regression caused by (bug #): User impact if declined: Won't ear notifications Testing completed (on m-c, etc.): yes Risk to taking this patch (and alternatives if risky):None String changes made by this patch: None
Attachment #622292 -
Flags: approval-mozilla-aurora?
Comment 15•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7944b9f57213
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Attachment #622292 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 16•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/56cc5a3c4316
status-firefox14:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•