Skip to content

Commit 59b1a97

Browse files
authored
Merge pull request #110 from mailchimp/bug/non-visible-merge-fields-will-display-in-settings-menu-as-options-in-the-include-column
Bug/merge fields not marked as "Visible?" in the user's Mailchimp account will still display as includable in the WP Mailchimp settings page
2 parents 478adea + aee1d9b commit 59b1a97

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

views/setup_page.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,7 @@ function ( $ele ) {
303303
<td><?php echo esc_html( ( 1 === $mv_var['required'] ) ? 'Y' : 'N' ); ?></td>
304304
<td>
305305
<?php
306-
if ( ! $mv_var['required'] ) {
306+
if ( ! $mv_var['required'] && $mv_var['public'] ) {
307307
$opt = 'mc_mv_' . $mv_var['tag'];
308308
?>
309309
<label class="screen-reader-text" for="<?php echo esc_attr( $opt ); ?>">

0 commit comments

Comments
 (0)