You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by tv...@apache.org on 2013/05/08 00:28:12 UTC

[34/50] [abbrv] git commit: [#6159] Changes default ticket permissions to allow Authenticated CREATE

[#6159] Changes default ticket permissions to allow Authenticated CREATE


Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/fae74792
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/fae74792
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/fae74792

Branch: refs/heads/tv/3854
Commit: fae74792795f9877821c26e33b46e52e195f072f
Parents: eca53d7
Author: Chris Tsai <ct...@users.sf.net>
Authored: Thu Apr 25 20:59:18 2013 +0000
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Mon May 6 15:15:31 2013 +0000

----------------------------------------------------------------------
 .../forgetracker/tests/test_tracker_roles.py       |    2 +-
 .../forgetracker/tests/unit/test_ticket_model.py   |    5 +++--
 ForgeTracker/forgetracker/tracker_main.py          |    2 +-
 3 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/fae74792/ForgeTracker/forgetracker/tests/test_tracker_roles.py
----------------------------------------------------------------------
diff --git a/ForgeTracker/forgetracker/tests/test_tracker_roles.py b/ForgeTracker/forgetracker/tests/test_tracker_roles.py
index c5f0d14..791268d 100644
--- a/ForgeTracker/forgetracker/tests/test_tracker_roles.py
+++ b/ForgeTracker/forgetracker/tests/test_tracker_roles.py
@@ -41,7 +41,7 @@ def test_role_assignments():
     assert check_access('configure') == (True, False, False)
     assert check_access('read') == (True, True, True)
     assert check_access('update') == (True, False, False)
-    assert check_access('create') == (True, False, False)
+    assert check_access('create') == (True, True, False)
     assert check_access('unmoderated_post') == (True, True, False)
     assert check_access('post') == (True, True, False)
     assert check_access('moderate') == (True, False, False)

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/fae74792/ForgeTracker/forgetracker/tests/unit/test_ticket_model.py
----------------------------------------------------------------------
diff --git a/ForgeTracker/forgetracker/tests/unit/test_ticket_model.py b/ForgeTracker/forgetracker/tests/unit/test_ticket_model.py
index 84d5a6e..9cca6df 100644
--- a/ForgeTracker/forgetracker/tests/unit/test_ticket_model.py
+++ b/ForgeTracker/forgetracker/tests/unit/test_ticket_model.py
@@ -109,6 +109,7 @@ class TestTicketModel(TrackerTestWithModel):
                         ACE.allow(role_developer, 'delete'),
                         ACE.allow(role_creator, 'read'),
                         ACE.allow(role_creator, 'post'),
+                        ACE.allow(role_creator, 'create'),
                         ACE.allow(role_creator, 'unmoderated_post'),
                         DENY_ALL])
         assert has_access(t, 'read', user=admin)()
@@ -117,7 +118,7 @@ class TestTicketModel(TrackerTestWithModel):
         assert has_access(t, 'read', user=creator)()
         assert has_access(t, 'post', user=creator)()
         assert has_access(t, 'unmoderated_post', user=creator)()
-        assert not has_access(t, 'create', user=creator)()
+        assert has_access(t, 'create', user=creator)()
         assert not has_access(t, 'update', user=creator)()
         assert has_access(t, 'read', user=developer)()
         assert has_access(t, 'create', user=developer)()
@@ -139,7 +140,7 @@ class TestTicketModel(TrackerTestWithModel):
         assert has_access(t, 'update', user=developer)()
         assert has_access(t, 'read', user=creator)()
         assert has_access(t, 'unmoderated_post', user=creator)()
-        assert not has_access(t, 'create', user=creator)()
+        assert has_access(t, 'create', user=creator)()
         assert not has_access(t, 'update', user=creator)()
         assert has_access(t, 'read', user=observer)()
         assert has_access(t, 'read', user=anon)()

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/fae74792/ForgeTracker/forgetracker/tracker_main.py
----------------------------------------------------------------------
diff --git a/ForgeTracker/forgetracker/tracker_main.py b/ForgeTracker/forgetracker/tracker_main.py
index 0f266aa..3f64528 100644
--- a/ForgeTracker/forgetracker/tracker_main.py
+++ b/ForgeTracker/forgetracker/tracker_main.py
@@ -333,8 +333,8 @@ class ForgeTrackerApp(Application):
             M.ACE.allow(role_anon, 'read'),
             M.ACE.allow(role_auth, 'post'),
             M.ACE.allow(role_auth, 'unmoderated_post'),
+            M.ACE.allow(role_auth, 'create'),
             M.ACE.allow(role_developer, 'update'),
-            M.ACE.allow(role_developer, 'create'),
             M.ACE.allow(role_developer, 'moderate'),
             M.ACE.allow(role_developer, 'save_searches'),
             M.ACE.allow(role_developer, 'delete'),