Last Comment Bug 787729 - Folder name with % makes feeds unhappy
: Folder name with % makes feeds unhappy
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Feed Reader (show other bugs)
: unspecified
: All All
: -- normal (vote)
: Thunderbird 18.0
Assigned To: alta88
:
:
Mentors:
Depends on:
Blocks: 124287
  Show dependency treegraph
 
Reported: 2012-09-01 14:46 PDT by alta88
Modified: 2012-09-11 07:11 PDT (History)
2 users (show)
ryanvm: in‑testsuite?
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch (1.01 KB, patch)
2012-09-01 14:53 PDT, alta88
mconley: review+
Details | Diff | Splinter Review

Description alta88 2012-09-01 14:46:16 PDT

    
Comment 1 alta88 2012-09-01 14:53:06 PDT
Created attachment 657604 [details] [diff] [review]
patch

the name is already decoded, doing it again throws on %..
Comment 2 Mike Conley (:mconley) 2012-09-09 03:45:52 PDT
Comment on attachment 657604 [details] [diff] [review]
patch

Review of attachment 657604 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks alta88!
Comment 3 Ryan VanderMeulen [:RyanVM] 2012-09-10 15:37:55 PDT
https://hg.mozilla.org/comm-central/rev/6494076610ad

Should this have a test?
Comment 4 Mike Conley (:mconley) 2012-09-11 07:11:52 PDT
Oh yes - a regression test would be very nice. alta88?

Note You need to log in before you can comment on or make changes to this bug.