From: Koosha Khajeh Moogahi <koosha.khajeh@gmail.com>
Date: Tue, 24 Jul 2012 16:00:32 +0000 (-0400)
Subject: Bug 520434 - Inconsistent naming of time fields (creation_time vs time)
X-Git-Tag: bugzilla-4.3.2~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=72ce8ddb3ef34d31237f5969dbea07652bd4c5cb;p=thirdparty%2Fbugzilla.git

Bug 520434 - Inconsistent naming of time fields (creation_time vs time)
r=dkl, a=LpSolit
---

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index bf6a162161..82bfff4450 100644
--- a/Bugzilla/WebService/Bug.pm
+++ b/Bugzilla/WebService/Bug.pm
@@ -297,6 +297,7 @@ sub _translate_comment {
         creator    => $self->type('string', $comment->author->login),
         author     => $self->type('string', $comment->author->login),
         time       => $self->type('dateTime', $comment->creation_ts),
+        creation_time => $self->type('dateTime', $comment->creation_ts),
         is_private => $self->type('boolean', $comment->is_private),
         text       => $self->type('string', $comment->body_full),
         attachment_id => $self->type('int', $attach_id),
@@ -1613,6 +1614,13 @@ Bugzillas. (However, this backwards-compatibility will go away in Bugzilla
 
 C<dateTime> The time (in Bugzilla's timezone) that the comment was added.
 
+=item creation_time
+
+C<dateTime> This is exactly same as the C<time> key. Use this field instead of
+C<time> for consistency with other methods including L</get> and L</attachments>.
+For compatibility, C<time> is still usable. However, please note that C<time>
+may be deprecated and removed in a future release.
+
 =item is_private
 
 C<boolean> True if this comment is private (only visible to a certain
@@ -1654,6 +1662,8 @@ C<creator>.
 
 =item C<count> was added to the return value in Bugzilla B<4.4>.
 
+=item C<creation_time> was added in Bugzilla B<4.4>.
+
 =back
 
 =back