From 258e86dc0b3575dabc946982aca880b0660b56af Mon Sep 17 00:00:00 2001
From: Kelly Dwan <ryelle@users.noreply.github.com>
Date: Tue, 26 Nov 2019 14:01:23 -0500
Subject: [PATCH] Hide contributor editing fields when`$readonly` is true

---
 plugins/wporg-5ftf/assets/css/admin.css       |  1 +
 .../wporg-5ftf/views/manage-contributors.php  | 30 ++++++++++++-------
 2 files changed, 20 insertions(+), 11 deletions(-)

diff --git a/plugins/wporg-5ftf/assets/css/admin.css b/plugins/wporg-5ftf/assets/css/admin.css
index 7e545e5..0645c5b 100755
--- a/plugins/wporg-5ftf/assets/css/admin.css
+++ b/plugins/wporg-5ftf/assets/css/admin.css
@@ -52,6 +52,7 @@
 	margin-bottom: 1.5rem;
 }
 
+.pledge-contributors.pledge-status__5ftf-deactivated .resend-confirm,
 .pledge-contributors.pledge-status__draft .resend-confirm {
 	display: none;
 }
diff --git a/plugins/wporg-5ftf/views/manage-contributors.php b/plugins/wporg-5ftf/views/manage-contributors.php
index fd697d7..e9b1c2a 100644
--- a/plugins/wporg-5ftf/views/manage-contributors.php
+++ b/plugins/wporg-5ftf/views/manage-contributors.php
@@ -5,8 +5,8 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 
 /** @var array $contributors */
 /** @var int   $pledge_id */
+/** @var bool  $readonly */
 ?>
-
 <script type="text/template" id="tmpl-5ftf-contributor-lists">
 	<# if ( data.publish.length ) { #>
 		<h3 class="contributor-list-heading"><?php esc_html_e( 'Confirmed', 'wporg-5ftf' ); ?></h3>
@@ -14,7 +14,9 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 			<thead>
 				<th scope="col"><?php esc_html_e( 'Contributor', 'wporg-5ftf' ); ?></th>
 				<th scope="col"><?php esc_html_e( 'Date Confirmed', 'wporg-5ftf' ); ?></th>
+				<?php if ( ! $readonly ) : ?>
 				<th scope="col"><?php esc_html_e( 'Remove Contributor', 'wporg-5ftf' ); ?></th>
+				<?php endif; ?>
 			</thead>
 			<tbody>{{{ data.publish }}}</tbody>
 		</table>
@@ -26,7 +28,9 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 				<tr>
 					<th scope="col"><?php esc_html_e( 'Contributor', 'wporg-5ftf' ); ?></th>
 					<th scope="col" class="resend-confirm"><?php esc_html_e( 'Resend Confirmation', 'wporg-5ftf' ); ?></th>
+					<?php if ( ! $readonly ) : ?>
 					<th scope="col"><?php esc_html_e( 'Remove Contributor', 'wporg-5ftf' ); ?></th>
+					<?php endif; ?>
 				</tr>
 			</thead>
 			<tbody>{{{ data.pending }}}</tbody>
@@ -58,6 +62,7 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 		<# } else { #>
 			<td>{{ data.publishDate }}</td>
 		<# } #>
+		<?php if ( ! $readonly ) : ?>
 		<td>
 			<button
 				class="button-link button-link-delete"
@@ -70,6 +75,7 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 				<?php esc_html_e( 'Remove', 'wporg-5ftf' ); ?>
 			</button>
 		</td>
+		<?php endif; ?>
 	</tr>
 </script> 
 
@@ -88,16 +94,18 @@ use function WordPressDotOrg\FiveForTheFuture\get_views_path;
 	</div>
 
 	<?php
-	$data = [ 'pledge-contributors' => '' ];
-	require get_views_path() . 'inputs-pledge-contributors.php';
-	?>
+	if ( ! $readonly ) :
+		$data = [ 'pledge-contributors' => '' ];
+		require get_views_path() . 'inputs-pledge-contributors.php';
+		?>
 
-	<div id="add-contrib-message" role="alert" aria-atomic="true"></div>
+		<div id="add-contrib-message" role="alert" aria-atomic="true"></div>
 
-	<button
-		class="button-primary"
-		data-action="add-contributor"
-	>
-		<?php esc_html_e( 'Add new contributors', 'wporg-5ftf' ); ?>
-	</button>
+		<button
+			class="button-primary"
+			data-action="add-contributor"
+		>
+			<?php esc_html_e( 'Add new contributors', 'wporg-5ftf' ); ?>
+		</button>
+	<?php endif; ?>
 </div>