Feature: Form inherits campaign goal

src/Campaigns/Actions/CreateDefaultCampaignForm.php

@@ -6,7 +6,9 @@
 use Give\Campaigns\Models\Campaign;
 use Give\Campaigns\Repositories\CampaignRepository;
 use Give\DonationForms\Models\DonationForm;
+use Give\DonationForms\Properties\FormSettings;
 use Give\DonationForms\ValueObjects\DonationFormStatus;
+use Give\DonationForms\ValueObjects\GoalType;
 
 /**
  * @unreleased
@@ -21,6 +23,12 @@ public function __invoke(Campaign $campaign)
         $defaultCampaignForm = DonationForm::factory()->create([
             'title' => $campaign->title,
             'status' => DonationFormStatus::DRAFT(),
+            'settings' => FormSettings::fromArray([
+                'enableDonationGoal' => true,
+                'goalAmount' => $campaign->goal,
+                'goalType' => $campaign->goalType->getValue(),
+                'designId' => 'classic',
+            ]),
         ]);
 
         give(CampaignRepository::class)->addCampaignForm($campaign, $defaultCampaignForm, true);

src/Campaigns/Actions/FormInheritsCampaignGoal.php

@@ -0,0 +1,33 @@
+<?php
+
+namespace Give\Campaigns\Actions;
+
+use Give\Campaigns\Models\Campaign;
+use Give\DonationForms\Models\DonationForm;
+use Give\DonationForms\ValueObjects\GoalType;
+
+/**
+ * @unreleased
+ *
+ * Form inherits campaign goal
+ *
+ * @event givewp_donation_form_creating
+ */
+class FormInheritsCampaignGoal
+{
+    /**
+     * @unreleased
+     */
+    public function __invoke(DonationForm $donationForm): void
+    {
+        if (isset($_GET['campaignId'])) {
+            $campaign = Campaign::find((int)$_GET['campaignId']);
+
+            if ($campaign) {
+                $donationForm->settings->enableDonationGoal = true;
+                $donationForm->settings->goalAmount = $campaign->goal;
+                $donationForm->settings->goalType = new GoalType($campaign->goalType->getValue());
+            }
+        }
+    }
+}

src/Campaigns/DataTransferObjects/CampaignGoalData.php

@@ -0,0 +1,123 @@
+<?php
+
+namespace Give\Campaigns\DataTransferObjects;
+
+use Give\Campaigns\CampaignDonationQuery;
+use Give\Campaigns\Models\Campaign;
+use Give\DonationForms\ValueObjects\GoalType;
+use Give\Framework\Support\Contracts\Arrayable;
+
+/**
+ * @unreleased
+ */
+class CampaignGoalData implements Arrayable
+{
+    /**
+     * @var Campaign
+     */
+    private $campaign;
+
+    /**
+     * @var int
+     */
+    public $actual;
+
+    /**
+     * @var int
+     */
+    public $percentage;
+
+    /**
+     * @var int
+     */
+    private $goal;
+
+    /**
+     * @var int|string
+     */
+    public $goalFormatted;
+
+    /**
+     * @var int|string
+     */
+    public $actualFormatted;
+
+    /**
+     * @unreleased
+     */
+    public function __construct(Campaign $campaign)
+    {
+        $this->campaign = $campaign;
+        $this->actual = $this->getActual();
+        $this->actualFormatted = $this->getActualFormatted();
+        $this->percentage = $this->getPercentage();
+        $this->goal = $campaign->goal;
+        $this->goalFormatted = $this->getGoalFormatted();
+    }
+
+    /**
+     * @unreleased
+     */
+    private function getActual(): int
+    {
+        $query = new CampaignDonationQuery($this->campaign);
+
+        switch ($this->campaign->goalType->getValue()) {
+            case GoalType::DONATIONS():
+                return $query->countDonations();
+
+            case GoalType::DONORS():
+                return $query->countDonors();
+
+            case GoalType::AMOUNT():
+            default:
+                return $query->sumIntendedAmount();
+        }
+    }
+
+    /**
+     * @unreleased
+     */
+    private function getPercentage(): float
+    {
+        return round($this->actual / $this->campaign->goal * 100, 2);
+    }
+
+    /**
+     * @unreleased
+     */
+    private function getActualFormatted(): string
+    {
+        if ($this->campaign->goalType == GoalType::AMOUNT) {
+            return give_currency_filter(give_format_amount($this->actual));
+        }
+
+        return $this->actual;
+    }
+
+    /**
+     * @unreleased
+     */
+    private function getGoalFormatted(): string
+    {
+        if ($this->campaign->goalType == GoalType::AMOUNT) {
+            return give_currency_filter(give_format_amount($this->goal));
+        }
+
+        return $this->goal;
+    }
+
+    /**
+     * @unreleased
+     */
+    public function toArray(): array
+    {
+        return [
+            'actual' => $this->actual,
+            'actualFormatted' => $this->actualFormatted,
+            'percentage' => $this->percentage,
+            'goal' => $this->goal,
+            'goalFormatted' => $this->goalFormatted,
+        ];
+    }
+}

src/Campaigns/ListTable/Columns/GoalColumn.php

@@ -2,6 +2,7 @@
 
 namespace Give\Campaigns\ListTable\Columns;
 
