diff options
| author | Frédéric Buclin <LpSolit@gmail.com> | 2011-08-17 13:05:31 +0200 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2011-08-17 13:05:31 +0200 |
| commit | 589632e9d5d35c25c932aafa164feb92c26f5e3d (patch) | |
| tree | a91df977bac1818f8a70fb7e758576a294a7221e /Bugzilla/Component.pm | |
| parent | 664920c7dd532d2243538b1fd3705a72e8d496e6 (diff) | |
| download | bugzilla-589632e9d5d35c25c932aafa164feb92c26f5e3d.tar.gz bugzilla-589632e9d5d35c25c932aafa164feb92c26f5e3d.tar.xz | |
Bug 661476: sanitycheck.pl should check if all products have components defined.
Also, creating a new product from the web UI asks you to create a component too.
r=glob a=LpSolit
Diffstat (limited to 'Bugzilla/Component.pm')
| -rw-r--r-- | Bugzilla/Component.pm | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/Bugzilla/Component.pm b/Bugzilla/Component.pm index 215119715..ceca7e794 100644 --- a/Bugzilla/Component.pm +++ b/Bugzilla/Component.pm @@ -155,6 +155,11 @@ sub remove_from_db { $dbh->bz_start_transaction(); + # Products must have at least one component. + if (scalar(@{$self->product->components}) == 1) { + ThrowUserError('component_is_last', { comp => $self }); + } + if ($self->bug_count) { if (Bugzilla->params->{'allowbugdeletion'}) { require Bugzilla::Bug; |