Closed Bug 880113 Opened 10 years ago Closed 10 years ago

Use the cache for the product and component name

Categories

(Bugzilla :: Bugzilla-General, enhancement)

Product:
Bugzilla
Component:
Bugzilla-General
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 5.0

People

(Reporter: mail, Assigned: mail)

Details

Attachments

(1 file)

v1 patch
925 bytes, patch
LpSolit
: review+
Details | Diff | Splinter Review 
Since Bugzilla 5.0 uses the cache to store product and components, it makes sense the the product and component subroutines use this cache in Bugzilla::Bug.
Attached patch v1 patch — — Splinter Review 

        Attachment #758947 -
        Flags: review?(LpSolit)

    
    

    
  
Comment on attachment 758947 [details] [diff] [review]
v1 patch

>+    ($self->{component}) //= $self->component_obj->name;

>+    ($self->{product}) //= $self->product_obj->name;

Parens are useless. Please remove them on checkin. r=LpSolit

        Attachment #758947 -
        Flags: review?(LpSolit) → review+

    
  
Severity: normal → enhancement
Flags: approval?

    
  
Flags: approval? → approval+

    
    

    
  
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified Bugzilla/Bug.pm
Committed revision 8666.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED

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