Closed Bug 880113 Opened 8 years ago Closed 8 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: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.