Closed Bug 85734 Opened 23 years ago Closed 23 years ago

new bug notification shouldn't include blank fields

Categories

(Bugzilla :: Bugzilla-General, enhancement, P2)

Product:
Bugzilla
Component:
Bugzilla-General
Platform:
x86
Windows 2000
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 2.14

People

(Reporter: timeless, Assigned: jacob)

Details

(Whiteboard: [please commit asap])

Attachments

(1 file)

patch
644 bytes, patch
Details | Diff | Splinter Review 
I just noticed this when the CC: field was added, but this applies to all 
fields.

it's a waste of bytes, and reading time to include blank fields. so let's fix 
the mailer to stop doing it.
Attached patch patch — — Splinter Review 

    
  
Assignee: tara → jake
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.16

    
    

    
  
Easy enough to do, but it's not security so it probably won't go in 2.14.  Of
course, if I get an r= sooner, there really isn't any reason it can't go in.

    
    

    
  
r=timeless
Whiteboard: [please commit asap]

    
    

    
  
Committed.
Obviously it won't show up unitl b.m.o updates to the tip again.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Target Milestone: Bugzilla 2.16 → Bugzilla 2.14

    
    

    
  
Moving to Bugzilla product
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Version: Bugzilla 2.13 → unspecified
QA Contact: matty_is_a_geek → default-qa

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: