fix(core,board,ecommerce,plugins): API 문서 미채움 마커 전수 채움 + 미해석 lang 키 정정 + IDV 조회 시도횟수 비노출
이슈 본연: API 레퍼런스의 미채움 마커 5종(실측 제외/TODO/필드 없음/대표 에러 없음)
1,194건을 코드에서 읽어 전수 채우고(0건), api:docgen 재생성이 사람이 채운 내용을
손실·열화시키던 멱등성 결함 4종(CRLF, 표 파이프 이스케이프, 에러 서술 보존, 중복
라우트명 키)을 근본 수정. --check 를 실측 제외와 드리프트가 구분되도록 재정의.
ParameterDescriber/ResourceFieldDescriber 에 leaf 폴백·SEO 스코프·공통 사전 확장.
파생 결함(문서 실측 중 발견): ResponseHelper 기본 메시지 키 6종이 존재하지 않는
messages.* 를 가리켜 응답 message 가 번역문 대신 키 문자열로 노출되던 문제를 common.*
으로 정정하고, 코어 7 + gdpr/marketing/pay_kginicis/verification/ecommerce 확장의
호출부와 누락 lang 키(게시판 10키 + 이커머스 category_images 4키 포함)를 ko/en/ja
전수 정정. 중복 그룹 키 2건은 기존 키로 호출부 통합.
보안: GET /api/identity/challenges/{id} 는 권한 가드 없는 공개 폴링 엔드포인트인데
Service::getStatus 가 attempts/max_attempts 를 응답에 담아 남은 시도 횟수를 추론할 수
있었다. 두 필드를 제거하고, 챌린지 화면(admin_basic/basic)의 서버 attempts 참조를
제거(남은 횟수 UI 는 query fallback + verify 실패 시 로컬 증가로 유지).
부수 정리(변경셋에 들어온 board 컨트롤러의 audit 사전 결함): FormRequest 4개 신설로
base Request 주입 제거, CommentController 의 Board 직접 호출을 BoardService 위임으로
전환, PHPDoc @return 보강. board 스위트에서 발견한 stale test 3건(seed SSoT 불일치,
admin 계정 user 역할 누락)도 같은 세션에서 정정.
버전 정렬(공개 release 기준): 코어 7.0.3→7.0.4, sirsoft-board 1.0.1→1.0.2,
verification_kginicis 1.0.0→1.0.1, admin_basic 1.0.1→1.0.4.
This commit is contained in:
1 parent
b07d6c8953
commit
6c8381f164
155 files changed
+17192
-1829
No files matched your search
@@ -248,8 +248,8 @@ class ResponseHelperTest extends TestCase
|
||||
/**
|
||||
* 깊은 호출 스택의 예외를 생성합니다.
|
||||
*
|
||||
* @param int $depth 호출 깊이
|
||||
* @return \RuntimeException
|
||||
* @param int $depth 호출 깊이
|
||||
* @return \RuntimeException 생성된 예외
|
||||
*/
|
||||
private function createDeepException(int $depth): \RuntimeException
|
||||
{
|
||||
@@ -294,4 +294,74 @@ class ResponseHelperTest extends TestCase
|
||||
|
||||
$this->assertSame('欢迎', $data['message']);
|
||||
}
|
||||
|
||||
/**
|
||||
* 회귀: 기본 메시지 키가 실제로 번역되어야 한다.
|
||||
*
|
||||
* `messages.success` 는 `lang/{ko,en}/messages.php` 가 존재하지 않아 번역되지 않았고,
|
||||
* `__()` 가 미해석 키 문자열을 그대로 반환해 API 응답에 `"message": "messages.success"` 가
|
||||
* 그대로 나갔다 (코어 7곳 + gdpr/marketing/pay_kginicis/verification_kginicis 등 43개 호출).
|
||||
* 정상 키는 `common.success` 다 (ko: "성공적으로 처리되었습니다." / en: "Successfully processed.").
|
||||
*/
|
||||
public function test_default_success_message_key_is_translated(): void
|
||||
{
|
||||
foreach (['ko', 'en'] as $locale) {
|
||||
App::setLocale($locale);
|
||||
|
||||
$data = ResponseHelper::success()->getData(true);
|
||||
|
||||
$this->assertNotSame(
|
||||
'messages.success',
|
||||
$data['message'],
|
||||
"[{$locale}] 기본 메시지 키가 번역되지 않고 키 문자열 그대로 응답에 나갔다"
|
||||
);
|
||||
|
||||
// 미해석 lang 키는 `도메인.키` 형태 그대로 반환된다 (공백 없는 dot notation).
|
||||
// 번역문은 문장이므로 공백을 포함한다.
|
||||
$this->assertDoesNotMatchRegularExpression(
|
||||
'/^[a-z_]+(\.[a-z_]+)+$/',
|
||||
$data['message'],
|
||||
"[{$locale}] 번역문이 아니라 미해석 lang 키로 보인다: {$data['message']}"
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* 회귀: 코드베이스에 미해석 lang 키(`messages.success`)를 넘기는 호출이 없어야 한다.
|
||||
*
|
||||
* ResponseHelper 기본값만 고치면 명시적으로 그 키를 넘기던 호출부는 여전히 미해석 키를 낸다.
|
||||
*/
|
||||
public function test_no_caller_passes_unresolvable_messages_success_key(): void
|
||||
{
|
||||
$roots = array_filter([
|
||||
base_path('app'),
|
||||
...glob(base_path('modules/_bundled/*/src')),
|
||||
...glob(base_path('plugins/_bundled/*/src')),
|
||||
], 'is_dir');
|
||||
|
||||
$offenders = [];
|
||||
|
||||
foreach ($roots as $root) {
|
||||
$files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($root));
|
||||
|
||||
foreach ($files as $file) {
|
||||
if (! $file->isFile() || $file->getExtension() !== 'php') {
|
||||
continue;
|
||||
}
|
||||
|
||||
$content = (string) file_get_contents($file->getPathname());
|
||||
|
||||
if (str_contains($content, "'messages.success'")) {
|
||||
$offenders[] = str_replace(base_path().DIRECTORY_SEPARATOR, '', $file->getPathname());
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
$this->assertSame(
|
||||
[],
|
||||
$offenders,
|
||||
"번역되지 않는 'messages.success' 키를 넘기는 호출이 남아 있다 (정상 키: 'common.success'):\n - "
|
||||
.implode("\n - ", $offenders)
|
||||
);
|
||||
}
|
||||
}
|
||||
Reference in new issue
Block a user