launchpad-reviewers team mailing list archive

[William Grant <me@xxxxxxxxxxxxxxxxxx>]

William Grant has proposed merging lp:~wgrant/launchpad/bug-1348153 into lp:launchpad.

Commit message:
DistributionSet:+add now always sets enabled_restricted_processors, as processors may nowadays be restricted even for non-virtualized archives.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1348153 in Launchpad itself: "DistributionSet:+add's restricted architecture checkboxes don't work"
  https://bugs.launchpad.net/launchpad/+bug/1348153

For more details, see:
https://code.launchpad.net/~wgrant/launchpad/bug-1348153/+merge/239278

DistributionSet:+add now always sets enabled_restricted_processors, as processors may nowadays be restricted even for non-virtualized archives.
-- 
https://code.launchpad.net/~wgrant/launchpad/bug-1348153/+merge/239278
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~wgrant/launchpad/bug-1348153 into lp:launchpad.

=== modified file 'lib/lp/registry/browser/distribution.py'
--- lib/lp/registry/browser/distribution.py	2014-08-01 06:04:58 +0000
+++ lib/lp/registry/browser/distribution.py	2014-10-22 19:30:37 +0000
@@ -877,9 +877,8 @@
             )
         archive = distribution.main_archive
         self.updateRequireVirtualized(data['require_virtualized'], archive)
-        if archive.require_virtualized is True:
-            archive.enabled_restricted_processors = data[
-                'enabled_restricted_processors']
+        archive.enabled_restricted_processors = data[
+            'enabled_restricted_processors']
 
         notify(ObjectCreatedEvent(distribution))
         self.next_url = canonical_url(distribution)