Skip to content

Commit

Permalink
Code cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
bencroker committed Feb 8, 2024
1 parent 5d2f325 commit 1cde659
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 9 deletions.
8 changes: 1 addition & 7 deletions src/helpers/SettingsHelper.php
Original file line number Diff line number Diff line change
Expand Up @@ -133,13 +133,7 @@ public static function isDynamicWebAliasUsed(int $siteId = null): bool
}

foreach ($sites as $site) {
// How this works was changed in 3.6.0
// https://github.com/craftcms/cms/issues/3964#issuecomment-737546660
if (version_compare(Craft::$app->getVersion(), '3.6.0', '>=')) {
$unparsedBaseUrl = $site->getBaseUrl(false);
} else {
$unparsedBaseUrl = $site->baseUrl;
}
$unparsedBaseUrl = $site->getBaseUrl(false);

if (stripos($unparsedBaseUrl, '@web') !== false) {
return true;
Expand Down
2 changes: 1 addition & 1 deletion src/services/FormsService.php
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ public function sendVerifyUnsubscribeEmail(ContactElement $contact, MailingListE
}

/**
* Creates and subscribes a contact, with verification if enabled on the mailing list.
* Creates and subscribes a contact, with verification if enabled on the mailing list type.
*
* @since 2.1.0
*/
Expand Down
1 change: 0 additions & 1 deletion src/templates/_settings/campaigntypes/edit.twig
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@
{{ forms.checkboxField({
label: "Show the Title field"|t('app'),
name: 'hasTitleField',
toggle: 'title-container',
reverseToggle: '#titleFormat-container, .fld-title-field-icon',
checked: campaignType.hasTitleField,
}) }}
Expand Down

0 comments on commit 1cde659

Please sign in to comment.