Show More
@@ -397,19 +397,20 b' class SettingsController(BaseController)' | |||||
397 | settings_model = IssueTrackerSettingsModel() |
|
397 | settings_model = IssueTrackerSettingsModel() | |
398 |
|
398 | |||
399 | form = IssueTrackerPatternsForm()().to_python(request.POST) |
|
399 | form = IssueTrackerPatternsForm()().to_python(request.POST) | |
400 | for uid in form['delete_patterns']: |
|
400 | if form: | |
401 | settings_model.delete_entries(uid) |
|
401 | for uid in form.get('delete_patterns', []): | |
|
402 | settings_model.delete_entries(uid) | |||
402 |
|
403 | |||
403 |
for pattern in form |
|
404 | for pattern in form.get('patterns', []): | |
404 | for setting, value, type_ in pattern: |
|
405 | for setting, value, type_ in pattern: | |
405 | sett = settings_model.create_or_update_setting( |
|
406 | sett = settings_model.create_or_update_setting( | |
406 | setting, value, type_) |
|
407 | setting, value, type_) | |
407 | Session().add(sett) |
|
408 | Session().add(sett) | |
408 |
|
409 | |||
409 | Session().commit() |
|
410 | Session().commit() | |
410 |
|
411 | |||
411 | SettingsModel().invalidate_settings_cache() |
|
412 | SettingsModel().invalidate_settings_cache() | |
412 | h.flash(_('Updated issue tracker entries'), category='success') |
|
413 | h.flash(_('Updated issue tracker entries'), category='success') | |
413 | return redirect(url('admin_settings_issuetracker')) |
|
414 | return redirect(url('admin_settings_issuetracker')) | |
414 |
|
415 | |||
415 | @HasPermissionAllDecorator('hg.admin') |
|
416 | @HasPermissionAllDecorator('hg.admin') |
@@ -494,6 +494,14 b' class TestAdminSettingsIssueTracker:' | |||||
494 | response = self.app.get(url('admin_settings_issuetracker')) |
|
494 | response = self.app.get(url('admin_settings_issuetracker')) | |
495 | assert response.status_code == 200 |
|
495 | assert response.status_code == 200 | |
496 |
|
496 | |||
|
497 | def test_add_empty_issuetracker_pattern( | |||
|
498 | self, request, autologin_user, csrf_token): | |||
|
499 | post_url = url('admin_settings_issuetracker_save') | |||
|
500 | post_data = { | |||
|
501 | 'csrf_token': csrf_token | |||
|
502 | } | |||
|
503 | self.app.post(post_url, post_data, status=302) | |||
|
504 | ||||
497 | def test_add_issuetracker_pattern( |
|
505 | def test_add_issuetracker_pattern( | |
498 | self, request, autologin_user, csrf_token): |
|
506 | self, request, autologin_user, csrf_token): | |
499 | pattern = 'issuetracker_pat' |
|
507 | pattern = 'issuetracker_pat' |
General Comments 0
You need to be logged in to leave comments.
Login now