결재 비활성 안내 개선 및 테이블 기능 보강

lib/features/inventory/outbound/presentation/pages/outbound_page.dart

@@ -844,6 +844,8 @@ class _OutboundPageState extends State<OutboundPage> {
 
     String? writerError;
     String? customerError;
+    String? warehouseError;
+    String? statusError;
     String? headerNotice;
     String customerSearchQuery = '';
     StateSetter? refreshForm;
@@ -855,6 +857,8 @@ class _OutboundPageState extends State<OutboundPage> {
     void handleSubmit() {
       final validation = _validateOutboundForm(
         writerController: writerController,
+        warehouseValue: warehouseController.text,
+        statusValue: statusValue.value,
         customerController: customerController,
         drafts: drafts,
         lineErrors: lineErrors,
@@ -862,6 +866,8 @@ class _OutboundPageState extends State<OutboundPage> {
 
       writerError = validation.writerError;
       customerError = validation.customerError;
+      warehouseError = validation.warehouseError;
+      statusError = validation.statusError;
       headerNotice = validation.headerNotice;
       refreshForm?.call(() {});
 
@@ -882,6 +888,7 @@ class _OutboundPageState extends State<OutboundPage> {
             ),
           )
           .toList();
+      items.sort((a, b) => a.product.compareTo(b.product));
       result = OutboundRecord(
         number: initial?.number ?? _generateOutboundNumber(processedAt.value),
         transactionNumber:
@@ -948,6 +955,7 @@ class _OutboundPageState extends State<OutboundPage> {
                       child: SuperportFormField(
                         label: '창고',
                         required: true,
+                        errorText: warehouseError,
                         child: ShadSelect<String>(
                           initialValue: warehouseController.text,
                           selectedOptionBuilder: (context, value) =>
@@ -955,7 +963,11 @@ class _OutboundPageState extends State<OutboundPage> {
                           onChanged: (value) {
                             if (value != null) {
                               warehouseController.text = value;
-                              setState(() {});
+                              setState(() {
+                                if (warehouseError != null) {
+                                  warehouseError = null;
+                                }
+                              });
                             }
                           },
                           options: [
@@ -970,6 +982,7 @@ class _OutboundPageState extends State<OutboundPage> {
                       child: SuperportFormField(
                         label: '상태',
                         required: true,
+                        errorText: statusError,
                         child: ShadSelect<String>(
                           initialValue: statusValue.value,
                           selectedOptionBuilder: (context, value) =>
@@ -977,7 +990,11 @@ class _OutboundPageState extends State<OutboundPage> {
                           onChanged: (value) {
                             if (value != null) {
                               statusValue.value = value;
-                              setState(() {});
+                              setState(() {
+                                if (statusError != null) {
+                                  statusError = null;
+                                }
+                              });
                             }
                           },
                           options: [
@@ -1609,6 +1626,8 @@ double _parseCurrency(String input) {
 
 _OutboundFormValidation _validateOutboundForm({
   required TextEditingController writerController,
+  required String warehouseValue,
+  required String statusValue,
   required ShadSelectController<String> customerController,
   required List<_OutboundLineItemDraft> drafts,
   required Map<_OutboundLineItemDraft, _OutboundLineErrors> lineErrors,
@@ -1616,6 +1635,8 @@ _OutboundFormValidation _validateOutboundForm({
   var isValid = true;
   String? writerError;
   String? customerError;
+  String? warehouseError;
+  String? statusError;
   String? headerNotice;
 
   if (writerController.text.trim().isEmpty) {
@@ -1623,17 +1644,29 @@ _OutboundFormValidation _validateOutboundForm({
     isValid = false;
   }
 
+  if (warehouseValue.trim().isEmpty) {
+    warehouseError = '창고를 선택하세요.';
+    isValid = false;
+  }
+
+  if (statusValue.trim().isEmpty) {
+    statusError = '상태를 선택하세요.';
+    isValid = false;
+  }
+
   if (customerController.value.isEmpty) {
     customerError = '최소 1개의 고객사를 선택하세요.';
     isValid = false;
   }
 
   var hasLineError = false;
+  final seenProductKeys = <String>{};
   for (final draft in drafts) {
     final errors = lineErrors.putIfAbsent(draft, _OutboundLineErrors.empty);
     errors.clearAll();
 
-    if (draft.product.text.trim().isEmpty) {
+    final productText = draft.product.text.trim();
+    if (productText.isEmpty) {
       errors.product = '제품을 입력하세요.';
       hasLineError = true;
       isValid = false;
@@ -1643,6 +1676,13 @@ _OutboundFormValidation _validateOutboundForm({
       isValid = false;
     }
 
+    final productKey = draft.catalogMatch?.code ?? productText.toLowerCase();
+    if (productKey.isNotEmpty && !seenProductKeys.add(productKey)) {
+      errors.product = '동일 제품이 중복되었습니다.';
+      hasLineError = true;
+      isValid = false;
+    }
+
     final quantity = int.tryParse(
       draft.quantity.text.trim().isEmpty ? '0' : draft.quantity.text.trim(),
     );
@@ -1668,6 +1708,8 @@ _OutboundFormValidation _validateOutboundForm({
     isValid: isValid,
     writerError: writerError,
     customerError: customerError,
+    warehouseError: warehouseError,
+    statusError: statusError,
     headerNotice: headerNotice,
   );
 }
@@ -1677,12 +1719,16 @@ class _OutboundFormValidation {
     required this.isValid,
     this.writerError,
     this.customerError,
+    this.warehouseError,
+    this.statusError,
     this.headerNotice,
   });
 
   final bool isValid;
   final String? writerError;
   final String? customerError;
+  final String? warehouseError;
+  final String? statusError;
   final String? headerNotice;
 }