update tender form
This commit is contained in:
@@ -0,0 +1,9 @@
|
||||
-- Convert leave start/end columns from DATE to TIMESTAMP(3)
|
||||
-- so that HOURLY leaves can store the actual start/end time of day.
|
||||
-- Without this, Postgres truncates the time and every hourly leave
|
||||
-- renders at the company timezone offset (e.g. 03:00) instead of the
|
||||
-- chosen time.
|
||||
|
||||
ALTER TABLE "leaves"
|
||||
ALTER COLUMN "startDate" TYPE TIMESTAMP(3) USING "startDate"::timestamp(3),
|
||||
ALTER COLUMN "endDate" TYPE TIMESTAMP(3) USING "endDate"::timestamp(3);
|
||||
@@ -81,11 +81,11 @@ router.post(
|
||||
[
|
||||
body('issuingBodyName').optional().trim(),
|
||||
body('title').optional().trim(),
|
||||
body('tenderNumber').optional().trim(),
|
||||
body('termsValue').optional().isNumeric(),
|
||||
body('bondValue').optional().isNumeric(),
|
||||
body('announcementDate').optional().isISO8601(),
|
||||
body('closingDate').optional().isISO8601(),
|
||||
body('tenderNumber').optional({ checkFalsy: true }).trim(),
|
||||
body('termsValue').optional({ checkFalsy: false }).isNumeric(),
|
||||
body('bondValue').optional({ checkFalsy: false }).isNumeric(),
|
||||
body('announcementDate').optional({ checkFalsy: true }).isISO8601(),
|
||||
body('closingDate').optional({ checkFalsy: true }).isISO8601(),
|
||||
],
|
||||
validate,
|
||||
tendersController.checkDuplicates
|
||||
@@ -101,14 +101,14 @@ router.post(
|
||||
'/',
|
||||
authorize('tenders', 'tenders', 'create'),
|
||||
[
|
||||
body('tenderNumber').notEmpty().trim(),
|
||||
body('tenderNumber').optional({ checkFalsy: true }).trim(),
|
||||
body('issueNumber').optional().trim(),
|
||||
body('issuingBodyName').notEmpty().trim(),
|
||||
body('title').notEmpty().trim(),
|
||||
body('termsValue').isNumeric(),
|
||||
body('bondValue').isNumeric(),
|
||||
body('announcementDate').isISO8601(),
|
||||
body('closingDate').isISO8601(),
|
||||
body('termsValue').optional({ checkFalsy: false }).isNumeric(),
|
||||
body('bondValue').optional({ checkFalsy: false }).isNumeric(),
|
||||
body('announcementDate').optional({ checkFalsy: true }).isISO8601(),
|
||||
body('closingDate').optional({ checkFalsy: true }).isISO8601(),
|
||||
body('source').notEmpty(),
|
||||
body('announcementType').notEmpty(),
|
||||
],
|
||||
|
||||
@@ -35,7 +35,7 @@ const DIRECTIVE_TYPE_VALUES = [
|
||||
export interface CreateTenderData {
|
||||
issuingBodyName: string;
|
||||
title: string;
|
||||
tenderNumber: string;
|
||||
tenderNumber?: string;
|
||||
issueNumber?: string;
|
||||
|
||||
termsValue: number;
|
||||
@@ -49,8 +49,8 @@ export interface CreateTenderData {
|
||||
siteVisitLocation?: string;
|
||||
termsPickupProvince?: string;
|
||||
|
||||
announcementDate: string;
|
||||
closingDate: string;
|
||||
announcementDate?: string;
|
||||
closingDate?: string;
|
||||
announcementLink?: string;
|
||||
|
||||
source: string;
|
||||
@@ -327,16 +327,33 @@ private getEffectiveTenderStatus(tender: {
|
||||
async create(data: CreateTenderData, userId: string): Promise<TenderWithDuplicates> {
|
||||
const possibleDuplicates = await this.findPossibleDuplicates(data);
|
||||
|
||||
// رقم المناقصة اختياري في الواجهة، لكن العمود في قاعدة البيانات مطلوب وفريد.
|
||||
// إذا تُرك فاضياً نولّد رقماً تلقائياً حتى لا يفشل الحفظ ولا نحتاج migration.
|
||||
let tenderNumber = data.tenderNumber?.trim() || '';
|
||||
if (tenderNumber) {
|
||||
const existing = await prisma.tender.findUnique({
|
||||
where: { tenderNumber: data.tenderNumber.trim() },
|
||||
where: { tenderNumber },
|
||||
});
|
||||
if (existing) {
|
||||
throw new AppError(400, 'Tender number already exists - رقم المناقصة موجود مسبقاً');
|
||||
}
|
||||
} else {
|
||||
// توليد رقم تلقائي فريد مع تفادي التعارض النادر
|
||||
tenderNumber = await this.generateTenderNumber();
|
||||
let guard = 0;
|
||||
while (await prisma.tender.findUnique({ where: { tenderNumber } })) {
|
||||
tenderNumber = `${await this.generateTenderNumber()}-${++guard}`;
|
||||
if (guard > 5) {
|
||||
tenderNumber = `TND-${Date.now()}`;
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
const tenderNumber = data.tenderNumber.trim();
|
||||
const announcementDate = new Date(data.announcementDate);
|
||||
const closingDate = new Date(data.closingDate);
|
||||
// تاريخ الإعلان/الإغلاق اختياريان في الواجهة، والعمودان مطلوبان في قاعدة البيانات.
|
||||
// إذا تُركا فاضيين نستخدم تاريخ اليوم افتراضياً.
|
||||
const announcementDate = data.announcementDate ? new Date(data.announcementDate) : new Date();
|
||||
const closingDate = data.closingDate ? new Date(data.closingDate) : new Date();
|
||||
|
||||
if (data.siteVisitRequired && !data.siteVisitLocation?.trim()) {
|
||||
throw new AppError(400, 'مكان زيارة الموقع مطلوب عند اختيار زيارة موقع إجبارية');
|
||||
@@ -357,7 +374,7 @@ private getEffectiveTenderStatus(tender: {
|
||||
issueNumber: data.issueNumber?.trim() || null,
|
||||
issuingBodyName: data.issuingBodyName.trim(),
|
||||
title: data.title.trim(),
|
||||
termsValue: data.termsValue,
|
||||
termsValue: Number(data.termsValue ?? 0),
|
||||
bondValue: Number(data.initialBondValue ?? data.bondValue ?? 0),
|
||||
announcementDate,
|
||||
closingDate,
|
||||
@@ -507,8 +524,16 @@ private getEffectiveTenderStatus(tender: {
|
||||
if (data.bondValue !== undefined || data.initialBondValue !== undefined) {
|
||||
updateData.bondValue = Number(data.initialBondValue ?? data.bondValue ?? existing.bondValue);
|
||||
}
|
||||
if (data.announcementDate !== undefined) updateData.announcementDate = new Date(data.announcementDate);
|
||||
if (data.closingDate !== undefined) updateData.closingDate = new Date(data.closingDate);
|
||||
// العمود مطلوب في قاعدة البيانات: نتجاهل القيمة الفاضية ونبقي على القيمة الحالية بدل كتابة null.
|
||||
if (data.tenderNumber !== undefined && data.tenderNumber?.trim()) {
|
||||
updateData.tenderNumber = data.tenderNumber.trim();
|
||||
}
|
||||
if (data.announcementDate !== undefined && data.announcementDate) {
|
||||
updateData.announcementDate = new Date(data.announcementDate);
|
||||
}
|
||||
if (data.closingDate !== undefined && data.closingDate) {
|
||||
updateData.closingDate = new Date(data.closingDate);
|
||||
}
|
||||
if (data.announcementLink !== undefined) updateData.announcementLink = data.announcementLink?.trim() || null;
|
||||
if (data.source !== undefined) updateData.source = data.source;
|
||||
if (data.sourceOther !== undefined) updateData.sourceOther = data.sourceOther?.trim() || null;
|
||||
|
||||
@@ -182,11 +182,8 @@ function TendersContent() {
|
||||
|
||||
const errors: Record<string, string> = {}
|
||||
|
||||
if (!formData.tenderNumber?.trim()) errors.tenderNumber = t('common.required')
|
||||
if (!formData.issuingBodyName?.trim()) errors.issuingBodyName = t('common.required')
|
||||
if (!formData.title?.trim()) errors.title = t('common.required')
|
||||
if (!formData.announcementDate) errors.announcementDate = t('common.required')
|
||||
if (!formData.closingDate) errors.closingDate = t('common.required')
|
||||
|
||||
if (Number(formData.initialBondValue || 0) < 0) {
|
||||
errors.initialBondValue = t('common.required')
|
||||
@@ -231,11 +228,8 @@ function TendersContent() {
|
||||
|
||||
const errors: Record<string, string> = {}
|
||||
|
||||
if (!formData.tenderNumber?.trim()) errors.tenderNumber = t('common.required')
|
||||
if (!formData.issuingBodyName?.trim()) errors.issuingBodyName = t('common.required')
|
||||
if (!formData.title?.trim()) errors.title = t('common.required')
|
||||
if (!formData.announcementDate) errors.announcementDate = t('common.required')
|
||||
if (!formData.closingDate) errors.closingDate = t('common.required')
|
||||
|
||||
if (Number(formData.initialBondValue || 0) < 0) {
|
||||
errors.initialBondValue = t('common.required')
|
||||
@@ -289,11 +283,11 @@ function TendersContent() {
|
||||
<div className="grid grid-cols-1 md:grid-cols-2 gap-4">
|
||||
<div>
|
||||
<label className="block text-sm font-medium text-gray-700 mb-1">
|
||||
{t('tenders.tenderNumber')} *
|
||||
{t('tenders.tenderNumber')}
|
||||
</label>
|
||||
<input
|
||||
type="text"
|
||||
value={formData.tenderNumber}
|
||||
value={formData.tenderNumber ?? ''}
|
||||
onChange={(e) => setFormData({ ...formData, tenderNumber: e.target.value })}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
@@ -354,9 +348,12 @@ function TendersContent() {
|
||||
<input
|
||||
type="number"
|
||||
min={0}
|
||||
value={formData.termsValue || ''}
|
||||
value={formData.termsValue ?? ''}
|
||||
onChange={(e) =>
|
||||
setFormData({ ...formData, termsValue: Number(e.target.value) || 0 })
|
||||
setFormData({
|
||||
...formData,
|
||||
termsValue: e.target.value === '' ? 0 : Number(e.target.value),
|
||||
})
|
||||
}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
@@ -369,14 +366,11 @@ function TendersContent() {
|
||||
<input
|
||||
type="number"
|
||||
min={0}
|
||||
value={formData.initialBondValue || ''}
|
||||
onChange={(e) =>
|
||||
setFormData({
|
||||
...formData,
|
||||
initialBondValue: Number(e.target.value) || 0,
|
||||
bondValue: Number(e.target.value) || 0,
|
||||
})
|
||||
}
|
||||
value={formData.initialBondValue ?? ''}
|
||||
onChange={(e) => {
|
||||
const v = e.target.value === '' ? 0 : Number(e.target.value)
|
||||
setFormData({ ...formData, initialBondValue: v, bondValue: v })
|
||||
}}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
{formErrors.initialBondValue && (
|
||||
@@ -391,9 +385,12 @@ function TendersContent() {
|
||||
<input
|
||||
type="number"
|
||||
min={0}
|
||||
value={formData.finalBondValue || ''}
|
||||
value={formData.finalBondValue ?? ''}
|
||||
onChange={(e) =>
|
||||
setFormData({ ...formData, finalBondValue: Number(e.target.value) || 0 })
|
||||
setFormData({
|
||||
...formData,
|
||||
finalBondValue: e.target.value === '' ? 0 : Number(e.target.value),
|
||||
})
|
||||
}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
@@ -418,11 +415,11 @@ function TendersContent() {
|
||||
<div className="grid grid-cols-1 md:grid-cols-2 gap-4">
|
||||
<div>
|
||||
<label className="block text-sm font-medium text-gray-700 mb-1">
|
||||
{t('tenders.announcementDate')} *
|
||||
{t('tenders.announcementDate')}
|
||||
</label>
|
||||
<input
|
||||
type="date"
|
||||
value={formData.announcementDate}
|
||||
value={formData.announcementDate ?? ''}
|
||||
onChange={(e) => setFormData({ ...formData, announcementDate: e.target.value })}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
@@ -433,11 +430,11 @@ function TendersContent() {
|
||||
|
||||
<div>
|
||||
<label className="block text-sm font-medium text-gray-700 mb-1">
|
||||
{t('tenders.closingDate')} *
|
||||
{t('tenders.closingDate')}
|
||||
</label>
|
||||
<input
|
||||
type="date"
|
||||
value={formData.closingDate}
|
||||
value={formData.closingDate ?? ''}
|
||||
onChange={(e) => setFormData({ ...formData, closingDate: e.target.value })}
|
||||
className="w-full px-3 py-2 border rounded-lg"
|
||||
/>
|
||||
|
||||
@@ -2,7 +2,7 @@ import { api } from '../api'
|
||||
|
||||
export interface Tender {
|
||||
id: string
|
||||
tenderNumber: string
|
||||
tenderNumber?: string | null
|
||||
issueNumber?: string | null
|
||||
issuingBodyName: string
|
||||
title: string
|
||||
@@ -18,8 +18,8 @@ export interface Tender {
|
||||
siteVisitLocation?: string | null
|
||||
termsPickupProvince?: string | null
|
||||
|
||||
announcementDate: string
|
||||
closingDate: string
|
||||
announcementDate?: string | null
|
||||
closingDate?: string | null
|
||||
announcementLink?: string
|
||||
source: string
|
||||
sourceOther?: string
|
||||
@@ -57,7 +57,7 @@ export interface TenderDirective {
|
||||
}
|
||||
|
||||
export interface CreateTenderData {
|
||||
tenderNumber: string
|
||||
tenderNumber?: string | null
|
||||
issueNumber?: string
|
||||
issuingBodyName: string
|
||||
title: string
|
||||
@@ -73,8 +73,8 @@ export interface CreateTenderData {
|
||||
siteVisitLocation?: string
|
||||
termsPickupProvince?: string
|
||||
|
||||
announcementDate: string
|
||||
closingDate: string
|
||||
announcementDate?: string | null
|
||||
closingDate?: string | null
|
||||
announcementLink?: string
|
||||
source: string
|
||||
sourceOther?: string
|
||||
|
||||
Reference in New Issue
Block a user