Skip to content

Commit adc623c

Browse files
committed
Remove NumVisit as no longer in SS4 (Fixes #666)
1 parent c5fda4e commit adc623c

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

src/Model/OrderItem.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
use SilverStripe\Forms\TextField;
88
use SilverStripe\ORM\FieldType\DBCurrency;
99

10-
1110
/**
1211
* An order item is a product which has been added to an order,
1312
* ready for purchase. An order item is typically a product itself,

src/Page/CheckoutPageController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,10 +151,10 @@ public function PaymentErrorMessage()
151151

152152
return $lastErrorMessage->Message;
153153
}
154-
154+
155155
/**
156156
* Override viewer to get correct template for first step
157-
*
157+
*
158158
* {@inheritDoc}
159159
* @see \SilverStripe\CMS\Controllers\ContentController::getViewer()
160160
*/

src/Reports/CustomerReport.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ public function columns()
3030
'Created' => 'Joined',
3131
'Spent' => 'Spent',
3232
'Orders' => 'Orders',
33-
'NumVisit' => 'Visits',
3433
'edit' => [
3534
'title' => 'Edit',
3635
'formatting' => '<a href=\"admin/security/EditForm/field/Members/item/$ID/edit\" target=\"_new\">edit</a>',
@@ -49,12 +48,15 @@ public function query($params)
4948
$query = parent::query($params);
5049
$query->selectField($this->periodfield, 'FilterPeriod')
5150
->addSelect(
52-
['"Member"."ID"', '"Member"."FirstName"', '"Member"."Surname"', '"Member"."Email"', '"Member"."NumVisit"', '"Member"."Created"']
51+
['"Member"."ID"', '"Member"."FirstName"', '"Member"."Surname"', '"Member"."Email"', '"Member"."Created"']
5352
)
5453
->selectField('COUNT("SilverShop_Order"."ID")', 'Orders')
5554
->selectField('SUM("SilverShop_Order"."Total")', 'Spent');
55+
5656
$query->addInnerJoin('SilverShop_Order', '"Member"."ID" = "SilverShop_Order"."MemberID"');
57+
5758
$query->addGroupBy('"Member"."ID"');
59+
5860
if (!$query->getOrderBy()) {
5961
$query->setOrderBy(
6062
[

0 commit comments

Comments
 (0)