From aa79c8147e97fc5e6decc2504b4fb1c65d699908 Mon Sep 17 00:00:00 2001
From: Kelly Dwan <ryelle@users.noreply.github.com>
Date: Wed, 13 Nov 2019 13:19:43 -0500
Subject: [PATCH] Add the "Add Contributors" input to the form

---
 plugins/wporg-5ftf/includes/pledge-form.php   |  3 ++-
 plugins/wporg-5ftf/includes/pledge-meta.php   |  1 +
 .../views/inputs-pledge-contributors.php      |  4 ++--
 .../wporg-5ftf/views/manage-contributors.php  | 19 ++++++++++++++++---
 4 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/plugins/wporg-5ftf/includes/pledge-form.php b/plugins/wporg-5ftf/includes/pledge-form.php
index c5eca80..b510fcd 100755
--- a/plugins/wporg-5ftf/includes/pledge-form.php
+++ b/plugins/wporg-5ftf/includes/pledge-form.php
@@ -155,7 +155,8 @@ function render_form_manage() {
 	$updated  = false;
 
 	// @todo Get pledge ID from somewhere.
-	$data = PledgeMeta\get_pledge_meta();
+	$data      = PledgeMeta\get_pledge_meta();
+	$is_manage = true;
 
 	if ( 'Update Pledge' === $action ) {
 		$processed = process_form_manage();
diff --git a/plugins/wporg-5ftf/includes/pledge-meta.php b/plugins/wporg-5ftf/includes/pledge-meta.php
index df5198e..28d849d 100755
--- a/plugins/wporg-5ftf/includes/pledge-meta.php
+++ b/plugins/wporg-5ftf/includes/pledge-meta.php
@@ -203,6 +203,7 @@ function add_meta_boxes() {
  */
 function render_meta_boxes( $pledge, $box ) {
 	$readonly = ! current_user_can( 'edit_page', $pledge->ID );
+	$is_manage = true;
 
 	$data = array();
 	foreach ( get_pledge_meta_config() as $key => $config ) {
diff --git a/plugins/wporg-5ftf/views/inputs-pledge-contributors.php b/plugins/wporg-5ftf/views/inputs-pledge-contributors.php
index 12c8b83..c2eedcc 100644
--- a/plugins/wporg-5ftf/views/inputs-pledge-contributors.php
+++ b/plugins/wporg-5ftf/views/inputs-pledge-contributors.php
@@ -2,7 +2,7 @@
 namespace WordPressDotOrg\FiveForTheFuture\View;
 
 /** @var array $data */
-/** @var bool  $readonly */
+/** @var bool  $is_manage */
 ?>
 
 <div class="form-field">
@@ -15,7 +15,7 @@ namespace WordPressDotOrg\FiveForTheFuture\View;
 		name="pledge-contributors"
 		placeholder="sanguine.zoe206, captain-mal, kayleefixesyou"
 		value="<?php echo esc_attr( $data['pledge-contributors'] ); ?>"
-		required
+		<?php echo $is_manage ? '' : 'required'; ?>
 		aria-describedby="5ftf-pledge-contributors-help"
 	/>
 	<p id="5ftf-pledge-contributors-help">
diff --git a/plugins/wporg-5ftf/views/manage-contributors.php b/plugins/wporg-5ftf/views/manage-contributors.php
index d345a70..54dcaa7 100644
--- a/plugins/wporg-5ftf/views/manage-contributors.php
+++ b/plugins/wporg-5ftf/views/manage-contributors.php
@@ -1,6 +1,8 @@
 <?php
 namespace WordPressDotOrg\FiveForTheFuture\View;
 
+use function WordPressDotOrg\FiveForTheFuture\get_views_path;
+
 /** @var array $contributors */
 /** @var array $data */
 /** @var bool  $readonly */
@@ -64,9 +66,20 @@ namespace WordPressDotOrg\FiveForTheFuture\View;
 		<p><?php esc_html_e( 'There are no contributors added to this pledge yet.', 'wporg' ); ?></p>
 	<?php endif; ?>
 
-	<!-- TODO This button doesn't do anything yet.
-	<button class="button-primary" data-action="add-contributor">
+	<hr />
+
+	<?php
+	$data = [ 'pledge-contributors' => '' ];
+	require get_views_path() . 'inputs-pledge-contributors.php';
+	?>
+
+	<div id="add-contrib-message"></div>
+
+	<button
+		class="button-primary"
+		data-action="add-contributor"
+		data-pledge-post="<?php the_ID(); ?>"
+	>
 		<?php esc_html_e( 'Add new contributor', 'wporg' ); ?>
 	</button>
-	-->
 </div>