fixed wrong list display due to date format , and added tds calculation in the payment proceed sheet

This commit is contained in:
Vaibhav Surve 2025-11-11 17:03:06 +05:30
parent af5bfcaa59
commit cffa4456b9
2 changed files with 188 additions and 110 deletions

View File

@ -1,7 +1,6 @@
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:get/get.dart'; import 'package:get/get.dart';
import 'package:intl/intl.dart'; import 'package:intl/intl.dart';
import 'package:marco/controller/finance/payment_request_detail_controller.dart'; import 'package:marco/controller/finance/payment_request_detail_controller.dart';
import 'package:marco/helpers/widgets/my_spacing.dart'; import 'package:marco/helpers/widgets/my_spacing.dart';
import 'package:marco/helpers/widgets/my_text.dart'; import 'package:marco/helpers/widgets/my_text.dart';
@ -24,10 +23,10 @@ class UpdatePaymentRequestWithReimbursement extends StatefulWidget {
@override @override
State<UpdatePaymentRequestWithReimbursement> createState() => State<UpdatePaymentRequestWithReimbursement> createState() =>
_UpdatePaymentRequestWithReimbursement(); _UpdatePaymentRequestWithReimbursementState();
} }
class _UpdatePaymentRequestWithReimbursement class _UpdatePaymentRequestWithReimbursementState
extends State<UpdatePaymentRequestWithReimbursement> { extends State<UpdatePaymentRequestWithReimbursement> {
final PaymentRequestDetailController controller = final PaymentRequestDetailController controller =
Get.find<PaymentRequestDetailController>(); Get.find<PaymentRequestDetailController>();
@ -37,7 +36,18 @@ class _UpdatePaymentRequestWithReimbursement
final TextEditingController tdsCtrl = TextEditingController(text: '0'); final TextEditingController tdsCtrl = TextEditingController(text: '0');
final TextEditingController baseAmountCtrl = TextEditingController(); final TextEditingController baseAmountCtrl = TextEditingController();
final TextEditingController taxAmountCtrl = TextEditingController(); final TextEditingController taxAmountCtrl = TextEditingController();
final RxString dateStr = ''.obs; final RxString dateStr = ''.obs;
final RxDouble tdsAmount = 0.0.obs;
final RxDouble netPayable = 0.0.obs;
@override
void initState() {
super.initState();
baseAmountCtrl.addListener(_recalculateTds);
taxAmountCtrl.addListener(_recalculateTds);
tdsCtrl.addListener(_recalculateTds);
}
@override @override
void dispose() { void dispose() {
@ -49,7 +59,21 @@ class _UpdatePaymentRequestWithReimbursement
super.dispose(); super.dispose();
} }
/// Employee selection bottom sheet void _recalculateTds() {
final double base = double.tryParse(baseAmountCtrl.text.trim()) ?? 0.0;
final double gst = double.tryParse(taxAmountCtrl.text.trim()) ?? 0.0;
final double tdsPercent = double.tryParse(tdsCtrl.text.trim()) ?? 0.0;
final double calculatedTds = (base * tdsPercent) / 100;
final double roundedTds = double.parse(calculatedTds.toStringAsFixed(2));
final double net = (base + gst) - roundedTds;
final double roundedNet = double.parse(net.toStringAsFixed(2));
tdsAmount.value = roundedTds;
netPayable.value = roundedNet;
}
void _showEmployeeList() async { void _showEmployeeList() async {
await showModalBottomSheet( await showModalBottomSheet(
context: context, context: context,
@ -67,7 +91,6 @@ class _UpdatePaymentRequestWithReimbursement
), ),
); );
// Optional cleanup
controller.employeeSearchController.clear(); controller.employeeSearchController.clear();
controller.employeeSearchResults.clear(); controller.employeeSearchResults.clear();
} }
@ -94,6 +117,29 @@ class _UpdatePaymentRequestWithReimbursement
); );
} }
Widget _readOnlyValueBox(String label, String value, Color color) {
return Container(
width: double.infinity,
padding: const EdgeInsets.symmetric(vertical: 14, horizontal: 16),
decoration: BoxDecoration(
color: color.withOpacity(0.1),
border: Border.all(color: color.withOpacity(0.3)),
borderRadius: BorderRadius.circular(12),
),
child: Row(
mainAxisAlignment: MainAxisAlignment.spaceBetween,
children: [
MyText.labelMedium(label),
MyText.bodyMedium(
"$value",
color: color,
fontWeight: 700,
),
],
),
);
}
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return Obx(() { return Obx(() {
@ -105,7 +151,6 @@ class _UpdatePaymentRequestWithReimbursement
Navigator.pop(context); Navigator.pop(context);
}, },
onSubmit: () async { onSubmit: () async {
// Mandatory fields validation
if (commentCtrl.text.trim().isEmpty || if (commentCtrl.text.trim().isEmpty ||
txnCtrl.text.trim().isEmpty || txnCtrl.text.trim().isEmpty ||
dateStr.value.isEmpty || dateStr.value.isEmpty ||
@ -120,27 +165,24 @@ class _UpdatePaymentRequestWithReimbursement
} }
try { try {
// Parse inputs
final parsedDate = final parsedDate =
DateFormat('dd-MM-yyyy').parse(dateStr.value, true); DateFormat('dd-MM-yyyy').parse(dateStr.value, true);
final baseAmount = double.tryParse(baseAmountCtrl.text.trim()) ?? 0; final baseAmt = double.tryParse(baseAmountCtrl.text.trim()) ?? 0;
final taxAmount = double.tryParse(taxAmountCtrl.text.trim()) ?? 0; final taxAmt = double.tryParse(taxAmountCtrl.text.trim()) ?? 0;
final tdsPercentage = final tdsPercentage =
tdsCtrl.text.trim().isEmpty ? null : tdsCtrl.text.trim(); tdsCtrl.text.trim().isEmpty ? null : tdsCtrl.text.trim();
// Call API
final success = await controller.updatePaymentRequestStatus( final success = await controller.updatePaymentRequestStatus(
statusId: widget.statusId, statusId: widget.statusId,
comment: commentCtrl.text.trim(), comment: commentCtrl.text.trim(),
paidTransactionId: txnCtrl.text.trim(), paidTransactionId: txnCtrl.text.trim(),
paidById: controller.selectedReimbursedBy.value?.id, paidById: controller.selectedReimbursedBy.value?.id,
paidAt: parsedDate, paidAt: parsedDate,
baseAmount: baseAmount, baseAmount: baseAmt,
taxAmount: taxAmount, taxAmount: taxAmt,
tdsPercentage: tdsPercentage, tdsPercentage: tdsPercentage,
); );
// Show snackbar
showAppSnackbar( showAppSnackbar(
title: success ? 'Success' : 'Error', title: success ? 'Success' : 'Error',
message: success message: success
@ -150,12 +192,11 @@ class _UpdatePaymentRequestWithReimbursement
); );
if (success) { if (success) {
// Ensure bottom sheet closes and callback is called widget.onClose();
widget.onClose(); // optional callback for parent refresh
if (Navigator.canPop(context)) { if (Navigator.canPop(context)) {
Navigator.pop(context); Navigator.pop(context);
} else { } else {
Get.close(1); // fallback if Navigator can't pop Get.close(1);
} }
} }
} catch (e, st) { } catch (e, st) {
@ -167,6 +208,7 @@ class _UpdatePaymentRequestWithReimbursement
); );
} }
}, },
child: SingleChildScrollView(
child: Column( child: Column(
crossAxisAlignment: CrossAxisAlignment.start, crossAxisAlignment: CrossAxisAlignment.start,
children: [ children: [
@ -177,21 +219,17 @@ class _UpdatePaymentRequestWithReimbursement
decoration: _inputDecoration("Enter transaction ID"), decoration: _inputDecoration("Enter transaction ID"),
), ),
MySpacing.height(16), MySpacing.height(16),
MyText.labelMedium("Transaction Date*"), MyText.labelMedium("Transaction Date*"),
MySpacing.height(8), MySpacing.height(8),
GestureDetector( GestureDetector(
onTap: () async { onTap: () async {
final today = DateTime.now();
final firstDate = DateTime(2020);
final lastDate = today;
final picked = await showDatePicker( final picked = await showDatePicker(
context: context, context: context,
initialDate: today, initialDate: DateTime.now(),
firstDate: firstDate, firstDate: DateTime(2020),
lastDate: lastDate, lastDate: DateTime.now(),
); );
if (picked != null) { if (picked != null) {
dateStr.value = DateFormat('dd-MM-yyyy').format(picked); dateStr.value = DateFormat('dd-MM-yyyy').format(picked);
} }
@ -206,6 +244,7 @@ class _UpdatePaymentRequestWithReimbursement
), ),
), ),
MySpacing.height(16), MySpacing.height(16),
MyText.labelMedium("Paid By (Optional)"), MyText.labelMedium("Paid By (Optional)"),
MySpacing.height(8), MySpacing.height(8),
GestureDetector( GestureDetector(
@ -233,15 +272,8 @@ class _UpdatePaymentRequestWithReimbursement
), ),
), ),
MySpacing.height(16), MySpacing.height(16),
MyText.labelMedium("TDS Percentage (Optional)"),
MySpacing.height(8), MyText.labelMedium("Base Amount"),
TextField(
controller: tdsCtrl,
keyboardType: TextInputType.number,
decoration: _inputDecoration("Enter TDS Percentage"),
),
MySpacing.height(16),
MyText.labelMedium("Base Amount*"),
MySpacing.height(8), MySpacing.height(8),
TextField( TextField(
controller: baseAmountCtrl, controller: baseAmountCtrl,
@ -249,22 +281,69 @@ class _UpdatePaymentRequestWithReimbursement
decoration: _inputDecoration("Enter Base Amount"), decoration: _inputDecoration("Enter Base Amount"),
), ),
MySpacing.height(16), MySpacing.height(16),
MyText.labelMedium("Tax Amount*"),
MyText.labelMedium("GST Amount"),
MySpacing.height(8), MySpacing.height(8),
TextField( TextField(
controller: taxAmountCtrl, controller: taxAmountCtrl,
keyboardType: TextInputType.number, keyboardType: TextInputType.number,
decoration: _inputDecoration("Enter Tax Amount"), decoration: _inputDecoration("Enter GST Amount"),
), ),
MySpacing.height(16), MySpacing.height(16),
MyText.labelMedium("TDS Percent"),
MySpacing.height(8),
TextField(
controller: tdsCtrl,
keyboardType: TextInputType.number,
decoration: _inputDecoration("Enter TDS Percent").copyWith(
suffixIcon: Padding(
padding: const EdgeInsets.only(right: 12),
child: Icon(Icons.percent,
size: 20, color: Colors.grey.shade600),
),
suffixIconConstraints:
const BoxConstraints(minWidth: 0, minHeight: 0),
),
onChanged: (_) => _recalculateTds(),
),
MySpacing.height(4),
MyText.bodySmall(
"TDS is applied on base amount only.",
color: Colors.grey.shade600,
),
MySpacing.height(16),
// Proper display section for TDS and Net Payable
Obx(() => Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
_readOnlyValueBox(
"TDS Amount",
tdsAmount.value.toStringAsFixed(2),
Colors.orange,
),
MySpacing.height(12),
_readOnlyValueBox(
"Net Payable",
netPayable.value.toStringAsFixed(2),
Colors.green,
),
],
)),
MySpacing.height(20),
MyText.labelMedium("Comment*"), MyText.labelMedium("Comment*"),
MySpacing.height(8), MySpacing.height(8),
TextField( TextField(
controller: commentCtrl, controller: commentCtrl,
maxLines: 2,
decoration: _inputDecoration("Enter comment"), decoration: _inputDecoration("Enter comment"),
), ),
], ],
), ),
),
); );
}); });
} }

View File

@ -76,11 +76,10 @@ class _PaymentRequestMainScreenState extends State<PaymentRequestMainScreen>
return isHistory return isHistory
? filtered ? filtered
.where((e) => e.dueDate.isBefore(DateTime(now.year, now.month))) .where((e) => e.dueDate.isBefore(DateTime(now.year, now.month, 1)))
.toList() .toList()
: filtered : filtered
.where((e) => .where((e) => e.dueDate.isAfter(DateTime(now.year, now.month, 0)))
e.dueDate.month == now.month && e.dueDate.year == now.year)
.toList(); .toList();
} }