Merge lp:~rharding/launchpad/bp_default_1062207 into lp:launchpad

[Richard Harding]

Add initial value date for the information_type and default to PUBLIC vs None

= Summary =

The information type input widget was not getting a default value so when you
submitted the form you had an error that the field was not set.

This branch forces a default initial value.

== Implementation Notes ==

If there was no initial value from the context, the code set information type
to None. This caused the input to not have a set value on render. This changes
that None value to PUBLIC since it's the usual default.

It adds a test for all of the test cases extending the base that checks this
default value is set. It also overrides for the one test case where PUBLIC is
not a valid information type.

There's a drive by lint and setting of this default on the product
registration as well.

Note: I've filed bug #1065161 about the issue that the UX issues presented by
the choice edit widget that complicated this issue.

== Q/A ==

Per the bug, going to the +addspec page and submitting a new spec should work
without changing the information type field at all. It should get a default
value of PUBLIC.

== Tests ==

browser/tests/test_specification.py

[Richard Harding (rharding)]

Per chat with Aaron in irc. We should be using the getDefaultSpecificationInformationType of the various pillars vs hard coding the default to being InformationType.PUBLIC.

[Aaron Bentley (abentley)]

At first, I thought that this would always set the value to PUBLIC, but it only does so if the context is not a Product, or the Product's default is PUBLIC. So this is a good change.

review: Approve

=== modified file 'lib/lp/blueprints/browser/specification.py'
--- lib/lp/blueprints/browser/specification.py	2012-09-28 19:59:35 +0000
+++ lib/lp/blueprints/browser/specification.py	2012-10-11 12:38:28 +0000
@@ -102,7 +102,10 @@
     DateTimeFormatterAPI,
     format_link,
     )
-from lp.app.enums import PUBLIC_PROPRIETARY_INFORMATION_TYPES
+from lp.app.enums import (
+    InformationType,
+    PUBLIC_PROPRIETARY_INFORMATION_TYPES,
+    )
 from lp.app.utilities import json_dump_information_types
 from lp.app.vocabularies import InformationTypeVocabulary
 from lp.app.widgets.itemswidgets import LaunchpadRadioWidgetWithDescription
@@ -309,7 +312,7 @@
     @property
     def initial_values(self):
         """Set initial values to honor sharing policy default value."""
-        information_type = None
+        information_type = InformationType.PUBLIC
         if (IProduct.providedBy(self.context) or
             IProductSeries.providedBy(self.context)):
             information_type = (

=== modified file 'lib/lp/blueprints/browser/tests/test_specification.py'
--- lib/lp/blueprints/browser/tests/test_specification.py	2012-10-10 16:56:03 +0000
+++ lib/lp/blueprints/browser/tests/test_specification.py	2012-10-11 12:38:28 +0000
@@ -352,6 +352,14 @@
         self.assertIsNot(None, info_data)
         self.assertEqual(self.expected_keys, set(info_data.keys()))
 
+    def test_default_info_type(self):
+        # The default selected information type needs to be PUBLIC for new
+        # specifications.
+        view = self.createInitializedView()
+        self.assertEqual(
+            InformationType.PUBLIC,
+            view.initial_values['information_type'])
+
 
 class TestNewSpecificationFromRootView(TestCaseWithFactory,
                                        NewSpecificationTests):
@@ -396,6 +404,14 @@
             specification_sharing_policy=policy)
         return create_initialized_view(product, '+addspec')
 
+    def test_default_info_type(self):
+        # In this case the default info type cannot be PUBlIC as it's not
+        # among the allowed types.
+        view = self.createInitializedView()
+        self.assertEqual(
+            InformationType.EMBARGOED,
+            view.initial_values['information_type'])
+
 
 class TestNewSpecificationFromDistributionView(TestCaseWithFactory,
                                                NewSpecificationTests):

=== modified file 'lib/lp/registry/browser/product.py'
--- lib/lp/registry/browser/product.py	2012-10-11 04:21:07 +0000
+++ lib/lp/registry/browser/product.py	2012-10-11 12:38:28 +0000
@@ -1962,6 +1962,7 @@
             'driver': self.user.name,
             'bug_supervisor': self.user.name,
             'owner': self.user.name,
+            'information_type': InformationType.PUBLIC,
         }
 
     def setUpFields(self):