Add warnings when improperly configured

hdgarrood / redmine_release_notes

A plugin to add release notes to Redmine

GNU General Public License v3.0

73 stars 47 forks source link

Add warnings when improperly configured #71

Open hdgarrood opened 11 years ago

hdgarrood commented 11 years ago

Warn on:

Issues#show (if module enabled)
ReleaseNotes#{index,generate}
Plugin settings (or perhaps, have a 'test' button?)

hdgarrood commented 11 years ago

ActionView::Template::Error (undefined method `name' for nil:NilClass):
    12:
    13: <% stats = version.release_notes_stats %>
    14: <% cf_none_value =  Setting.plugin_redmine_release_notes[:field_value_not_required] %>
    15: <% cf_name = CustomField.find_by_id(Setting.plugin_redmine_release_notes[:issue_custom_field_id]).name %>
    16:
    17: <% # Overview %>
    18:

hdgarrood commented 11 years ago

check:

custom field exists
done, todo, not required values are real values for the field

(don't need to check that the format exists because it fails gracefully)

dipanm commented 11 years ago

+1 This is very important. Actually the version page itself now fails to load if CF for "release notes status" per issue is not defined. This needs a fix.