From: Koosha Khajeh Moogahi <koosha.khajeh@gmail.com>
Date: Tue, 4 Sep 2012 17:16:45 +0000 (+0200)
Subject: Bug 783866: Add support to call Bugzilla::Milestone->check({ id => $id })
X-Git-Tag: bugzilla-4.4rc1~74
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1a062f4ebf162ed403d67971c9e7422b3cdc1802;p=thirdparty%2Fbugzilla.git

Bug 783866: Add support to call Bugzilla::Milestone->check({ id => $id })
r/a=LpSolit
---

diff --git a/Bugzilla/Milestone.pm b/Bugzilla/Milestone.pm
index ef8ecbab57..caa4afcdd4 100644
--- a/Bugzilla/Milestone.pm
+++ b/Bugzilla/Milestone.pm
@@ -64,7 +64,7 @@ sub new {
     my $dbh = Bugzilla->dbh;
 
     my $product;
-    if (ref $param) {
+    if (ref $param and !defined $param->{id}) {
         $product = $param->{product};
         my $name = $param->{name};
         if (!defined $product) {
@@ -241,7 +241,9 @@ Bugzilla::Milestone - Bugzilla product milestone class.
 
     use Bugzilla::Milestone;
 
-    my $milestone = new Bugzilla::Milestone({ name => $name, product => $product });
+    my $milestone = new Bugzilla::Milestone({ name => $name, product => $product_obj });
+    my $milestone = Bugzilla::Milestone->check({ name => $name, product => $product_obj });
+    my $milestone = Bugzilla::Milestone->check({ id => $id });
 
     my $name       = $milestone->name;
     my $product_id = $milestone->product_id;
@@ -265,7 +267,7 @@ Milestone.pm represents a Product Milestone object.
 
 =over
 
-=item C<new({name => $name, product => $product})>
+=item C<< new({name => $name, product => $product}) >>
 
  Description: The constructor is used to load an existing milestone
               by passing a product object and a milestone name.
@@ -355,7 +357,7 @@ Milestone.pm represents a Product Milestone object.
 
 =over
 
-=item C<create({value => $value, product => $product, sortkey => $sortkey})>
+=item C<< create({value => $value, product => $product, sortkey => $sortkey}) >>
 
  Description: Create a new milestone for the given product.