rename branding to email_branding

to help transition to having letter branding as well
This commit is contained in:
Leo Hemsted
2019-02-01 16:31:34 +00:00
parent b81704ab69
commit 0e20ca44a3
2 changed files with 9 additions and 9 deletions

View File

@@ -782,7 +782,7 @@ class RadioFieldWithRequiredMessage(RadioField):
raise ValueError(self.required_message)
class ServiceSetBranding(StripWhitespaceForm):
class ServiceSetEmailBranding(StripWhitespaceForm):
branding_style = RadioFieldWithNoneOption(
'Branding style',
@@ -793,14 +793,14 @@ class ServiceSetBranding(StripWhitespaceForm):
DEFAULT = (FieldWithNoneOption.NONE_OPTION_VALUE, 'GOV.UK')
def __init__(self, all_email_brandings, current_email_branding):
def __init__(self, all_branding_options, current_branding):
super().__init__(branding_style=current_email_branding)
super().__init__(branding_style=current_branding)
self.branding_style.choices = sorted(
all_email_brandings + [self.DEFAULT],
all_branding_options + [self.DEFAULT],
key=lambda branding: (
branding[0] != current_email_branding,
branding[0] != current_branding,
branding[0] is not self.DEFAULT[0],
branding[1].lower(),
),

View File

@@ -45,7 +45,7 @@ from app.main.forms import (
ServiceLetterContactBlockForm,
ServicePreviewBranding,
ServiceReplyToEmailForm,
ServiceSetBranding,
ServiceSetEmailBranding,
ServiceSmsSenderForm,
ServiceSwitchChannelForm,
SMSPrefixForm,
@@ -759,9 +759,9 @@ def set_free_sms_allowance(service_id):
def service_set_email_branding(service_id):
email_branding = email_branding_client.get_all_email_branding()
form = ServiceSetBranding(
all_email_brandings=get_branding_as_value_and_label(email_branding),
current_email_branding=current_service.email_branding_id,
form = ServiceSetEmailBranding(
all_branding_options=get_branding_as_value_and_label(email_branding),
current_branding=current_service.email_branding_id,
)
if form.validate_on_submit():