bundles: Don't display 'Change order' button on public bundle view

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

diff --git a/apps/patchwork/tests/bundles.py b/apps/patchwork/tests/bundles.py
index 0032978e3eb69f3bf69fd9fd1f3b17778941341a..5e8b95ba92f7fafd3dcda2f04698b0c731133275 100644 (file)
--- a/apps/patchwork/tests/bundles.py
+++ b/apps/patchwork/tests/bundles.py
@@ -238,6 +238,7 @@ class BundlePublicModifyTest(BundleTestBase):
                 password = self.other_user.username)
         response = self.client.get(bundle_url(self.bundle))
         self.assertNotContains(response, 'name="form" value="bundle"')
                 password = self.other_user.username)
         response = self.client.get(bundle_url(self.bundle))
         self.assertNotContains(response, 'name="form" value="bundle"')

+        self.assertNotContains(response, 'Change order')

 
     def testBundleFormSubmission(self):
         oldname = 'oldbundlename'
 
     def testBundleFormSubmission(self):
         oldname = 'oldbundlename'

diff --git a/apps/patchwork/views/bundle.py b/apps/patchwork/views/bundle.py
index 3c89338b40ef6ec1f518eb25ca7a2a6b9186da8c..7a120dd2e74ae4855dbfd7b1db51dde8f742267e 100644 (file)
--- a/apps/patchwork/views/bundle.py
+++ b/apps/patchwork/views/bundle.py
@@ -179,7 +179,7 @@ def bundle(request, username, bundlename):
                          'bundlename': bundle.name},
             filter_settings = filter_settings,
             patches = bundle.ordered_patches(),
                          'bundlename': bundle.name},
             filter_settings = filter_settings,
             patches = bundle.ordered_patches(),

-            editable_order = True)
+            editable_order = is_owner)

 
     context['bundle'] = bundle
     context['bundleform'] = form
 
     context['bundle'] = bundle
     context['bundleform'] = form