Changeset 7434 for sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/views/html/ticket-revenue.php

Timestamp:

07/12/2018 12:22:11 AM (8 years ago)

Author:

coreymckrill

Message:

WordCamp Reports: Update Ticket Revenue report, other minor changes

• Show ticket revenue results split out by payment method (instead of WPCS vs non-WPCS).
• Minor HTML markup changes on several reports.
• Minor code cleanup changes on several reports.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/views/html/ticket-revenue.php (modified) (3 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-reports/views/html/ticket-revenue.php

@@ -9 +9 @@
 /** @var \DateTime $start_date */
 /** @var \DateTime $end_date */
-/** @var string $wordcamp_name */
-/** @var array $data */
+/** @var \DateTime $xrt_date */
+/** @var string    $wordcamp_name */
+/** @var array     $data */
+/** @var array     $total */
 
 $asterisk2 = false;
-$groups    = 0;
 ?>
 
 <?php foreach ( $data as $key => $group ) : ?>
     <?php if ( empty( $group['gross_revenue_by_currency'] ) ) continue; ?>
-    <?php if ( 'total' === $key && $groups < 2 ) continue; ?>
 
     <h3>
@@ -36 +36 @@
     <?php endif; ?>
 
-    <ul>
-        <li>Tickets sold: <?php echo number_format_i18n( $group['tickets_sold'] ); ?></li>
-        <li>Tickets refunded: <?php echo number_format_i18n( $group['tickets_refunded'] ); ?></li>
-    </ul>
+    <table class="striped widefat but-not-too-wide">
+        <tbody>
+        <tr>
+            <td>Tickets sold:</td>
+            <td class="number"><?php echo number_format_i18n( $group['tickets_sold'] ); ?></td>
+        </tr>
+        <tr>
+            <td>Tickets refunded:</td>
+            <td class="number"><?php echo number_format_i18n( $group['tickets_refunded'] ); ?></td>
+        </tr>
+        </tbody>
+    </table>
 
     <table class="striped widefat but-not-too-wide">
@@ -83 +91 @@
         </tbody>
     </table>
-    <?php $groups ++; ?>
 <?php endforeach; ?>
 
 <?php if ( ! empty( $total['net_revenue_by_currency'] ) ) : ?>
     <p class="description">
-        * Estimate based on exchange rates for <?php echo esc_html( $end_date->format( 'M jS, Y' ) ); ?>
+        * Estimate based on exchange rates for <?php echo esc_html( $xrt_date->format( 'M jS, Y' ) ); ?>.
         <?php if ( $asterisk2 ) : ?>
             <br />** Currency exchange rate not available.