From 75ba28973df56d50c3a313bb41857a7f690726c9 Mon Sep 17 00:00:00 2001
From: George Dawoud <george@dawouds.com>
Date: Sun, 23 Feb 2025 14:55:52 -0800
Subject: [PATCH 1/2] fixed saving new why come

---
 src/WhyCameEditor.php | 30 ++++++++++++------------------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/src/WhyCameEditor.php b/src/WhyCameEditor.php
index baa6436385..c7d5737bf3 100644
--- a/src/WhyCameEditor.php
+++ b/src/WhyCameEditor.php
@@ -29,27 +29,21 @@
     $tSuggest = InputUtils::legacyFilterInput($_POST['Suggest']);
     $tHearOfUs = InputUtils::legacyFilterInput($_POST['HearOfUs']);
 
-    // New input (add)
-    if (strlen($iWhyCameID) < 1) {
+    $whyCame = WhyCameQuery::create()->findOneByPerId($iPerson);
+    if ($whyCame === null) {
+        LoggerUtils::getAppLogger()->info("person id " . $iPerson . " whycame id null" );
         $whyCame = new WhyCame();
-        $whyCame
-            ->setPerId($iPerson)
-            ->setJoin($tJoin)
-            ->setCome($tCome)
-            ->setSuggest($tSuggest)
-            ->setHearOfUs($tHearOfUs);
-        $whyCame->save();
-    // Existing record (update)
-    } else {
-        $whyCame = WhyCameQuery::create()->findOneByPerId($iPerson);
-        $whyCame
-            ->setJoin($tJoin)
-            ->setCome($tCome)
-            ->setSuggest($tSuggest)
-            ->setHearOfUs($tHearOfUs);
-        $whyCame->save();
+        $whyCame->setPerId($iPerson);
     }
 
+    $whyCame
+        ->setJoin($tJoin)
+        ->setCome($tCome)
+        ->setSuggest($tSuggest)
+        ->setHearOfUs($tHearOfUs);
+        
+    $whyCame->save();
+
     if (isset($_POST['Submit'])) {
         // Check for redirection to another page after saving information: (ie. PledgeEditor.php?previousPage=prev.php?a=1;b=2;c=3)
         if ($linkBack != '') {

From 4c3ff2aa98c3f0acf79bd91664a3c5ac5d466db4 Mon Sep 17 00:00:00 2001
From: George Dawoud <george@dawouds.com>
Date: Sun, 23 Feb 2025 15:04:12 -0800
Subject: [PATCH 2/2] added test of why Come of edit a blank person and view it

---
 .../e2e/ui/people/standard.person.profile.spec.js    | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/cypress/e2e/ui/people/standard.person.profile.spec.js b/cypress/e2e/ui/people/standard.person.profile.spec.js
index ddfd56bd18..7fc075333b 100644
--- a/cypress/e2e/ui/people/standard.person.profile.spec.js
+++ b/cypress/e2e/ui/people/standard.person.profile.spec.js
@@ -34,7 +34,17 @@ context("Person Profile", () => {
 
         cy.get("#editWhyCame").click();
         cy.url().should("contains", `WhyCameEditor.php?PersonID=${personId}`);
+        cy.get('tr:nth-child(1) textarea').clear().type('{backspace}');
+        cy.get('tr:nth-child(1) textarea').clear().type('I love the lord ');
+        cy.get('tr:nth-child(2) textarea').clear().type('{backspace}');
+        cy.get('tr:nth-child(2) textarea').clear().type('{backspace}');
+        cy.get('tr:nth-child(2) textarea').clear().type('the feeling of being included');
+        cy.get('tr:nth-child(3) textarea').clear().type('More Youth Meetings');
+        cy.get('tr:nth-child(4) textarea').clear().type('The website ');
+        cy.get('td > .btn-primary').click();
+
+        cy.url().should('contains', 'WhyCameEditor.php');
+        cy.contains('More Youth Meetings');
 
-        // TODO: add editing
     });
 });