The default bug view has changed. See this FAQ.

backport new date custom field type from bug 801664 to bmo/4.2

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
General
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

Production
x86_64
Linux

Details

Attachments

(2 attachments)

(Assignee)

Description

4 years ago
backport new date custom field type from bug 801664 to bmo/4.2 to give a choice to create fields without a time portion.

Comment 1

4 years ago
Thanks.  It would be great if we could change the field type.  I was very excited to have a "due date" field but the current field type adds a lot of extra noise to my dashboard and it's information I do not need, personally. I'm not aware of anyone who relies on the time-stamp either.  Most projects seem to be guided by a Date vs. specific Time and if the time is relevant then I think it's worth adding that in a different field vs. appending it to the Date field. Technically Date and Time (hour, minute, second) are different data.  Right now everything shows up with a 00:00:00 appended to it and it's not helpful :(
(Assignee)

Comment 2

4 years ago
Created attachment 744715 [details] [diff] [review]
Patch to add date only custom field type (v1)

Please verify proper creation/removal of the custom field in you tests. I had timeout issues a few times when doing so on my local system.

Thanks
Attachment #744715 - Flags: review?(glob)
(Assignee)

Updated

4 years ago
Blocks: 868167
Comment on attachment 744715 [details] [diff] [review]
Patch to add date only custom field type (v1)

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

r=glob

i had no issues during my testing.

please update contrib/addcustomfield.pl before committing.
Attachment #744715 - Flags: review?(glob) → review+
(Assignee)

Comment 4

4 years ago
Thanks

Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2           
modified importxml.pl
modified Bugzilla/Bug.pm                                                                
modified Bugzilla/Constants.pm
modified Bugzilla/Field.pm
modified Bugzilla/Migrate.pm
modified Bugzilla/Search.pm
modified Bugzilla/DB/Schema/Mysql.pm
modified Bugzilla/DB/Schema/Oracle.pm                                              
modified Bugzilla/DB/Schema/Pg.pm
modified Bugzilla/DB/Schema/Sqlite.pm
modified Bugzilla/WebService/Bug.pm
modified contrib/addcustomfield.pl
modified template/en/default/bug/field.html.tmpl
modified template/en/default/global/field-descs.none.tmpl
modified template/en/default/search/field.html.tmpl
Committed revision 8770. 

dkl
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 5

4 years ago
this is awesome. thanks all. quick question... when will I see the change when viewing bug lists? I'm still seeing the 0's (screenshot attached)

Comment 6

4 years ago
Created attachment 746091 [details]
0's attached to date
(Assignee)

Comment 7

4 years ago
this should be fixed soon. Checkin of the script was the first part. I am working with IT now to get it executed on the production db to convert the dates. Then you will not see the 0's.

dkl
You need to log in before you can comment on or make changes to this bug.