Last Comment Bug 883527 - Don't store empty message for empty feed <item>
: Don't store empty message for empty feed <item>
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Feed Reader (show other bugs)
: unspecified
: All All
: -- normal (vote)
: Thunderbird 24.0
Assigned To: alta88
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-15 07:32 PDT by alta88
Modified: 2013-06-25 05:23 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch (8.38 KB, patch)
2013-06-15 08:26 PDT, alta88
mkmelin+mozilla: review+
Details | Diff | Splinter Review

Description alta88 2013-06-15 07:32:11 PDT

    
Comment 1 alta88 2013-06-15 08:26:11 PDT
Created attachment 763111 [details] [diff] [review]
patch


1. fix empty <item> tag, empty message store problem.
2. log actual (new) messages stored.
Comment 2 Magnus Melin 2013-06-15 11:23:11 PDT
Comment on attachment 763111 [details] [diff] [review]
patch

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

Looks reasonable to me. r=mkmelin
Comment 3 Ryan VanderMeulen [:RyanVM] 2013-06-18 08:39:59 PDT
https://hg.mozilla.org/comm-central/rev/d8bde53f98da

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