Add description to type defaults

app/contracts/types/base_contract.rb

@@ -43,6 +43,7 @@ module Types
     attribute :is_default
     attribute :color_id
     attribute :project_ids
+    attribute :description
     attribute :attribute_groups
 
     validate :validate_current_user_is_admin

app/models/permitted_params.rb

@@ -606,6 +606,8 @@ class PermittedParams
           :is_milestone,
           :is_default,
           :color_id,
+          :default,
+          :description,
           project_ids: []
         ],
         user: %i(

app/views/types/form/_settings.html.erb

@@ -55,6 +55,14 @@ See docs/COPYRIGHT.rdoc for more details.
                               container_class: '-slim') %>
       </div>
     <% end %>
+
+    <div class="form--field">
+      <%= f.text_area :description,
+                      class: 'wiki-edit wiki-toolbar',
+                      container_class: '-xxwide',
+                      with_text_formatting: true %>
+    </div>
+
     <!--[eoform:type]-->
 
   </div>

config/locales/en.yml

@@ -435,6 +435,7 @@ en:
         spent_on: "Date"
         type: "Type"
       type:
+        description: "Default text for description"
         attribute_groups: ''
         is_in_roadmap: "Displayed in roadmap by default"
         is_default: "Activated for new projects by default"

db/migrate/20190509071101_add_default_description_to_types.rb

@@ -0,0 +1,5 @@
+class AddDefaultDescriptionToTypes < ActiveRecord::Migration[5.2]
+  def change
+    add_column :types, :description, :text
+  end
+end

frontend/src/app/components/work-packages/wp-single-view/wp-single-view.html

@@ -40,7 +40,7 @@
       <div class="attributes-group--header-container"></div>
     </div>
     <div>
-      <p [hidden]="projectContext.href" [textContent]="text.project.required"></p>
+      <p class="wp-project-context--warning" [hidden]="projectContext.href" [textContent]="text.project.required"></p>
       <div class="attributes-key-value"
            [ngClass]="{'-span-all-columns': descriptor.spanAll }"
            *ngFor="let descriptor of projectContext.field; trackBy:trackByName">

frontend/src/app/components/wp-edit-form/work-package-changeset.ts

@@ -87,6 +87,16 @@ export class WorkPackageChangeset {
     this.resetForm();
   }
 
+  /**
+   * Remove some of the changes by key
+   * @param changes
+   */
+  public clearSome(...changes:string[]) {
+    changes.forEach((key) => {
+      delete this.changes[key];
+    });
+  }
+
   public resetForm() {
     this.form = null;
   }
