Vote #78232
完了Allow to disable description field in tracker setting
0%
説明
Since we can create a full width layout custom field which look like a description field in upcoming 3.4.0 (#21705), I think that we will sometimes want to replace a description field with a full width layout custom field for better and specific naming of the field.
I am sure that it would be great if Redmine 3.4.0 has an ability to disable description field on TrackersController#edit page, along with #21705.
journals
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Thanks Go MAEDA for this patch. It was on my plan to create it because we need it on the following use case:
- we want to define a "Test Case" tracker with 3 fields instead of the description field: Preconditions, Steps, Expected Results.
Currently, we can define the new fields, but we aren't able to disable the description.
--------------------------------------------------------------------------------
I think this feature will make #21705 much greater.
Setting target version to 3.4.0.
--------------------------------------------------------------------------------
Go MAEDA wrote:
> [...] that it would be great if Redmine 3.4.0 has an ability to disable description field on TrackersController#edit page, along with #21705.
Although I'm not very glad with the integration of #21705, I do think that the ability to disable the description field for trackers would be a good addition to it.
I did some quick tests with your patch applied, and it seems to work without any side-effects. But this is something to look-out for by enabling the field to be disabled, as this is a long-time expectation (plugins?).
There is a test that fails with the changes applied though: @IssueTest#test_safe_attributes_names_should_not_include_disabled_field@ (source:/trunk/test/unit/issue_test.rb@16333#L806). This is however expected as we've turned the description field to be 'disablable'. I'll leave an updated, fixed (SVN) patch against source:/trunk@16333 which adds the removal of the test assertion.
--------------------------------------------------------------------------------
Mischa The Evil wrote:
> There is a test that fails with the changes applied though: @IssueTest#test_safe_attributes_names_should_not_include_disabled_field@ (source:/trunk/test/unit/issue_test.rb@16333#L806). This is however expected as we've turned the description field to be 'disablable'. I'll leave an updated, fixed (SVN) patch against source:/trunk@16333 which adds the removal of the test assertion.
Thanks, I had missed it.
--------------------------------------------------------------------------------
Committed, thanks to both of you.
--------------------------------------------------------------------------------
related_issues
relates,Closed,15918,Managing "description" field
relates,Closed,21705,Option for long text custom fields to be displayed using full width