From 50a83d1108fc26f61cd15d1cd5f9348c5c1de62d Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Wed, 12 Apr 2017 15:44:05 -0500 Subject: [PATCH] request_splitter: use capital for boolean strings in supplement(). Given that str(boolean) is used a capital is sometimes present, but should be consistent to remove the possibility for issues. --- osclib/request_splitter.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/osclib/request_splitter.py b/osclib/request_splitter.py index f95e8958..822df6ef 100644 --- a/osclib/request_splitter.py +++ b/osclib/request_splitter.py @@ -107,9 +107,9 @@ class RequestSplitter(object): if request_id in self.requests_ignored: request.set('ignored', str(self.requests_ignored[request_id])) else: - request.set('ignored', 'false') + request.set('ignored', 'False') - request.set('postponed', 'false') + request.set('postponed', 'False') def ring_get(self, target_package): if self.api.crings: @@ -259,7 +259,7 @@ class RequestSplitter(object): return for request in self.grouped[group]['requests']: - request.set('postponed', 'true') + request.set('postponed', 'True') def propose_staging(self, choose_bootstrapped): found = False @@ -347,8 +347,8 @@ class Strategy(object): class StrategyNone(Strategy): def apply(self, splitter): splitter.filter_add('./action[not(@type="add_role" or @type="change_devel")]') - splitter.filter_add('@ignored="false"') - splitter.filter_add('@postponed="false"') + splitter.filter_add('@ignored="False"') + splitter.filter_add('@postponed="False"') class StrategyRequests(Strategy): def apply(self, splitter):