+use Give\Campaigns\DataTransferObjects\CampaignGoalData;
 use Give\Campaigns\Models\Campaign;
 use Give\Framework\ListTable\ModelColumn;
 
@@ -35,20 +36,7 @@ public function getLabel(): string
      */
     public function getCellValue($model): string
     {
-        // Temp value considering only the default form associated with the campaign
-        if ($model->defaultForm()) {
-            $goal = give_goal_progress_stats($model->defaultForm()->id);
-            $goalPercentage = ('percentage' === $goal['format']) ? str_replace('%', '',
-                $goal['actual']) : max(min($goal['progress'], 100), 0);
-            $goalActual = $goal['actual'];
-            $goalFormat = $goal['format'];
-            $campaignGoal = $model->goal;
-        } else {
-            $goalPercentage = 0;
-            $goalActual = 0;
-            $goalFormat = '';
-            $campaignGoal = $model->goal;
-        }
+        $goalData = new CampaignGoalData($model);
 
         $template = '
             <div
@@ -69,16 +57,16 @@ class="goalProgress"
         return sprintf(
             $template,
             $model->id,
-            $goalPercentage,
-            $goalActual,
+            $goalData->percentage,
+            $goalData->actualFormatted,
             sprintf(
-                ($goalFormat !== 'percentage' ? ' %s %s' : ''),
+                ' %s %s',
                 __('of', 'give'),
-                $campaignGoal
+                $goalData->goalFormatted
             ),
             sprintf(
                 '<span style="opacity:%1$s" class="goalProgress--achieved"><img src="%2$s" alt="%3$s" />%4$s</span>',
-                apply_filters('givewp_list_table_goal_progress_achieved_opacity', $goalPercentage >= 100 ? 1 : 0),
+                apply_filters('givewp_list_table_goal_progress_achieved_opacity', $goalData->percentage >= 100 ? 1 : 0),
                 GIVE_PLUGIN_URL . 'assets/dist/images/list-table/star-icon.svg',
                 __('Goal achieved icon', 'give'),
                 __('Goal achieved!', 'give')

src/Campaigns/ListTable/Columns/RevenueColumn.php

@@ -37,7 +37,7 @@ public function getLabel(): string
     public function getCellValue($model, $locale = ''): string
     {
         $query = new CampaignDonationQuery($model);
-        $revenue = $query->sumIntendedAmount();
+        $revenue = give_currency_filter(give_format_amount($query->sumIntendedAmount()));
 
         return sprintf(
             '<a class="column-earnings-value" href="%s" aria-label="%s">%s</a>',

src/Campaigns/Migrations/RevenueTable/AddCampaignID.php

@@ -0,0 +1,59 @@
+<?php
+
+namespace Give\Campaigns\Migrations\RevenueTable;
+
+use Give\Framework\Database\Exceptions\DatabaseQueryException;
+use Give\Framework\Migrations\Contracts\Migration;
+use Give\Framework\Migrations\Exceptions\DatabaseMigrationException;
+
+/**
+ * @unreleased
+ */
+class AddCampaignID extends Migration
+{
+    /**
+     * @inheritDoc
+     */
+    public static function id(): string
+    {
+        return 'add_campaign_id_to_revenue_table';
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public static function title(): string
+    {
+        return 'Add Campaign ID to revenue table';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public static function timestamp(): string
+    {
+        return strtotime('2024-10-14 00:00:00');
+    }
+
+    /**
+     * @inheritDoc
+     * @throws DatabaseMigrationException
+     */
+    public function run()
+    {
+        global $wpdb;
+
+        $table = $wpdb->give_revenue;
+
+        $sql = "
+            ALTER TABLE $table
+            ADD COLUMN campaign_id INT UNSIGNED NOT NULL DEFAULT '0'
+        ";
+
+        try {
+            $wpdb->query($sql);
+        } catch (DatabaseQueryException $exception) {
+            throw new DatabaseMigrationException("An error occurred while updating the $table table", 0, $exception);
+        }
+    }
+}

src/Campaigns/Migrations/RevenueTable/AddIndexes.php

@@ -0,0 +1,54 @@
+<?php
+
+namespace Give\Campaigns\Migrations\RevenueTable;
+
+use Give\Framework\Database\DB;
+use Give\Framework\Database\Exceptions\DatabaseQueryException;
+use Give\Framework\Migrations\Contracts\Migration;
+use Give\Framework\Migrations\Exceptions\DatabaseMigrationException;
+
+/**
+ * @unreleased
+ */
+class AddIndexes extends Migration
+{
+    /**
+     * @inheritDoc
+     */
+    public static function id(): string
+    {
+        return 'add_indexes_to_revenue_table';
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public static function title(): string
+    {
+        return 'Add indexes to revenue table';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public static function timestamp(): string
+    {
+        return strtotime('2024-10-14 00:00:02');
+    }
+
+    /**
+     * @inheritDoc
+     * @throws DatabaseMigrationException
+     */
+    public function run()
+    {
+        global $wpdb;
+
+        try {
+            DB::query("ALTER TABLE {$wpdb->give_revenue} ADD INDEX (form_id), ADD INDEX (campaign_id)");
+        } catch (DatabaseQueryException $exception) {
+            throw new DatabaseMigrationException("An error occurred while updating the {$wpdb->give_revenue} table", 0,
+                $exception);
+        }
+    }
+}