todos

Author: i-just

src/elements/db/SubscriptionQuery.php

@@ -957,7 +957,7 @@ protected function beforePrepare(): bool
             // todo: should we check for isSuspended too?
             $this->subQuery->leftJoin(
                 ['stripe_invoicedata' => '{{%stripe_invoicedata}}'],
-                "[[stripe_subscriptions.stripeId]] = [[stripe_invoicedata.subscriptionId",
+                "[[stripe_subscriptions.stripeId]] = [[stripe_invoicedata.subscriptionId]]",
             );
 
             $this->subQuery->andWhere(Db::parseTimestampParam(

src/helpers/Price.php

@@ -323,15 +323,17 @@ public static function renderCardHtml(PriceElement $price): string
                 }
             }
 
-            $meta[Craft::t('stripe', 'Metadata')] = collect($stripePrice['metadata'])
+            $meta[Craft::t('stripe', 'Metadata')] =
+                Html::beginTag('div', ['class' => 'pec-metadata']) .
+                collect($stripePrice['metadata'])
                 ->map(function($value, $key) {
-                    // todo: style me!
                     return Html::beginTag('div', ['class' => 'fullwidth']) .
                         Html::tag('em', $key . ': ') .
                         $value .
                         Html::endTag('div');
                 })
-                ->join(' ');
+                ->join(' ') .
+                Html::endTag('div');
         }
 
         $meta[Craft::t('stripe', 'Created at')] = $formatter->asDatetime(

src/helpers/Product.php

@@ -94,15 +94,17 @@ public static function renderCardHtml(ProductElement $product): string
                                     ->join(', ');
                                 break;
                             case 'metadata':
-                                $meta[Craft::t('stripe', $label)] = collect($value)
+                                $meta[Craft::t('stripe', $label)] =
+                                    Html::beginTag('div', ['class' => 'pec-metadata']) .
+                                    collect($value)
                                     ->map(function($val, $i) {
-                                        // todo: style me!
                                         return Html::beginTag('div', ['class' => 'fullwidth']) .
                                             Html::tag('em', $i . ': ') .
                                             $val .
                                             Html::endTag('div');
                                     })
-                                    ->join(' ');
+                                    ->join(' ') .
+                                    Html::endTag('div');
                                 break;
                             case 'default_price':
                                 $defaultPrice = $product->getDefaultPrice();
@@ -148,7 +150,7 @@ public static function renderCardHtml(ProductElement $product): string
                     'id' => $product->id,
                 ]),
                 'swap' => 'outerHTML',
-                'trigger' => 'every 15s',
+                'trigger' => 'every 15m',
             ],
         ]);
     }

src/helpers/Subscription.php

@@ -129,15 +129,17 @@ public static function renderCardHtml(SubscriptionElement $subscription): string
 
         $footer = '';
         if (count($stripeSubscription) > 0) {
-            $meta[Craft::t('stripe', 'Metadata')] = collect($stripeSubscription['metadata'])
+            $meta[Craft::t('stripe', 'Metadata')] =
+                Html::beginTag('div', ['class' => 'pec-metadata']) .
+                collect($stripeSubscription['metadata'])
                 ->map(function($value, $key) {
-                    // todo: style me!
                     return Html::beginTag('div', ['class' => 'fullwidth']) .
                         Html::tag('em', $key . ': ') .
                         $value .
                         Html::endTag('div');
                 })
-                ->join(' ');
+                ->join(' ') .
+                Html::endTag('div');
 
             $meta[Craft::t('stripe', 'Created at')] = $formatter->asDatetime($stripeSubscription['created'], Formatter::FORMAT_WIDTH_SHORT);
 

src/migrations/Install.php

@@ -232,7 +232,6 @@ public function createIndexes(): void
         $this->createIndex(null, Table::SUBSCRIPTIONDATA, ['stripeId'], true);
         $this->createIndex(null, Table::SUBSCRIPTIONDATA, ['customerId']);
         $this->createIndex(null, Table::SUBSCRIPTIONDATA, ['latestInvoiceId'], true);
-        // todo: add for price id?
 
         $this->createIndex(null, Table::CUSTOMERDATA, ['stripeId'], true);
         $this->createIndex(null, Table::CUSTOMERDATA, ['email']);