Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
M
mailchimp-3352943
Manage
Activity
Members
Labels
Plan
Custom issue tracker
Code
Merge requests
0
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Model registry
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Insights
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Issue forks
mailchimp-3352943
Commits
9be504ef
Commit
9be504ef
authored
8 years ago
by
Stephan Zeidler
Committed by
amyvs
8 years ago
Browse files
Options
Downloads
Patches
Plain Diff
Issue
#2776765
by szeidler: Check for undefined mergevar list-id, when no list available
parent
118c9589
No related branches found
Branches containing commit
Tags
8.x-1.7
Tags containing commit
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
modules/mailchimp_signup/src/Form/MailchimpSignupPageForm.php
+1
-0
1 addition, 0 deletions
...les/mailchimp_signup/src/Form/MailchimpSignupPageForm.php
with
1 addition
and
0 deletions
modules/mailchimp_signup/src/Form/MailchimpSignupPageForm.php
+
1
−
0
View file @
9be504ef
...
...
@@ -111,6 +111,7 @@ class MailchimpSignupPageForm extends FormBase {
}
}
$mergevars_wrapper_id
=
isset
(
$list
->
id
)
?
$list
->
id
:
''
;
$form
[
'mergevars'
]
=
array
(
'#prefix'
=>
'<div id="mailchimp-newsletter-'
.
$list
->
id
.
'-mergefields" class="mailchimp-newsletter-mergefields">'
,
'#suffix'
=>
'</div>'
,
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment