diff --git a/pagure/lib/model.py b/pagure/lib/model.py index d23f426..853f723 100644 --- a/pagure/lib/model.py +++ b/pagure/lib/model.py @@ -626,7 +626,6 @@ class Project(BASE): "fedmsg_notifications": True, "stomp_notifications": True, "pull_request_access_only": False, - "roadmap_on_issues_page": False, "notify_on_pull-request_flag": False, "notify_on_commit_flag": False, "issue_tracker_read_only": False, @@ -642,6 +641,10 @@ class Project(BASE): current[key] = int(current[key]) elif is_true(current[key]): current[key] = True + # Update the current dict, removing the old keys + for key in sorted(current): + if key not in default: + del current[key] return current else: return default diff --git a/pagure/templates/repo_master.html b/pagure/templates/repo_master.html index 7f9e5e2..1f09789 100644 --- a/pagure/templates/repo_master.html +++ b/pagure/templates/repo_master.html @@ -160,7 +160,7 @@ {% endif %}
- @@ -260,16 +260,11 @@ + namespace=repo.namespace) }}"> Issues  diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py index d96af01..6977406 100644 --- a/tests/test_pagure_flask_ui_issues.py +++ b/tests/test_pagure_flask_ui_issues.py @@ -625,43 +625,6 @@ class PagureFlaskIssuestests(tests.Modeltests): @patch('pagure.lib.git.update_git') @patch('pagure.lib.notify.send_email') - def test_view_issues_roadmap_view(self, p_send_email, p_ugt): - """ Test the view_issues endpoint when view is set as roadmap view """ - p_send_email.return_value = True - p_ugt.return_value = True - - tests.create_projects(self.session) - tests.create_projects_git( - os.path.join(self.path, 'repos'), bare=True) - - # Change settings to show roadmap on issue page - repo = pagure.lib.get_authorized_project(self.session, 'test') - old_settings = repo.settings - old_settings['roadmap_on_issues_page'] = True - repo.settings = old_settings - - self.session.add(repo) - self.session.commit() - - # check on landing page of project for the URL that on the Issues tab - output = self.app.get('/test/') - self.assertEqual(output.status_code, 200) - self.assertIn('