Using SubscriptionStats.lastTryIsoDate instead of SubscriptionStats.lastTryDate

Author: salarcode

src/core/SubscriptionUpdater.ts

@@ -85,8 +85,8 @@ export class SubscriptionUpdater {
 				let interval = subscription.refreshRate * 60 * 1000;
 				let firstFetchTimeoutMs: number;
 
-				if (subscription.stats?.lastTryDate) {
-					const timeSinceLastTry = Date.now() - (new Date(subscription.stats.lastTryDate)).getTime();
+				if (subscription.stats?.lastTryIsoDate) {
+					const timeSinceLastTry = Date.now() - (new Date(subscription.stats.lastTryIsoDate)).getTime();
 
 					firstFetchTimeoutMs = Math.max(0, interval - timeSinceLastTry);
 				} else {
@@ -265,8 +265,8 @@ export class SubscriptionUpdater {
 					let interval = subscription.refreshRate * 60 * 1000;
 					let firstFetchTimeoutMs: number;
 
-					if (subscription.stats?.lastTryDate) {
-						const timeSinceLastTry = Date.now() - (new Date(subscription.stats.lastTryDate)).getTime();
+					if (subscription.stats?.lastTryIsoDate) {
+						const timeSinceLastTry = Date.now() - (new Date(subscription.stats.lastTryIsoDate)).getTime();
 
 						firstFetchTimeoutMs = Math.max(0, interval - timeSinceLastTry);
 					} else {

src/core/definitions.ts

@@ -2,7 +2,6 @@
 import { api, environment } from '../lib/environment';
 import { Utils } from '../lib/Utils';
 import { ProfileOperations } from './ProfileOperations';
-import { Debug } from '../lib/Debug';
 
 /*
  * This file is part of SmartProxy <https://github.com/salarcode/SmartProxy>,
@@ -956,17 +955,19 @@ export enum ProxyServerSubscriptionFormat {
 export class SubscriptionStats {
 	lastSuccessDate: string;
 	lastTryDate: string;
+	lastTryIsoDate: string;
 	lastStatus: boolean;
 	lastStatusMessage: string;
 	lastStatusProxyServerName: string;
 
 	public static updateStats(stats: SubscriptionStats, success: boolean, errorResult?: any) {
 		let now = new Date();
-		stats.lastTryDate = now.toISOString();
+		stats.lastTryIsoDate = now.toISOString();
+		stats.lastTryDate = now.toLocaleDateString() + ' ' + now.toLocaleTimeString();
 		if (success) {
 			stats.lastStatus = true;
 			stats.lastStatusMessage = null;
-			stats.lastSuccessDate = now.toLocaleDateString() + ' ' + now.toLocaleTimeString();
+			stats.lastSuccessDate = stats.lastTryDate;
 		}
 		else {
 			stats.lastStatus = false;
@@ -985,13 +986,7 @@ export class SubscriptionStats {
 
 		if (!stats.lastStatus) {
 			if (stats.lastTryDate) {
-				try {
-					let lastTryDate = new Date(stats.lastTryDate);
-					let lastTryDateText = lastTryDate.toLocaleDateString() + ' ' + lastTryDate.toLocaleTimeString();
-					status += `\r\n${api.i18n.getMessage("settingsSubscriptionStatsLastTry")} ${lastTryDateText}`
-				} catch (error) {
-					Debug.warn(`SubscriptionStats.lastTryDate has invalid (probably old) value`, error);
-				}
+				status += `\r\n${api.i18n.getMessage("settingsSubscriptionStatsLastTry")} ${stats.lastTryDate}`
 			}
 			else {
 				status += `\r\n${api.i18n.getMessage("settingsSubscriptionStatsLastTry")} -`