@@ -156,6 +166,7 @@ export class WorkPackageChangeset {
         .update(payload)
         .then((form:FormResource) => {
           this.form = form;
+          this.workPackage.$initialize({ ...form.payload.$source, id: this.workPackage.id });
 
           this.buildResource();
 
@@ -359,7 +370,7 @@ export class WorkPackageChangeset {
       return;
     }
 
-    let payload:any = this.workPackage.$source;
+    let payload:any = this.workPackage.$plain();
 
     const resource = this.halResourceService.createHalResourceOfType('WorkPackage', this.mergeWithPayload(payload));
 

frontend/src/app/modules/fields/edit/edit-field.component.ts

@@ -60,7 +60,7 @@ export class EditFieldComponent extends Field implements OnInit, OnDestroy {
   public self = this;
 
   /** JQuery accessor to element ref */
-  protected $element:JQuery<HTMLElement>;
+  protected $element:JQuery;
 
   constructor(readonly I18n:I18nService,
               readonly elementRef:ElementRef,

frontend/src/app/modules/fields/edit/field-types/formattable-edit-field.component.ts

@@ -131,6 +131,8 @@ export class FormattableEditFieldComponent extends EditFieldComponent implements
   }
 
   public reset() {
+    this.changeset.clearSome(this.name);
+
     if (this.instance) {
       this.instance.content = this.rawValue;
     }
@@ -155,4 +157,19 @@ export class FormattableEditFieldComponent extends EditFieldComponent implements
   public get isFormattable() {
     return true;
   }
+
+  protected initialize() {
+    if (this.resource.isNew && this.instance) {
+      this.instance
+        .getTransformedContent()
+        .then((val) => {
+
+          // Reset CKEditor when reloading after type/form changes
+          if (!val) {
+            this.reset();
+          }
+          this.rawValue = val;
+        });
+    }
+  }
 }

lib/api/v3/work_packages/work_package_representer.rb

@@ -324,7 +324,14 @@ module API
         property :description,
                  exec_context: :decorator,
                  getter: ->(*) {
-                   ::API::Decorators::Formattable.new(represented.description, object: represented)
+                   type = represented&.type
+                   description = represented&.description
+
+                   if !description.present? && type&.description
+                     ::API::Decorators::Formattable.new(type.description, object: represented)
+                   else
+                     ::API::Decorators::Formattable.new(description, object: represented)
+                   end
                  },
                  setter: ->(fragment:, **) {
                    represented.description = fragment['raw']

spec/factories/type_factory.rb

@@ -30,6 +30,7 @@ FactoryBot.define do
   factory :type do
     sequence(:position) { |p| p }
     name { |a| "Type No. #{a.position}" }
+    description { nil }
     created_at { Time.now }
     updated_at { Time.now }
 

spec/features/work_packages/new/work_package_default_description_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+require 'support/work_packages/work_package_field'
+require 'features/work_packages/work_packages_page'
+require 'features/page_objects/notification'
+
+describe 'new work package', js: true do
+  let(:type_task) { FactoryBot.create(:type_task, description: "# New Task template\n\nHello there") }
+  let!(:status) { FactoryBot.create(:status, is_default: true) }
+  let!(:priority) { FactoryBot.create(:priority, is_default: true) }
+  let!(:project) do
+    FactoryBot.create(:project, types: [type_task])
+  end
+
+  let(:user) { FactoryBot.create :admin }
+
+  let(:subject_field) { wp_page.edit_field :subject }
+  let(:description_field) { wp_page.edit_field :description }
+  let(:project_field) { wp_page.edit_field :project }
+  let(:type_field) { wp_page.edit_field :type }
+  let(:notification) { PageObjects::Notifications.new(page) }
+  let(:wp_page) { Pages::FullWorkPackageCreate.new }
+
+  before do
+    login_as(user)
+  end
+
+  describe 'global work package create' do
+    it 'shows the template after selection of project and type' do
+      visit '/work_packages/new'
+      wp_page.expect_fully_loaded
+
+      project_field.openSelectField
+      project_field.set_value project
+
+      subject_field.set_value 'Foobar!'
+
+      # Wait until project is set
+      expect(page).to have_no_selector('.wp-project-context--warning')
+
+      type_field.openSelectField
+      type_field.set_value type_task
+
+      scroll_to_and_click find('#work-packages--edit-actions-save')
+      wp_page.expect_notification message: 'Successful creation.'
+
+      expect(page).to have_selector('.wp-edit-field--display-field.description h1', text: 'New Task template')
+    end
+  end
+
+  describe 'project work package create' do
+    let(:wp_table) { Pages::WorkPackagesTable.new project }
+    let(:wp_page) { Pages::SplitWorkPackageCreate.new project: project }
+
+    it 'shows the template after selection of project and type' do
+      wp_table.visit!
+      wp_table.create_wp_split_screen type_task
+
+      wp_page.expect_fully_loaded
+
+      subject_field.set_value 'Foobar!'
+
+      scroll_to_and_click find('#work-packages--edit-actions-save')
+      wp_page.expect_notification message: 'Successful creation.'
+
+      expect(page).to have_selector('.wp-edit-field--display-field.description h1', text: 'New Task template')
+    end
+  end
+end