From 29029625f2c53d11105512b4b8ba164d7a05fd5c Mon Sep 17 00:00:00 2001 From: Vikrant Date: Tue, 21 Apr 2026 14:51:53 +0530 Subject: [PATCH 01/12] fix: update thisMonthYear assignment to use variable directly --- controllers/fund.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/controllers/fund.js b/controllers/fund.js index 92c755f..490029d 100644 --- a/controllers/fund.js +++ b/controllers/fund.js @@ -127,7 +127,7 @@ module.exports.fund = catchAsync(async (req, res) => { total, view, balance, - thisMonthYear: `${month}-${year}`, + thisMonthYear, thisMonthData, stuThisMonth, archiveFeesThisMonth, From 1bf3c2785fdcfdd3d4fbb02147a50a3098440d80 Mon Sep 17 00:00:00 2001 From: Vikrant Date: Mon, 4 May 2026 07:14:50 +0530 Subject: [PATCH 02/12] feat: enhance fund report with total students and paid students count --- controllers/fund.js | 22 +++++++++++++++++----- public/css/fund.css | 17 ++++++++++------- public/js/previousReports.js | 2 +- views/listings/fund.ejs | 23 +++++++++++++++++++---- 4 files changed, 47 insertions(+), 17 deletions(-) diff --git a/controllers/fund.js b/controllers/fund.js index 490029d..7a1c6f9 100644 --- a/controllers/fund.js +++ b/controllers/fund.js @@ -2,10 +2,6 @@ const Student = require("../models/students"); const catchAsync = require("../utils/catchAsync"); const MonthlyReport = require("../models/monthlyReport"); const archivedStudent = require("../models/archivedStudent"); -let thisMonthYear = new Date().toLocaleString("en-US", { - month: "short", - year: "numeric", -}); module.exports.fund = catchAsync(async (req, res) => { const now = new Date(); const startOfMonth = new Date(now.getFullYear(), now.getMonth(), 1); @@ -32,6 +28,19 @@ module.exports.fund = catchAsync(async (req, res) => { }, }, ], + totalStudents: [{ $count: "count" }], + paidStudents: [ + { + $match: { + feesHistory: { + $elemMatch: { + paidDate: { $gte: startOfMonth, $lt: endOfMonth }, + }, + }, + }, + }, + { $count: "count" }, + ], feesThisMonth: [ { $unwind: "$feesHistory" }, { @@ -93,6 +102,8 @@ module.exports.fund = catchAsync(async (req, res) => { // ✅ Extract data from facet const data = studentResult[0]; const totalDue = data.totalDue[0]?.totalDue || 0; + const totalStudents = data.totalStudents[0]?.count || 0; + const paidStudents = data.paidStudents[0]?.count || 0; const feesThisMonth = data.feesThisMonth; const stuThisMonth = data.studentsThisMonth; const todayDate = new Date().toISOString().split("T")[0]; @@ -127,10 +138,11 @@ module.exports.fund = catchAsync(async (req, res) => { total, view, balance, - thisMonthYear, thisMonthData, stuThisMonth, archiveFeesThisMonth, + totalStudents, + paidStudents, }); }); module.exports.addExpense = catchAsync(async (req, res) => { diff --git a/public/css/fund.css b/public/css/fund.css index 69fc1d2..1030b0b 100644 --- a/public/css/fund.css +++ b/public/css/fund.css @@ -46,11 +46,12 @@ body { .stat { width: 100%; - padding: 0.6rem; color: white; - border-bottom: 1px dashed #ddd; display: flex; - justify-content: space-around; + justify-content: start; +} +.stats{ + justify-content: center; } .stat .total { border-radius: 30px; @@ -69,7 +70,8 @@ body { .stat div { margin: 0; - font-size: 14px; + font-size: 12px; + margin-left: 10px; } .new-stu { @@ -102,7 +104,7 @@ body { display: flex; justify-content: space-between; padding: 8px 0; - border-bottom: 1px dashed #ddd; + border-bottom: 1px dashed #515151; color: white; } .history { @@ -116,8 +118,9 @@ body { margin-bottom: 10px; } .feeHistory { + margin-top: .8rem; width: 23vw; - height: 55vh; + max-height: 55vh; overflow-y: scroll; } .feeHistory p{ @@ -127,7 +130,7 @@ body { color: white; text-decoration: none; font-size: 1.4rem; - border-bottom: 1px dashed #ddd; + /* border-bottom: 1px dashed #515151; */ padding: 5px 0; cursor: pointer; } diff --git a/public/js/previousReports.js b/public/js/previousReports.js index 1224db4..d7a901a 100644 --- a/public/js/previousReports.js +++ b/public/js/previousReports.js @@ -36,7 +36,7 @@ function renderReportCard(data) {

${monthTitle}

-
+
${data.totalEarning.toLocaleString("en-IN")}₹Monthly Reports
-

<%= thisMonthYear %>

+

<%= new Date(thisMonthData.year, thisMonthData.month -1).toLocaleString( + "en-US",{ month: "short", year: "numeric" })%> +

<%=balance.toLocaleString('en-IN')%> ₹
Fund Balance
+
+ <%= total.toLocaleString('en-IN') %> ₹
Total Expenses +
+
+
<%=thisMonthData.totalEarning.toLocaleString('en-IN')%> ₹
Total Revenue
+
+ <%= totalDue.toLocaleString('en-IN') %> + ₹
Total Due +
@@ -33,6 +47,7 @@
+ <% if (thisMonthData && thisMonthData.expenses.length) { %>

Expense Details

    <%for(let exp of thisMonthData.expenses){%> @@ -47,8 +62,7 @@ <%}%>
-

Total Expenses : <%= total.toLocaleString('en-IN') %> ₹

-

Total Due : <%= totalDue.toLocaleString('en-IN') %> ₹

+ <% } %>
@@ -93,13 +107,14 @@

-  Fees This Month +  Fees History

<% if (view === "month") { %> Show Last 3 Days <% } else { %> Show Full Month <% } %> + Paid Students: <%=paidStudents%> / <%=totalStudents%>
<% if (feesThisMonth && feesThisMonth.length) { %> <% feesThisMonth.forEach(fee => { %> From d451954a96ae29a5400b4ebcf186d9bd85018683 Mon Sep 17 00:00:00 2001 From: Vikrant Date: Tue, 5 May 2026 19:36:13 +0530 Subject: [PATCH 03/12] feat: update fund report to include fees history and improve data retrieval --- controllers/fund.js | 22 ++++++------ public/css/fund.css | 1 + views/listings/fund.ejs | 76 ++++++++++++++++++++--------------------- 3 files changed, 51 insertions(+), 48 deletions(-) diff --git a/controllers/fund.js b/controllers/fund.js index 7a1c6f9..8902eff 100644 --- a/controllers/fund.js +++ b/controllers/fund.js @@ -109,13 +109,10 @@ module.exports.fund = catchAsync(async (req, res) => { const todayDate = new Date().toISOString().split("T")[0]; const month = now.getMonth() + 1; const year = now.getFullYear(); - let thisMonthData = await MonthlyReport.findOne({ - owner: req.user._id, - month, - year, - }).lean(); - if (!thisMonthData) { - thisMonthData = await MonthlyReport.create({ + let thisMonthData = await MonthlyReport.findOneAndUpdate( + { owner: req.user._id, month, year }, + { + $setOnInsert: { owner: req.user._id, month, year, @@ -125,11 +122,16 @@ module.exports.fund = catchAsync(async (req, res) => { newStudents: 0, studentsLeft: 0, createdAt: new Date(), - }); + }, + }, + { + upsert: true, + new: true, + lean: true, } +); // ✅ Calculate expenses - let total = 0; - thisMonthData.expenses.forEach((e) => (total += e.amount)); + const total = thisMonthData.expenses.reduce((sum, e) => sum + e.amount, 0); const balance = Number(thisMonthData.totalEarning) - total; res.render("listings/fund", { totalDue, diff --git a/public/css/fund.css b/public/css/fund.css index 1030b0b..c3e7d8f 100644 --- a/public/css/fund.css +++ b/public/css/fund.css @@ -261,6 +261,7 @@ input::-webkit-inner-spin-button { } .history { padding: 3.5vw; + margin: 3vw 0 0 0 ; } .fee-entry { font-size: 1rem; diff --git a/views/listings/fund.ejs b/views/listings/fund.ejs index a071d69..f9549b3 100644 --- a/views/listings/fund.ejs +++ b/views/listings/fund.ejs @@ -65,6 +65,44 @@ <% } %>
+
+

+  Fees History +

+ <% if (view === "month") { %> + Show Last 3 Days + <% } else { %> + Show Full Month + <% } %> + Paid Students: <%=paidStudents%> / <%=totalStudents%> +
+ <% if (feesThisMonth && feesThisMonth.length) { %> <% + feesThisMonth.forEach(fee => { %> +
  • + <%= fee.feesHistory.amount %>₹ — <%= fee.name %> / <%= fee.grade %> + On : <%= new + Date(fee.feesHistory.paidDate) .toLocaleDateString('en-GB', { + weekday: 'short', day: 'numeric', month: 'short' }) + .replace(/,/g, '') %> <%if(fee.feesHistory.note){%>
    Note + : <%= fee.feesHistory.note %> <%}%>
    +
    • + <% }) %> <% } else { %> +

    No fees recorded this month.

    + <% } %> + <% if (archiveFeesThisMonth && archiveFeesThisMonth.length) { %> +

    Archive fees record

    + <% archiveFeesThisMonth.forEach(fee => { %> +
  • + <%= fee.feesHistory.amount %>₹ — <%= fee.name %> / <%= fee.grade %> + On : <%= new + Date(fee.feesHistory.paidDate) .toLocaleDateString('en-GB', { + weekday: 'short', day: 'numeric', month: 'short' }) + .replace(/,/g, '') %> <%if(fee.feesHistory.note){%>
    Note + : <%= fee.feesHistory.note %> <%}%>
    +
    • + <% })} %> +
    +

    Add Expenses

    -
    -

    -  Fees History -

    - <% if (view === "month") { %> - Show Last 3 Days - <% } else { %> - Show Full Month - <% } %> - Paid Students: <%=paidStudents%> / <%=totalStudents%> -
    - <% if (feesThisMonth && feesThisMonth.length) { %> <% - feesThisMonth.forEach(fee => { %> -
  • - <%= fee.feesHistory.amount %>₹ — <%= fee.name %> / <%= fee.grade %> - On : <%= new - Date(fee.feesHistory.paidDate) .toLocaleDateString('en-GB', { - weekday: 'short', day: 'numeric', month: 'short' }) - .replace(/,/g, '') %> <%if(fee.feesHistory.note){%>
    Note - : <%= fee.feesHistory.note %> <%}%>
    -
    • - <% }) %> <% } else { %> -

    No fees recorded this month.

    - <% } %> - <% if (archiveFeesThisMonth && archiveFeesThisMonth.length) { %> -

    Archive fees record

    - <% archiveFeesThisMonth.forEach(fee => { %> -
  • - <%= fee.feesHistory.amount %>₹ — <%= fee.name %> / <%= fee.grade %> - On : <%= new - Date(fee.feesHistory.paidDate) .toLocaleDateString('en-GB', { - weekday: 'short', day: 'numeric', month: 'short' }) - .replace(/,/g, '') %> <%if(fee.feesHistory.note){%>
    Note - : <%= fee.feesHistory.note %> <%}%>
    -
    • - <% })} %> -
    -
    From 4896df2dc310aef547933795d0d2c57c87dae90c Mon Sep 17 00:00:00 2001 From: Vikrant Date: Wed, 13 May 2026 09:27:50 +0530 Subject: [PATCH 04/12] feat: refactor date filtering and fees aggregation in fund report --- controllers/fund.js | 48 ++++++++++++++++++++------------------------- 1 file changed, 21 insertions(+), 27 deletions(-) diff --git a/controllers/fund.js b/controllers/fund.js index 8902eff..7c96418 100644 --- a/controllers/fund.js +++ b/controllers/fund.js @@ -7,14 +7,25 @@ module.exports.fund = catchAsync(async (req, res) => { const startOfMonth = new Date(now.getFullYear(), now.getMonth(), 1); const endOfMonth = new Date(now.getFullYear(), now.getMonth() + 1, 1); const view = req.query.view; - let dateFilter; - if (view === "month") { - dateFilter = { $gte: startOfMonth, $lt: endOfMonth }; - } else { - const threeDaysAgo = new Date(); - threeDaysAgo.setDate(threeDaysAgo.getDate() - 3); - dateFilter = { $gte: threeDaysAgo }; - } + const dateFilter = + view === "month" + ? { "feesHistory.paidDate": { $gte: startOfMonth, $lt: endOfMonth } } + : { "feesHistory.paidDate": { $exists: true, $ne: null } }; + const feesPipeline = [ + { $unwind: "$feesHistory" }, + { $match: dateFilter }, + { $sort: { "feesHistory.paidDate": -1 } }, + ...(view !== "month" ? [{ $limit: 3 }] : []), + { + $project: { + name: 1, + grade: 1, + "feesHistory.amount": 1, + "feesHistory.paidDate": 1, + "feesHistory.note": 1, + }, + }, + ]; const [studentResult, archiveFeesThisMonth] = await Promise.all([ Student.aggregate([ { $match: { owner: req.user._id } }, @@ -41,24 +52,7 @@ module.exports.fund = catchAsync(async (req, res) => { }, { $count: "count" }, ], - feesThisMonth: [ - { $unwind: "$feesHistory" }, - { - $match: { - "feesHistory.paidDate": dateFilter, - }, - }, - { $sort: { "feesHistory.paidDate": -1 } }, - { - $project: { - name: 1, - grade: 1, - "feesHistory.amount": 1, - "feesHistory.paidDate": 1, - "feesHistory.note": 1, - }, - }, - ], + feesThisMonth: feesPipeline, studentsThisMonth: [ { $match: { @@ -84,7 +78,7 @@ module.exports.fund = catchAsync(async (req, res) => { { $unwind: "$feesHistory" }, { $match: { - "feesHistory.paidDate": dateFilter, + "feesHistory.paidDate": { $gte: startOfMonth, $lt: endOfMonth }, }, }, { $sort: { "feesHistory.paidDate": -1 } }, From 4f7034fb61fa212db1f6737c0938e7c3e858cad8 Mon Sep 17 00:00:00 2001 From: Vikrant Date: Wed, 13 May 2026 09:35:14 +0530 Subject: [PATCH 05/12] feat: enhance fees aggregation logic in fund report for improved monthly and historical data retrieval --- controllers/fund.js | 78 ++++++++++++++++++++++++++++++++++++--------- 1 file changed, 63 insertions(+), 15 deletions(-) diff --git a/controllers/fund.js b/controllers/fund.js index 7c96418..157ce39 100644 --- a/controllers/fund.js +++ b/controllers/fund.js @@ -11,21 +11,69 @@ module.exports.fund = catchAsync(async (req, res) => { view === "month" ? { "feesHistory.paidDate": { $gte: startOfMonth, $lt: endOfMonth } } : { "feesHistory.paidDate": { $exists: true, $ne: null } }; - const feesPipeline = [ - { $unwind: "$feesHistory" }, - { $match: dateFilter }, - { $sort: { "feesHistory.paidDate": -1 } }, - ...(view !== "month" ? [{ $limit: 3 }] : []), - { - $project: { - name: 1, - grade: 1, - "feesHistory.amount": 1, - "feesHistory.paidDate": 1, - "feesHistory.note": 1, - }, - }, - ]; + const feesPipeline = + view === "month" + ? [ + { $unwind: "$feesHistory" }, + { + $match: { + "feesHistory.paidDate": { + $gte: startOfMonth, + $lt: endOfMonth, + }, + }, + }, + { $sort: { "feesHistory.paidDate": -1 } }, + { + $project: { + name: 1, + grade: 1, + "feesHistory.amount": 1, + "feesHistory.paidDate": 1, + "feesHistory.note": 1, + }, + }, + ] + : [ + { $unwind: "$feesHistory" }, + { + $match: { + "feesHistory.paidDate": { $exists: true, $ne: null }, + }, + }, + { $sort: { "feesHistory.paidDate": -1 } }, + { + $group: { + _id: { + $dateToString: { + format: "%Y-%m-%d", + date: "$feesHistory.paidDate", + }, + }, + records: { + $push: { + name: "$name", + grade: "$grade", + amount: "$feesHistory.amount", + paidDate: "$feesHistory.paidDate", + note: "$feesHistory.note", + }, + }, + }, + }, + { $sort: { _id: -1 } }, + { $limit: 3 }, + { $unwind: "$records" }, + { + $project: { + name: "$records.name", + grade: "$records.grade", + "feesHistory.amount": "$records.amount", + "feesHistory.paidDate": "$records.paidDate", + "feesHistory.note": "$records.note", + }, + }, + ]; const [studentResult, archiveFeesThisMonth] = await Promise.all([ Student.aggregate([ { $match: { owner: req.user._id } }, From c18235c37b6d26b90aa3bc28601d45a636a7e10d Mon Sep 17 00:00:00 2001 From: Vikrant Date: Wed, 13 May 2026 09:45:03 +0530 Subject: [PATCH 06/12] feat: add text transformation to fee entry for improved readability --- public/css/fund.css | 1 + 1 file changed, 1 insertion(+) diff --git a/public/css/fund.css b/public/css/fund.css index c3e7d8f..7f090ea 100644 --- a/public/css/fund.css +++ b/public/css/fund.css @@ -129,6 +129,7 @@ body { .fee-entry { color: white; text-decoration: none; + text-transform: capitalize; font-size: 1.4rem; /* border-bottom: 1px dashed #515151; */ padding: 5px 0; From 0821af6dbc8e9473a305c231b44e6fc7767ff053 Mon Sep 17 00:00:00 2001 From: vikrant Date: Sun, 14 Jun 2026 09:17:29 +0530 Subject: [PATCH 07/12] feat: update dotenv version and improve student details layout in archive view --- package-lock.json | 178 ++++++------------------------------- package.json | 2 +- views/listings/archive.ejs | 7 +- 3 files changed, 33 insertions(+), 154 deletions(-) diff --git a/package-lock.json b/package-lock.json index 8bf35d4..9f8b0b9 100644 --- a/package-lock.json +++ b/package-lock.json @@ -13,7 +13,7 @@ "connect-flash": "^0.1.1", "connect-mongo": "^5.1.0", "cookie-parser": "^1.4.7", - "dotenv": "^17.2.3", + "dotenv": "^17.4.2", "ejs": "^3.1.10", "ejs-mate": "^4.0.0", "express": "^5.1.0", @@ -164,9 +164,10 @@ } }, "node_modules/bn.js": { - "version": "4.12.2", - "resolved": "https://registry.npmjs.org/bn.js/-/bn.js-4.12.2.tgz", - "integrity": "sha512-n4DSx829VRTRByMRGdjQ9iqsN0Bh4OolPsFnaZBLcbi8iXcB+kJ9s7EnRt4wILZNV3kPLHkRVfOc/HvhC3ovDw==" + "version": "4.12.3", + "resolved": "https://registry.npmjs.org/bn.js/-/bn.js-4.12.3.tgz", + "integrity": "sha512-fGTi3gxV/23FTYdAoUtLYp6qySe2KE3teyZitipKNRuVYcBkoP/bB3guXN/XVKUe9mxCHXnc9C4ocyz8OmgN0g==", + "license": "MIT" }, "node_modules/body-parser": { "version": "2.2.1", @@ -193,9 +194,9 @@ } }, "node_modules/brace-expansion": { - "version": "2.0.2", - "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-2.0.2.tgz", - "integrity": "sha512-Jt0vHyM+jmUBqojB7E1NIYadt0vI0Qxjxd2TErW94wDz+E2LAm5vKMXXwg6ZZBTHPuUlDgQHKXvjGBdfcF1ZDQ==", + "version": "2.1.1", + "resolved": "https://registry.npmjs.org/brace-expansion/-/brace-expansion-2.1.1.tgz", + "integrity": "sha512-WR1cURNjuvBLMZBMbqM0UoE+WAfdUcEV1ccD8PVBVOI+Z3ND4+SZbN8RsfT2bMuG1qwz5RFvPukSZm5fF2D5eA==", "license": "MIT", "dependencies": { "balanced-match": "^1.0.0" @@ -364,9 +365,10 @@ } }, "node_modules/dotenv": { - "version": "17.2.3", - "resolved": "https://registry.npmjs.org/dotenv/-/dotenv-17.2.3.tgz", - "integrity": "sha512-JVUnt+DUIzu87TABbhPmNfVdBDt18BLOWjMUFJMSi/Qqg7NTYtabbvSNJGOJ7afbRuv9D/lngizHtP7QyLQ+9w==", + "version": "17.4.2", + "resolved": "https://registry.npmjs.org/dotenv/-/dotenv-17.4.2.tgz", + "integrity": "sha512-nI4U3TottKAcAD9LLud4Cb7b2QztQMUEfHbvhTH09bqXTxnSie8WnjPALV/WMCrJZ6UV/qHJ6L03OqO3LcdYZw==", + "license": "BSD-2-Clause", "engines": { "node": ">=12" }, @@ -662,117 +664,6 @@ "node": ">=18" } }, - "node_modules/gcp-metadata": { - "version": "5.3.0", - "resolved": "https://registry.npmjs.org/gcp-metadata/-/gcp-metadata-5.3.0.tgz", - "integrity": "sha512-FNTkdNEnBdlqF2oatizolQqNANMrcqJt6AAYt99B3y1aLLC8Hc5IOBb+ZnnzllodEEf6xMBp6wRcBbc16fa65w==", - "license": "Apache-2.0", - "optional": true, - "peer": true, - "dependencies": { - "gaxios": "^5.0.0", - "json-bigint": "^1.0.0" - }, - "engines": { - "node": ">=12" - } - }, - "node_modules/gcp-metadata/node_modules/agent-base": { - "version": "6.0.2", - "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-6.0.2.tgz", - "integrity": "sha512-RZNwNclF7+MS/8bDg70amg32dyeZGZxiDuQmZxKLAlQjr3jGyLx+4Kkk58UO7D2QdgFIQCovuSuZESne6RG6XQ==", - "license": "MIT", - "optional": true, - "peer": true, - "dependencies": { - "debug": "4" - }, - "engines": { - "node": ">= 6.0.0" - } - }, - "node_modules/gcp-metadata/node_modules/gaxios": { - "version": "5.1.3", - "resolved": "https://registry.npmjs.org/gaxios/-/gaxios-5.1.3.tgz", - "integrity": "sha512-95hVgBRgEIRQQQHIbnxBXeHbW4TqFk4ZDJW7wmVtvYar72FdhRIo1UGOLS2eRAKCPEdPBWu+M7+A33D9CdX9rA==", - "license": "Apache-2.0", - "optional": true, - "peer": true, - "dependencies": { - "extend": "^3.0.2", - "https-proxy-agent": "^5.0.0", - "is-stream": "^2.0.0", - "node-fetch": "^2.6.9" - }, - "engines": { - "node": ">=12" - } - }, - "node_modules/gcp-metadata/node_modules/https-proxy-agent": { - "version": "5.0.1", - "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-5.0.1.tgz", - "integrity": "sha512-dFcAjpTQFgoLMzC2VwU+C/CbS7uRL0lWmxDITmqm7C+7F0Odmj6s9l6alZc6AELXhrnggM2CeWSXHGOdX2YtwA==", - "license": "MIT", - "optional": true, - "peer": true, - "dependencies": { - "agent-base": "6", - "debug": "4" - }, - "engines": { - "node": ">= 6" - } - }, - "node_modules/gcp-metadata/node_modules/node-fetch": { - "version": "2.7.0", - "resolved": "https://registry.npmjs.org/node-fetch/-/node-fetch-2.7.0.tgz", - "integrity": "sha512-c4FRfUm/dbcWZ7U+1Wq0AwCyFL+3nt2bEw05wfxSz+DWpWsitgmSgYmy2dQdWyKC1694ELPqMs/YzUSNozLt8A==", - "license": "MIT", - "optional": true, - "peer": true, - "dependencies": { - "whatwg-url": "^5.0.0" - }, - "engines": { - "node": "4.x || >=6.0.0" - }, - "peerDependencies": { - "encoding": "^0.1.0" - }, - "peerDependenciesMeta": { - "encoding": { - "optional": true - } - } - }, - "node_modules/gcp-metadata/node_modules/tr46": { - "version": "0.0.3", - "resolved": "https://registry.npmjs.org/tr46/-/tr46-0.0.3.tgz", - "integrity": "sha512-N3WMsuqV66lT30CrXNbEjx4GEwlow3v6rr4mCcv6prnfwhS01rkgyFdjPNBYd9br7LpXV1+Emh01fHnq2Gdgrw==", - "license": "MIT", - "optional": true, - "peer": true - }, - "node_modules/gcp-metadata/node_modules/webidl-conversions": { - "version": "3.0.1", - "resolved": "https://registry.npmjs.org/webidl-conversions/-/webidl-conversions-3.0.1.tgz", - "integrity": "sha512-2JAn3z8AR6rjK8Sm8orRC0h/bcl/DqL7tRPdGZ4I1CjdF+EaMLmYxBHyXuKL849eucPFhvBoxMsflfOb8kxaeQ==", - "license": "BSD-2-Clause", - "optional": true, - "peer": true - }, - "node_modules/gcp-metadata/node_modules/whatwg-url": { - "version": "5.0.0", - "resolved": "https://registry.npmjs.org/whatwg-url/-/whatwg-url-5.0.0.tgz", - "integrity": "sha512-saE57nupxk6v3HY35+jzBwYa0rKSy0XR8JSxZPwgLr7ys0IBzhGviA1/TUGJLmSVqs8pb9AnvICXEuOHLprYTw==", - "license": "MIT", - "optional": true, - "peer": true, - "dependencies": { - "tr46": "~0.0.3", - "webidl-conversions": "^3.0.0" - } - }, "node_modules/get-intrinsic": { "version": "1.3.0", "resolved": "https://registry.npmjs.org/get-intrinsic/-/get-intrinsic-1.3.0.tgz", @@ -985,20 +876,6 @@ "resolved": "https://registry.npmjs.org/is-promise/-/is-promise-4.0.0.tgz", "integrity": "sha512-hvpoI6korhJMnej285dSg6nu1+e6uxs7zG3BYAm5byqDsgJNWwxzM6z6iZiAgQR4TJ30JmBTOwqZUw3WlyH3AQ==" }, - "node_modules/is-stream": { - "version": "2.0.1", - "resolved": "https://registry.npmjs.org/is-stream/-/is-stream-2.0.1.tgz", - "integrity": "sha512-hFoiJiTl63nn+kstHGBtewWSKnQLpyb155KHheA1l39uvtO9nWIop1p3udqPcUd/xbF1VLMO4n7OI6p7RbngDg==", - "license": "MIT", - "optional": true, - "peer": true, - "engines": { - "node": ">=8" - }, - "funding": { - "url": "https://github.com/sponsors/sindresorhus" - } - }, "node_modules/jake": { "version": "10.9.4", "resolved": "https://registry.npmjs.org/jake/-/jake-10.9.4.tgz", @@ -1221,9 +1098,9 @@ "integrity": "sha512-UtJcAD4yEaGtjPezWuO9wC4nwUnVH/8/Im3yEHQP4b67cXlD/Qr9hdITCU1xDbSEXg2XKNaP8jsReV7vQd00/A==" }, "node_modules/minimatch": { - "version": "5.1.6", - "resolved": "https://registry.npmjs.org/minimatch/-/minimatch-5.1.6.tgz", - "integrity": "sha512-lKwV/1brpG6mBUFHtb7NUmtABCb2WZZmm2wNiOA5hAb8VdCS4B3dtMWyvcoViccwAW/COERjXLt0zP1zXUN26g==", + "version": "5.1.9", + "resolved": "https://registry.npmjs.org/minimatch/-/minimatch-5.1.9.tgz", + "integrity": "sha512-7o1wEA2RyMP7Iu7GNba9vc0RWWGACJOCZBJX2GJWip0ikV+wcOsgVuY9uE8CPiyQhkGFSlhuSkZPavN7u1c2Fw==", "license": "ISC", "dependencies": { "brace-expansion": "^2.0.1" @@ -1287,9 +1164,10 @@ } }, "node_modules/mongoose": { - "version": "8.19.1", - "resolved": "https://registry.npmjs.org/mongoose/-/mongoose-8.19.1.tgz", - "integrity": "sha512-oB7hGQJn4f8aebqE7mhE54EReb5cxVgpCxQCQj0K/cK3q4J3Tg08nFP6sM52nJ4Hlm8jsDnhVYpqIITZUAhckQ==", + "version": "8.24.0", + "resolved": "https://registry.npmjs.org/mongoose/-/mongoose-8.24.0.tgz", + "integrity": "sha512-EEZwOibDPZ5uZN3bFapfnRskEbdljAf6sP9ln6u+P4e5IfkOAh6Tqw2g8/Tag++KHOAJ095WXT/c0uqRq4Vckg==", + "license": "MIT", "dependencies": { "bson": "^6.10.4", "kareem": "2.6.3", @@ -1558,11 +1436,13 @@ } }, "node_modules/path-to-regexp": { - "version": "8.2.0", - "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-8.2.0.tgz", - "integrity": "sha512-TdrF7fW9Rphjq4RjrW0Kp2AW0Ahwu9sRGTkS6bvDi0SCwZlEZYmcfDbEsTz8RVk0EHIS/Vd1bv3JhG+1xZuAyQ==", - "engines": { - "node": ">=16" + "version": "8.4.2", + "resolved": "https://registry.npmjs.org/path-to-regexp/-/path-to-regexp-8.4.2.tgz", + "integrity": "sha512-qRcuIdP69NPm4qbACK+aDogI5CBDMi1jKe0ry5rSQJz8JVLsC7jV8XpiJjGRLLol3N+R5ihGYcrPLTno6pAdBA==", + "license": "MIT", + "funding": { + "type": "opencollective", + "url": "https://opencollective.com/express" } }, "node_modules/pause": { @@ -1597,9 +1477,9 @@ } }, "node_modules/qs": { - "version": "6.15.0", - "resolved": "https://registry.npmjs.org/qs/-/qs-6.15.0.tgz", - "integrity": "sha512-mAZTtNCeetKMH+pSjrb76NAM8V9a05I9aBZOHztWy/UqcJdQYNsf59vrRKWnojAT9Y+GbIvoTBC++CPHqpDBhQ==", + "version": "6.15.2", + "resolved": "https://registry.npmjs.org/qs/-/qs-6.15.2.tgz", + "integrity": "sha512-Rzq0KEyX/w/tEybncDgdkZrJgVUsUMk3xjh3t5bv3S1HTAtg+uOYt72+ZfwiQwKdysThkTBdL/rTi6HDmX9Ddw==", "license": "BSD-3-Clause", "dependencies": { "side-channel": "^1.1.0" diff --git a/package.json b/package.json index 83c3d37..22b06f7 100644 --- a/package.json +++ b/package.json @@ -14,7 +14,7 @@ "connect-flash": "^0.1.1", "connect-mongo": "^5.1.0", "cookie-parser": "^1.4.7", - "dotenv": "^17.2.3", + "dotenv": "^17.4.2", "ejs": "^3.1.10", "ejs-mate": "^4.0.0", "express": "^5.1.0", diff --git a/views/listings/archive.ejs b/views/listings/archive.ejs index 464bb92..92bc898 100644 --- a/views/listings/archive.ejs +++ b/views/listings/archive.ejs @@ -12,12 +12,12 @@ src="https://ui-avatars.com/api/?name=<%= students.name %>&bold=true" alt="" /> -
    +

    <%= students.name %> -

    +

    <%= students.grade %> Due <%= students.dueFees %>₹ @@ -38,5 +38,4 @@ <%}%>
    - - + \ No newline at end of file From c9c31d179646fcd9e79530e481f50084da3c0825 Mon Sep 17 00:00:00 2001 From: vikrant Date: Sun, 14 Jun 2026 10:05:26 +0530 Subject: [PATCH 08/12] feat: enhance student retrieval logic and improve archive view layout --- controllers/students.js | 5 ++++- public/css/students.css | 7 +++++++ views/listings/archive.ejs | 2 ++ 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/controllers/students.js b/controllers/students.js index 2d9e05d..f88f4b2 100644 --- a/controllers/students.js +++ b/controllers/students.js @@ -42,7 +42,10 @@ module.exports.students = catchAsync(async (req, res) => { }); module.exports.showStudent = catchAsync(async (req, res, next) => { const { id } = req.params; - const student = await Student.findOne({ _id: id, owner: req.user._id }); + let student = await Student.findOne({ _id: id, owner: req.user._id }); + if (!student) { + student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); + } if (!student) throw new ExpressError(404, "Student not found"); const formattedDate = formatDate(student.joiningDate); res.render("listings/show", { student, formattedDate }); diff --git a/public/css/students.css b/public/css/students.css index 0e41793..52dc173 100644 --- a/public/css/students.css +++ b/public/css/students.css @@ -91,6 +91,13 @@ div.links-group:has(a.active)::after { display: flex; flex-wrap: wrap; } +.students a{ + text-decoration: none; +} +.students a .list { + padding: 0; + margin: 0; +} ul { list-style-type: none; } diff --git a/views/listings/archive.ejs b/views/listings/archive.ejs index 92bc898..bef4811 100644 --- a/views/listings/archive.ejs +++ b/views/listings/archive.ejs @@ -6,6 +6,7 @@

     Archive Students

    <%for(let students of studentsData){%> +
    + <%}%>
    From caae1844cdaa02d3a867d4f82787d295ab5ec024 Mon Sep 17 00:00:00 2001 From: vikrant Date: Sun, 14 Jun 2026 10:25:40 +0530 Subject: [PATCH 09/12] feat: add comments to indicate performance concerns in student retrieval methods --- controllers/students.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/controllers/students.js b/controllers/students.js index f88f4b2..6035399 100644 --- a/controllers/students.js +++ b/controllers/students.js @@ -43,6 +43,7 @@ module.exports.students = catchAsync(async (req, res) => { module.exports.showStudent = catchAsync(async (req, res, next) => { const { id } = req.params; let student = await Student.findOne({ _id: id, owner: req.user._id }); + // This method is too much time taking if (!student) { student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); } @@ -203,6 +204,13 @@ module.exports.deleteStudent = catchAsync(async (req, res) => { _id: id, owner: req.user._id, }); + //this method is too much time taking + if(!removedStudent) { + let archivedStudent = await ArchivedStudent.findOne({ + _id: id, + owner: req.user._id, + }); + } if (!removedStudent) { req.flash("error", "Student not found or already deleted."); return res.redirect("/students"); From ff7a449bb755bd84139a7b4dd4edb9db23626338 Mon Sep 17 00:00:00 2001 From: vikrant Date: Sun, 14 Jun 2026 11:59:14 +0530 Subject: [PATCH 10/12] feat: implement showArchiveStu route and view for archived student details --- controllers/students.js | 18 ++-- routes/students.js | 3 + views/listings/archive.ejs | 12 +-- views/listings/showArchiveStu.ejs | 155 ++++++++++++++++++++++++++++++ 4 files changed, 166 insertions(+), 22 deletions(-) create mode 100644 views/listings/showArchiveStu.ejs diff --git a/controllers/students.js b/controllers/students.js index 6035399..0f19b7b 100644 --- a/controllers/students.js +++ b/controllers/students.js @@ -43,10 +43,6 @@ module.exports.students = catchAsync(async (req, res) => { module.exports.showStudent = catchAsync(async (req, res, next) => { const { id } = req.params; let student = await Student.findOne({ _id: id, owner: req.user._id }); - // This method is too much time taking - if (!student) { - student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); - } if (!student) throw new ExpressError(404, "Student not found"); const formattedDate = formatDate(student.joiningDate); res.render("listings/show", { student, formattedDate }); @@ -204,13 +200,6 @@ module.exports.deleteStudent = catchAsync(async (req, res) => { _id: id, owner: req.user._id, }); - //this method is too much time taking - if(!removedStudent) { - let archivedStudent = await ArchivedStudent.findOne({ - _id: id, - owner: req.user._id, - }); - } if (!removedStudent) { req.flash("error", "Student not found or already deleted."); return res.redirect("/students"); @@ -269,6 +258,13 @@ module.exports.archived = catchAsync(async (req, res) => { .lean(); res.render("listings/archive", { studentsData: archived }); }); +module.exports.showArchiveStu = catchAsync(async (req, res, next) => { + const { id } = req.params; + let student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); + if (!student) throw new ExpressError(404, "Student not found"); + const formattedDate = formatDate(student.joiningDate); + res.render("listings/showArchiveStu", { student, formattedDate }); +}); module.exports.restoreStudent = catchAsync(async (req, res) => { const { id } = req.params; const archivedStudent = await ArchivedStudent.findOne({ diff --git a/routes/students.js b/routes/students.js index cc1ed78..bc2233e 100644 --- a/routes/students.js +++ b/routes/students.js @@ -17,6 +17,9 @@ router .get(isLoggedIn, validateObjectId, student.editStudent) .put(isLoggedIn, validateObjectId, student.saveEditStudent); router.get("/:id/deactivate", student.deactivateStudent); +router + .route("/:id/archive") + .get(isLoggedIn, validateObjectId, student.showArchiveStu) router.post("/:id/restore", student.restoreStudent); router.delete("/:id/delete", student.deletearchiveStudent); router.post("/:id/addFees", isLoggedIn, student.addFees); diff --git a/views/listings/archive.ejs b/views/listings/archive.ejs index bef4811..53b30cf 100644 --- a/views/listings/archive.ejs +++ b/views/listings/archive.ejs @@ -6,7 +6,7 @@

     Archive Students

    <%for(let students of studentsData){%> - +
    Due <%= students.dueFees %>₹
    -
    -
    - -
    -
    - -
    -
    diff --git a/views/listings/showArchiveStu.ejs b/views/listings/showArchiveStu.ejs new file mode 100644 index 0000000..9a4e530 --- /dev/null +++ b/views/listings/showArchiveStu.ejs @@ -0,0 +1,155 @@ +<% layout('layouts/boilerPlate') -%> + + +
    +
    +
    +
    + <%= student.name %> Img +

    <%= student.name %>

    +
    +

    Grade : <%= student.grade %>

    +

    Fees : <%= student.fees %>

    +

    Due Fees : <%= student.dueFees %>

    +

    Joining Date : <%= formattedDate %>

    + <%if(student.parent){%> +

    Parent : <%= student.parent %>

    + <%}%> + <%if(student.contact){%> +

    + Contact : <%= student.contact %> + + + + +

    + <%}%> + <%if(student.phone){%> +

    + Phone : <%= student.phone %> +   + + + +

    + <%}%> + <%if(student.dueFees >= student.fees && (student.phone || student.contact)){%> + <%if(student.contact){%> +

    + Remind on : Contact +   + +

    + <%}%> + <%if(student.phone){%> +

    + Remind on : Phone +   + +

    + <%}%> + <%}%> + <%if(student.note){%> +

    Note : <%= student.note %>

    + <%}%> +
    +
    +
    +

    Fees

    +

    +   Add a new payment +

    + + + + +
    +
    +
    +

    +  Payment History +

    + <% if(student.feesHistory && student.feesHistory.length > 0) { %> + <%student.feesHistory.reverse().forEach(function(fees) { %> +
    +

    + ₹ <%= fees.amount %> + Paid On : <%= fees.paidDate.toLocaleDateString("en-GB", { weekday: + "short", day: "2-digit", month: "short", year: "numeric" }) %> +

    + <%if(fees.note){%> +

    Note : <%=fees.note%>

    + <%}%> +
    +
    + <% }); %> <% } else { %> +

    No payment history found.

    + <% } %> +
    +
    +
    +
    + +
    +
    + +
    +
    +
    + From d215c1aa1784c3603afcbb992226668e3f708843 Mon Sep 17 00:00:00 2001 From: vikrant Date: Thu, 18 Jun 2026 19:49:06 +0530 Subject: [PATCH 11/12] feat: add functionality to record fees for archived students --- controllers/students.js | 45 +++++++++++++++++++++++++++++++ routes/students.js | 3 ++- views/listings/showArchiveStu.ejs | 2 +- 3 files changed, 48 insertions(+), 2 deletions(-) diff --git a/controllers/students.js b/controllers/students.js index 0f19b7b..47e2a95 100644 --- a/controllers/students.js +++ b/controllers/students.js @@ -265,6 +265,51 @@ module.exports.showArchiveStu = catchAsync(async (req, res, next) => { const formattedDate = formatDate(student.joiningDate); res.render("listings/showArchiveStu", { student, formattedDate }); }); +module.exports.addArchiveStuFee = catchAsync(async (req, res) => { + let { id } = req.params; + let { note, amount, paidOn } = req.body; + if (!amount || Number(amount) <= 0) { + req.flash("error", "Amount must be greater than 0"); + return res.redirect(`/students/${id}`); + } + amount = Number(amount); + let paidDate = paidOn ? new Date(paidOn) : new Date(); + const student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); + if (!student) { + return res.status(404).send("Student not found"); + } + student.feesHistory.push({ + note, + amount, + paidDate, + }); + const month = new Date().getMonth() + 1; + const year = new Date().getFullYear(); + let thisMonthData = await MonthlyReport.findOne({ + owner: req.user._id, + month, + year, + }); + if (!thisMonthData) { + thisMonthData = await MonthlyReport.create({ + owner: req.user._id, + month, + year, + totalEarning: 0, + expenses: [], + totalExpenses: 0, + newStudents: 0, + studentsLeft: 0, + createdAt: new Date(), + }); + } + thisMonthData.totalEarning += amount; + student.dueFees -= amount; + await student.save(); + await thisMonthData.save(); + req.flash("success", `Fees added for ${student.name}. `); + res.redirect(`/students`); +}); module.exports.restoreStudent = catchAsync(async (req, res) => { const { id } = req.params; const archivedStudent = await ArchivedStudent.findOne({ diff --git a/routes/students.js b/routes/students.js index bc2233e..340ed51 100644 --- a/routes/students.js +++ b/routes/students.js @@ -19,8 +19,9 @@ router router.get("/:id/deactivate", student.deactivateStudent); router .route("/:id/archive") - .get(isLoggedIn, validateObjectId, student.showArchiveStu) + .get(isLoggedIn, validateObjectId, student.showArchiveStu); router.post("/:id/restore", student.restoreStudent); router.delete("/:id/delete", student.deletearchiveStudent); router.post("/:id/addFees", isLoggedIn, student.addFees); +router.post("/:id/addArchiveStuFee", isLoggedIn, student.addArchiveStuFee); module.exports = router; diff --git a/views/listings/showArchiveStu.ejs b/views/listings/showArchiveStu.ejs index 9a4e530..5335a60 100644 --- a/views/listings/showArchiveStu.ejs +++ b/views/listings/showArchiveStu.ejs @@ -73,7 +73,7 @@
    From fa4f89d8890453e6745cd1f516b55cb512befb09 Mon Sep 17 00:00:00 2001 From: vikrant Date: Fri, 19 Jun 2026 16:26:27 +0530 Subject: [PATCH 12/12] feat: add edit functionality for archived students with corresponding views --- controllers/students.js | 46 ++++++++++++ routes/students.js | 4 ++ views/listings/editArchiveStu.ejs | 115 ++++++++++++++++++++++++++++++ views/listings/showArchiveStu.ejs | 3 + 4 files changed, 168 insertions(+) create mode 100644 views/listings/editArchiveStu.ejs diff --git a/controllers/students.js b/controllers/students.js index 47e2a95..869a8df 100644 --- a/controllers/students.js +++ b/controllers/students.js @@ -66,6 +66,13 @@ module.exports.deactivateStudent = catchAsync(async (req, res) => { req.flash("success", "Student deactivated successfully"); res.redirect("/students"); }); +module.exports.editArchiveStu = catchAsync(async (req, res, next) => { + const { id } = req.params; + const student = await ArchivedStudent.findOne({ _id: id, owner: req.user._id }); + if (!student) throw new ExpressError(404, "Archived student not found"); + const formattedDate = formatDate(student.joiningDate, "input"); + res.render("listings/editArchiveStu", { student, formattedDate }); +}); module.exports.editStudent = catchAsync(async (req, res, next) => { const { id } = req.params; const student = await Student.findOne({ _id: id, owner: req.user._id }); @@ -73,6 +80,45 @@ module.exports.editStudent = catchAsync(async (req, res, next) => { const formattedDate = formatDate(student.joiningDate, "input"); res.render("listings/edit", { student, formattedDate }); // create this view }); +module.exports.saveEditArchiveStu = catchAsync(async (req, res, next) => { + const { id } = req.params; + const { + name, + grade, + parent, + contact, + phone, + note, + joiningDate, + fees, + dueFees, + } = req.body; + let student = await ArchivedStudent.findOneAndUpdate( + { _id: id, owner: req.user._id }, + { + name, + grade, + parent, + contact, + phone, + note, + joiningDate, + dueFees, + fees, + }, + { new: true, runValidators: true }, + ); + if (!student) { + req.flash("error", "Student not found"); + return res.redirect("/students"); + } + const formattedDate = formatDate(student.joiningDate); + res.render("listings/show", { + student, + formattedDate, + success: "Details updated.", + }); +}); module.exports.saveEditStudent = catchAsync(async (req, res, next) => { const { id } = req.params; const { diff --git a/routes/students.js b/routes/students.js index 340ed51..9cfbefc 100644 --- a/routes/students.js +++ b/routes/students.js @@ -24,4 +24,8 @@ router.post("/:id/restore", student.restoreStudent); router.delete("/:id/delete", student.deletearchiveStudent); router.post("/:id/addFees", isLoggedIn, student.addFees); router.post("/:id/addArchiveStuFee", isLoggedIn, student.addArchiveStuFee); +router + .route("/:id/editArchiveStu") + .get(isLoggedIn, validateObjectId, student.editArchiveStu) + .put(isLoggedIn, validateObjectId, student.saveEditArchiveStu); module.exports = router; diff --git a/views/listings/editArchiveStu.ejs b/views/listings/editArchiveStu.ejs new file mode 100644 index 0000000..1ae6831 --- /dev/null +++ b/views/listings/editArchiveStu.ejs @@ -0,0 +1,115 @@ +<% layout('layouts/boilerPlate') -%> + + +
    + Student reading a book + +

    Edit Details

    + +
    + + + +
    + + + + + + + +
    + + diff --git a/views/listings/showArchiveStu.ejs b/views/listings/showArchiveStu.ejs index 5335a60..b76d42b 100644 --- a/views/listings/showArchiveStu.ejs +++ b/views/listings/showArchiveStu.ejs @@ -142,6 +142,9 @@
    + + +