Do not try to add parameters to empty URL
Closes #14505 (grafted from a014cc36dac8a16f6c3ebd2450f1ba930a7f614f) --HG-- branch : 7.0
This commit is contained in:
@@ -183,7 +183,10 @@ class Email(DeactivableMixin, ModelSQL, ModelView):
|
||||
def get_email_subscribe_url(self, url=None):
|
||||
if url is None:
|
||||
url = config.get('marketing', 'email_subscribe_url')
|
||||
return _add_params(url, token=self.email_token)
|
||||
if url is not None:
|
||||
return _add_params(url, token=self.email_token)
|
||||
else:
|
||||
return ''
|
||||
|
||||
@classmethod
|
||||
def subscribe_url(cls, url):
|
||||
@@ -215,7 +218,10 @@ class Email(DeactivableMixin, ModelSQL, ModelView):
|
||||
def get_email_unsubscribe_url(self, url=None):
|
||||
if url is None:
|
||||
url = config.get('marketing', 'email_unsubscribe_url')
|
||||
return _add_params(url, token=self.email_token)
|
||||
if url is not None:
|
||||
return _add_params(url, token=self.email_token)
|
||||
else:
|
||||
return ''
|
||||
|
||||
@classmethod
|
||||
def unsubscribe_url(cls, url):
|
||||
|
||||
@@ -318,7 +318,10 @@ class User(avatar_mixin(100), DeactivableMixin, ModelSQL, ModelView):
|
||||
def get_email_validation_url(self, url=None):
|
||||
if url is None:
|
||||
url = config.get('web', 'email_validation_url')
|
||||
return _add_params(url, token=self.email_token)
|
||||
if url is not None:
|
||||
return _add_params(url, token=self.email_token)
|
||||
else:
|
||||
return ''
|
||||
|
||||
@classmethod
|
||||
def validate_email_url(cls, url):
|
||||
@@ -372,8 +375,11 @@ class User(avatar_mixin(100), DeactivableMixin, ModelSQL, ModelView):
|
||||
def get_email_reset_password_url(self, url=None):
|
||||
if url is None:
|
||||
url = config.get('web', 'reset_password_url')
|
||||
return _add_params(
|
||||
url, token=self.reset_password_token, email=self.email)
|
||||
if url is not None:
|
||||
return _add_params(
|
||||
url, token=self.reset_password_token, email=self.email)
|
||||
else:
|
||||
return ''
|
||||
|
||||
@classmethod
|
||||
def set_password_url(cls, url, password):
|
||||
|
||||
Reference in New Issue
Block a user