| @@ -92,7 +92,11 class IssueImport < Import | |||
|
|
92 | 92 | 'subject' => row_value(row, 'subject'), |
|
|
93 | 93 | 'description' => row_value(row, 'description') |
|
|
94 | 94 | } |
|
|
95 | attributes | |
|
|
95 | if status_name = row_value(row, 'status') | |
|
|
96 | if status_id = IssueStatus.named(status_name).first.try(:id) | |
|
|
97 | attributes['status_id'] = status_id | |
|
|
98 | end | |
|
|
99 | end | |
|
|
96 | 100 | issue.send :safe_attributes=, attributes, user |
|
|
97 | 101 | |
|
|
98 | 102 | attributes = {} |
| @@ -9,6 +9,10 | |||
|
|
9 | 9 | <%= mapping_select_tag @import, 'tracker', :required => true, |
|
|
10 | 10 | :values => @import.allowed_target_trackers.sorted.map {|t| [t.name, t.id]} %> |
|
|
11 | 11 | </p> |
|
|
12 | <p> | |
|
|
13 | <label><%= l(:field_status) %></label> | |
|
|
14 | <%= mapping_select_tag @import, 'status' %> | |
|
|
15 | </p> | |
|
|
12 | 16 | |
|
|
13 | 17 | <div class="splitcontent"> |
|
|
14 | 18 | <div class="splitcontentleft"> |
| @@ -1,4 +1,4 | |||
|
|
1 | priority;subject;description;start_date;due_date;parent;private;progress;custom;version;category;user;estimated_hours;tracker | |
|
|
2 | High;First;First description;2015-07-08;2015-08-25;;no;;PostgreSQL;;New category;dlopper;1;bug | |
|
|
3 | Normal;Child 1;Child description;;;1;yes;10;MySQL;2.0;New category;;2;feature request | |
|
|
4 | Normal;Child of existing issue;Child description;;;#2;no;20;;2.1;Printing;;3;bug | |
|
|
1 | priority;subject;description;start_date;due_date;parent;private;progress;custom;version;category;user;estimated_hours;tracker;status | |
|
|
2 | High;First;First description;2015-07-08;2015-08-25;;no;;PostgreSQL;;New category;dlopper;1;bug;new | |
|
|
3 | Normal;Child 1;Child description;;;1;yes;10;MySQL;2.0;New category;;2;feature request;new | |
|
|
4 | Normal;Child of existing issue;Child description;;;#2;no;20;;2.1;Printing;;3;bug;assigned | |
| @@ -89,6 +89,15 class IssueImportTest < ActiveSupport::TestCase | |||
|
|
89 | 89 | assert_include "Tracker cannot be blank", item.message |
|
|
90 | 90 | end |
|
|
91 | 91 | |
|
|
92 | def test_status_should_be_set | |
|
|
93 | import = generate_import_with_mapping | |
|
|
94 | import.mapping.merge!('status' => '14') | |
|
|
95 | import.save! | |
|
|
96 | ||
|
|
97 | issues = new_records(Issue, 3) { import.run } | |
|
|
98 | assert_equal ['New', 'New', 'Assigned'], issues.map(&:status).map(&:name) | |
|
|
99 | end | |
|
|
100 | ||
|
|
92 | 101 | def test_parent_should_be_set |
|
|
93 | 102 | import = generate_import_with_mapping |
|
|
94 | 103 | import.mapping.merge!('parent_issue_id' => '5') |
General Comments 0
You need to be logged in to leave comments.
Login now