Closed Bug 339379 Opened 20 years ago Closed 20 years ago

Make Bugzilla::Product use Bugzilla::Object

Categories

(Bugzilla :: Bugzilla-General, enhancement)

2.23
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 3.0

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

I'd also like to make the "check" method generic, if I can.
Attached patch v1Splinter Review
Here's a first go at it. I actually didn't test the patch, but it passes runtests and it should work fine.
Attachment #228498 - Flags: review?(bugzilla-mozilla)
Status: NEW → ASSIGNED
Attachment #228498 - Flags: review?(bugzilla-mozilla) → review?(LpSolit)
Comment on attachment 228498 [details] [diff] [review] v1 That's the kind of cleanup I love. And the way get_all() works will also fix the perf issue when having > 1000 products in your installation. r=LpSolit
Attachment #228498 - Flags: review?(LpSolit) → review+
Flags: approval?
Flags: approval? → approval+
Checking in collectstats.pl; /cvsroot/mozilla/webtools/bugzilla/collectstats.pl,v <-- collectstats.pl new revision: 1.55; previous revision: 1.54 done Checking in editflagtypes.cgi; /cvsroot/mozilla/webtools/bugzilla/editflagtypes.cgi,v <-- editflagtypes.cgi new revision: 1.41; previous revision: 1.40 done Checking in Bugzilla/Product.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Product.pm,v <-- Product.pm new revision: 1.17; previous revision: 1.16 done Checking in Bugzilla/User.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v <-- User.pm new revision: 1.119; previous revision: 1.118 done
Blocks: 322848
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: