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

chore(admin-dashboard): dedupe BackUrl fragment links for recent order hints

Browse Source
This commit is contained in:
萝卜
2026-03-16 17:02:26 +08:00
parent 0cef9b6186
commit 73e557230a
2 changed files with 54 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
resources/views/admin/dashboard.blade.php
View File

@@ -454,7 +454,7 @@
<span class="muted muted-xs">原因过长</span>
@endif
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'sync-failed') !!}">查看失败详情</a>
<a class="link" href="{!! $scanGoSyncFailedUrl !!}">查看失败详情</a>
</div>
@endif
@if($bmpaErrMsg !== '')
@@ -470,7 +470,7 @@
<span class="muted muted-xs">原因过长</span>
@endif
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'bmpa-failed') !!}">查看失败详情</a>
<a class="link" href="{!! $scanGoBmpaFailedUrl !!}">查看失败详情</a>
</div>
@endif
@if((string) $po->status === 'pending' && (string) $po->payment_status === 'paid' && $po->isReconcileMismatch())
@@ -482,9 +482,9 @@
<span class="muted">｜</span>
<a class="link" href="{!! $reconcileMismatchUrl !!}">进入集合</a>
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'payment-receipts') !!}">去对账</a>
<a class="link" href="{!! $scanGoReconcileUrl !!}">去对账</a>
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'add-payment-receipt') !!}">去补回执</a>
<a class="link" href="{!! $fixReceiptUrl !!}">去补回执</a>
</div>
@endif
@if($po->isRefundInconsistent())
@@ -496,7 +496,7 @@
<span class="muted">｜</span>
<a class="link" href="{!! $refundInconsistentUrl !!}">进入集合</a>
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'refund-receipts') !!}">去核对退款</a>
<a class="link" href="{!! $scanGoRefundUrl !!}">去核对退款</a>
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'add-refund-receipt') !!}">去补退款记录</a>
</div>
@@ -510,7 +510,7 @@
<span class="muted">｜</span>
<a class="link" href="{!! $renewalMissingSubscriptionUrl !!}">进入集合</a>
<span class="muted">｜</span>
<a class="link" href="{!! \App\Support\BackUrl::withBackAndFragment('/admin/platform-orders/' . $po->id, $selfWithoutBack, 'relation-subscription') !!}">去关联订阅</a>
<a class="link" href="{!! $scanGoRelationUrl !!}">去关联订阅</a>
</div>
@endif
</td>

48
tests/Feature/AdminDashboardRecentPlatformOrdersShouldNotUseDuplicateBackUrlWithBackAndFragmentCallsTest.php Normal file
View File

@@ -0,0 +1,48 @@
<?php
namespace Tests\Feature;
use Tests\TestCase;
class AdminDashboardRecentPlatformOrdersShouldNotUseDuplicateBackUrlWithBackAndFragmentCallsTest extends TestCase
{
public function test_dashboard_recent_platform_orders_should_not_use_duplicate_back_url_with_back_and_fragment_calls(): void
{
$blade = (string) file_get_contents(resource_path('views/admin/dashboard.blade.php'));
// 回归保护：recent orders 相关的锚点 URL 构建应收敛到上方统一变量定义，
// 避免在下方提示块/扫描行里重复手写 withBackAndFragment（易回退/易漏改）。
// 允许出现 1 次（用于变量定义处）。
$this->assertSame(1, substr_count(
$blade,
"BackUrl::withBackAndFragment('/admin/platform-orders/' . \$po->id, \$selfWithoutBack, 'sync-failed')"
));
$this->assertSame(1, substr_count(
$blade,
"BackUrl::withBackAndFragment('/admin/platform-orders/' . \$po->id, \$selfWithoutBack, 'bmpa-failed')"
));
$this->assertSame(1, substr_count(
$blade,
"BackUrl::withBackAndFragment('/admin/platform-orders/' . \$po->id, \$selfWithoutBack, 'payment-receipts')"
));
$this->assertSame(1, substr_count(
$blade,
"BackUrl::withBackAndFragment('/admin/platform-orders/' . \$po->id, \$selfWithoutBack, 'refund-receipts')"
));
$this->assertSame(1, substr_count(
$blade,
"BackUrl::withBackAndFragment('/admin/platform-orders/' . \$po->id, \$selfWithoutBack, 'relation-subscription')"
));
// 仍应存在变量引用（说明已收敛到统一变量）
$this->assertStringContainsString('$scanGoSyncFailedUrl', $blade);
$this->assertStringContainsString('$scanGoBmpaFailedUrl', $blade);
$this->assertStringContainsString('$scanGoReconcileUrl', $blade);
$this->assertStringContainsString('$scanGoRefundUrl', $blade);
$this->assertStringContainsString('$scanGoRelationUrl', $blade);
}
}