linxianzhuang/saasshop
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

平台订单列表回执退款摘要链接补齐 data-role 与 back 护栏

Browse Source
This commit is contained in:
萝卜
2026-03-18 18:27:57 +08:00
parent 08b61b266c
commit 04e823ec0a
2 changed files with 77 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
resources/views/admin/platform_orders/index.blade.php
View File

@@ -729,16 +729,16 @@
<div class="card">
<h3>有退款订单 / 无退款订单</h3>
<div class="metric-number">
<a class="link" href="{!! $safeFullUrlWithQuery(['refund_status' => 'has', 'page' => null]) !!}">{{ $summaryStats['refund_orders'] ?? 0 }}</a>
<a data-role="po-summary-link-refund-orders" class="link" href="{!! $safeFullUrlWithQuery(['refund_status' => 'has', 'page' => null]) !!}">{{ $summaryStats['refund_orders'] ?? 0 }}</a>
/
<a class="link" href="{!! $safeFullUrlWithQuery(['refund_status' => 'none', 'page' => null]) !!}">{{ $summaryStats['no_refund_orders'] ?? 0 }}</a>
<a data-role="po-summary-link-no-refund-orders" class="link" href="{!! $safeFullUrlWithQuery(['refund_status' => 'none', 'page' => null]) !!}">{{ $summaryStats['no_refund_orders'] ?? 0 }}</a>
</div>
<div class="muted muted-xs">口径：refund_summary.total_amount 存在或 refund_receipts 有记录</div>
</div>
<div class="card">
<h3>有回执订单 / 回执总额</h3>
<div class="metric-number">
<a class="link" href="{!! $safeFullUrlWithQuery(['receipt_status' => 'has', 'page' => null]) !!}">{{ $summaryStats['receipt_orders'] ?? 0 }}</a>
<a data-role="po-summary-link-receipt-orders" class="link" href="{!! $safeFullUrlWithQuery(['receipt_status' => 'has', 'page' => null]) !!}">{{ $summaryStats['receipt_orders'] ?? 0 }}</a>
/ ¥{{ number_format((float) ($summaryStats['total_receipt_amount'] ?? 0), 2) }}
</div>
<div class="muted muted-xs">有回执口径：payment_summary.total_amount 存在或 payment_receipts 有记录</div>
@@ -749,7 +749,7 @@
<div class="card">
<h3>无回执订单</h3>
<div class="metric-number">
<a class="link" href="{!! $safeFullUrlWithQuery(['receipt_status' => 'none', 'page' => null]) !!}">{{ $summaryStats['no_receipt_orders'] ?? 0 }}</a>
<a data-role="po-summary-link-no-receipt-orders" class="link" href="{!! $safeFullUrlWithQuery(['receipt_status' => 'none', 'page' => null]) !!}">{{ $summaryStats['no_receipt_orders'] ?? 0 }}</a>
</div>
<div class="muted muted-xs">无 payment_summary 且无 payment_receipts</div>
<div class="muted muted-xs">

73
tests/Feature/AdminPlatformOrderIndexSummaryReceiptRefundLinksShouldHaveDataRoleAndKeepBackTest.php Normal file
View File

@@ -0,0 +1,73 @@
<?php
namespace Tests\Feature;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Tests\TestCase;
class AdminPlatformOrderIndexSummaryReceiptRefundLinksShouldHaveDataRoleAndKeepBackTest extends TestCase
{
use RefreshDatabase;
protected function loginAsPlatformAdmin(): void
{
$this->seed();
$this->post('/admin/login', [
'email' => 'platform.admin@demo.local',
'password' => 'Platform@123456',
])->assertRedirect('/admin');
}
public function test_platform_orders_index_summary_receipt_refund_links_should_have_data_role_and_keep_back(): void
{
$this->loginAsPlatformAdmin();
$res = $this->get('/admin/platform-orders?merchant_id=1&back=/admin&page=2');
$res->assertOk();
$html = (string) $res->getContent();
$cases = [
[
'role' => 'po-summary-link-refund-orders',
'expect' => ['refund_status' => 'has'],
],
[
'role' => 'po-summary-link-no-refund-orders',
'expect' => ['refund_status' => 'none'],
],
[
'role' => 'po-summary-link-receipt-orders',
'expect' => ['receipt_status' => 'has'],
],
[
'role' => 'po-summary-link-no-receipt-orders',
'expect' => ['receipt_status' => 'none'],
],
];
foreach ($cases as $c) {
$role = (string) ($c['role'] ?? '');
$this->assertNotSame('', $role);
$re = '/<a[^>]*data-role="' . preg_quote($role, '/') . '"[^>]*href="([^"]+)"/u';
$this->assertMatchesRegularExpression($re, $html);
preg_match($re, $html, $m);
$rawHref = (string) ($m[1] ?? '');
$this->assertStringNotContainsString('&amp;back=', $rawHref);
$href = html_entity_decode($rawHref);
$query = parse_url($href, PHP_URL_QUERY) ?: '';
parse_str($query, $q);
foreach (($c['expect'] ?? []) as $k => $v) {
$this->assertSame((string) $v, (string) ($q[$k] ?? ''), $role . ' missing expected ' . $k);
}
$this->assertSame('/admin', (string) ($q['back'] ?? ''));
$this->assertArrayNotHasKey('page', $q);
}
}
}