1032883 - update FlagDefaultRequestee extension to use object hooks

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[:glob ✱]

the FlagDefaultRequestee extension doesn't use object hooks to add the default_requestee field, which results in a large number of redundant database queries.

we need to update it to use object_* hooks.

Comment 1

[:glob ✱]

Attachment: 1032883_1.patch

Comment 2

[David Lawrence [:dkl]]

Comment on attachment 8448796 [details] [diff] [review]
1032883_1.patch

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

Fixes on commit. r=dkl

::: extensions/FlagDefaultRequestee/Extension.pm
@@ +25,4 @@
>  
>  sub install_update_db {
>      my $dbh = Bugzilla->dbh;
> +    dbh->bz_add_column('flagtypes', 'default_requestee', {

s/dbh/$dbh/

Can't locate object method "bz_add_column" via package "dbh" (perhaps you forgot to load "dbh"?) at ./extensions/FlagDefaultRequestee/Extension.pm line 28.

@@ +153,5 @@
> +
> +    $changes->{default_requestee} = [
> +        $old_id ? $old_id : undef,
> +        $new_id ? $new_id : undef
> +    ];

Get rid of this unless you are going to alter template/en/default/messages.html.tmpl to show the old and new values. If yes, then change to user names instead of ids.

Comment 3

[:glob ✱]

(In reply to David Lawrence [:dkl] from comment #2)
> Get rid of this unless you are going to alter
> template/en/default/messages.html.tmpl to show the old and new values. If
> yes, then change to user names instead of ids.

nice catch; i opted to fix the 'updated' messages.

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   b2af966..a6b6a99  master -> master