BAH-3245 | Fix. Set Default Date Format (Bahmni#720)

* [Parvathy, Rahul] | BAH-3245 | Fix. Set Default Date Format Co-authored-by: Parvathy Babu <parvathy.babu@thoughtworks.com> The default year format is YYYY * [Parvathy, Rahul] | BAH-3245 | Fix. Set Default Date Format Co-authored-by: Rahul Ramesh <rahul.ramesh@thoughtworks.com> --------- Co-authored-by: Rahul Ramesh <rahul.ramesh@thoughtworks.com>
harshkumar8789 · Oct 4, 2023 · 1e592fb · 1e592fb
1 parent 4b5507e
commit 1e592fb
Show file tree

Hide file tree

Showing 15 changed files with 33 additions and 33 deletions.
diff --git a/ui/app/clinical/common/controllers/visitController.js b/ui/app/clinical/common/controllers/visitController.js
@@ -63,7 +63,7 @@ angular.module('bahmni.clinical')
             };
 
             $scope.displayDate = function (date) {
-                return moment(date).format("DD-MMM-YY");
+                return moment(date).format("DD-MMM-YYYY");
             };
 
             $scope.$on("event:printVisitTab", function () {

diff --git a/ui/app/clinical/consultation/views/bacteriology.html b/ui/app/clinical/consultation/views/bacteriology.html
@@ -89,7 +89,7 @@ <h2 class="section-title fl">{{'BACTERIOLOGY_TAB_TITLE_KEY' | translate}}</h2>
             <div class="fl savedSpecimens-text">
                 <span>{{getDisplayName(savedSpecimen)}}</span>
                 <span>#{{savedSpecimen.identifier}}</span>
-                <span>{{ 'BACTERIOLOGY_ADDED_ON_KEY' | translate }} {{savedSpecimen.dateCollected | date: 'dd MMM yy'}}</span>
+                <span>{{ 'BACTERIOLOGY_ADDED_ON_KEY' | translate }} {{savedSpecimen.dateCollected | date: 'dd MMM yyyy'}}</span>
             </div>
             <div class="fr savedSpecimens-action">
                 <i class="fa fa-pencil" ng-click="editSpecimen(savedSpecimen)"></i>

diff --git a/ui/app/common/ui-helper/directives.js b/ui/app/common/ui-helper/directives.js
@@ -24,7 +24,7 @@ angular.module('bahmni.common.uiHelper')
         var link = function ($scope, element, attrs, ngModel) {
             var maxDate = attrs.maxDate;
             var minDate = attrs.minDate || "-120y";
-            var format = attrs.dateFormat || 'dd-mm-yy';
+            var format = attrs.dateFormat || 'dd-mm-yyyy';
             element.datepicker({
                 changeYear: true,
                 changeMonth: true,

diff --git a/ui/app/common/util/dateUtil.js b/ui/app/common/util/dateUtil.js
@@ -83,7 +83,7 @@ Bahmni.Common.Util.DateUtil = {
     },
 
     getDateInMonthsAndYears: function (date, format) {
-        var format = format || "MMM YY";
+        var format = format || "MMM YYYY";
         var dateRepresentation = isNaN(Number(date)) ? date : Number(date);
         if (!moment(dateRepresentation).isValid()) {
             return date;
@@ -96,24 +96,24 @@ Bahmni.Common.Util.DateUtil = {
         if (!moment(dateRepresentation).isValid()) {
             return datetime;
         }
-        return dateRepresentation ? moment(dateRepresentation).format("DD MMM YY h:mm a") : null;
+        return dateRepresentation ? moment(dateRepresentation).format("DD MMM YYYY h:mm a") : null;
     },
 
     formatDateWithoutTime: function (date) {
         var dateRepresentation = isNaN(Number(date)) ? date : Number(date);
         if (!moment(dateRepresentation).isValid()) {
             return date;
         }
-        return dateRepresentation ? moment(dateRepresentation).format("DD MMM YY") : null;
+        return dateRepresentation ? moment(dateRepresentation).format("DD MMM YYYY") : null;
     },
 
     formatDateInStrictMode: function (date) {
         var dateRepresentation = isNaN(Number(date)) ? date : Number(date);
         if (moment(dateRepresentation, 'YYYY-MM-DD', true).isValid()) {
-            return moment(dateRepresentation).format("DD MMM YY");
+            return moment(dateRepresentation).format("DD MMM YYYY");
         }
         if (moment(dateRepresentation, 'YYYY-MM-DDTHH:mm:ss.SSSZZ', true).isValid()) {
-            return moment(dateRepresentation).format("DD MMM YY");
+            return moment(dateRepresentation).format("DD MMM YYYY");
         }
         return date;
     },

diff --git a/ui/app/reports/views/myReports.html b/ui/app/reports/views/myReports.html
@@ -21,8 +21,8 @@ <h2 class="section-title">{{::'REPORTS_QUEUE' | translate}}</h2>
             <tr ng-repeat="report in reports[day.unixTimeStamp]" ng-hide="report.hidden">
                 <td>{{:: report.name }}</td>
                 <td>{{ :: convertToDate(report.requestDatetime , 'hh:mm A')}}</td>
-                <td>{{ :: convertToDate(report.startDate , "DD MMM YY")}}</td>
-                <td>{{ :: convertToDate(report.endDate , "DD MMM YY")}}</td>
+                <td>{{ :: convertToDate(report.startDate , "DD MMM YYYY")}}</td>
+                <td>{{ :: convertToDate(report.endDate , "DD MMM YYYY")}}</td>
                 <td> {{ :: getFormat(report.format) }}</td>
                 <td ng-if="report.status != 'Completed' && report.status != 'Error'" class="reportStatus reportStatus--{{report.status}}">{{ report.status }}</td>
                 <td ng-if="report.status == 'Completed'" class="reportStatus reportStatus--link">

diff --git a/ui/test/unit/clinical/common/controllers/visitController.spec.js b/ui/test/unit/clinical/common/controllers/visitController.spec.js
@@ -125,7 +125,7 @@ describe('VisitController', function () {
         });
 
         it('should check data format', function () {
-            expect(scope.displayDate(new Date('2014', '7', '15', '12'))).toBe('15-Aug-14');
+            expect(scope.displayDate(new Date('2014', '7', '15', '12'))).toBe('15-Aug-2014');
         });
 
         it('should check test result class for expected style', function () {

diff --git a/ui/test/unit/common/displaycontrols/obsVsObsFlowSheet/directives/obsToObsFlowSheet.spec.js b/ui/test/unit/common/displaycontrols/obsVsObsFlowSheet/directives/obsToObsFlowSheet.spec.js
@@ -611,7 +611,7 @@ describe('obsToObsFlowSheet DisplayControl', function () {
                 }
             ];
 
-            expect(compiledElementScope.commafy(observations)).toEqual("10 Oct 15");
+            expect(compiledElementScope.commafy(observations)).toEqual("10 Oct 2015");
         });
 
         it('should return just month and year if the concept datatype is date and configured to show month and year', function () {
@@ -662,7 +662,7 @@ describe('obsToObsFlowSheet DisplayControl', function () {
                 }
             ];
 
-            expect(compiledElementScope.commafy(observations)).toEqual("Oct 15");
+            expect(compiledElementScope.commafy(observations)).toEqual("Oct 2015");
         });
 
         it('should return abbreviation of the coded answer if the concept is coded and configured to show abbreviation', function () {

diff --git a/ui/test/unit/common/displaycontrols/orders/directives/ordersControl.spec.js b/ui/test/unit/common/displaycontrols/orders/directives/ordersControl.spec.js
@@ -90,7 +90,7 @@ describe("OrdersDisplayControl", function () {
         var compiledElementScope = element.isolateScope();
         scope.$digest();
 
-        var expectedDate = moment(orders[0].orderDate).format("DD MMM YY h:mm a");
+        var expectedDate = moment(orders[0].orderDate).format("DD MMM YYYY h:mm a");
         expect(compiledElementScope.getTitle(orders[0])).toBe("Absconding on " + expectedDate + " by Surajkumar Surajkumar Surajkumar")
     });
 

diff --git a/ui/test/unit/common/domain/mappers/observationValueMapper.spec.js b/ui/test/unit/common/domain/mappers/observationValueMapper.spec.js
@@ -7,7 +7,7 @@ describe("Observation Value Mapper", function () {
 
     it("should return datetime value", function() {
         var obs = {type: "Datetime", value: "2015-12-05 16:02:00", concept: {}};
-        expect(mapper.map(obs)).toBe("05 Dec 15 4:02 pm");
+        expect(mapper.map(obs)).toBe("05 Dec 2015 4:02 pm");
     });
 
     it("should return multiselect values", function() {

diff --git a/ui/test/unit/common/models/age.spec.js b/ui/test/unit/common/models/age.spec.js
@@ -31,7 +31,7 @@ describe("Age", function(){
             var age = ageFactory.create(0, 0, 0);
             var birthDate = dateUtil.now();
 
-            expect(moment(ageFactory.calculateBirthDate(age)).format("DD MMM YY").toString()).toEqual(moment(birthDate).format("DD MMM YY").toString());
+            expect(moment(ageFactory.calculateBirthDate(age)).format("DD MMM YYYY").toString()).toEqual(moment(birthDate).format("DD MMM YYYY").toString());
         });
 
         it("should return date of birth", function(){

diff --git a/ui/test/unit/common/obs/directives/showObservation.spec.js b/ui/test/unit/common/obs/directives/showObservation.spec.js
@@ -103,7 +103,7 @@ describe("ensure that the directive show-observation works properly", function (
         scope.$digest();
 
         expect(compiledScope).not.toBeUndefined();
-        expect(compiledScope.dateString(observation)).toBe(moment(obsDate).format("DD MMM YY h:mm a"));
+        expect(compiledScope.dateString(observation)).toBe(moment(obsDate).format("DD MMM YYYY h:mm a"));
     });
 
     it("should open video in a new dialog", function () {

diff --git a/ui/test/unit/common/obs/observation.spec.js b/ui/test/unit/common/obs/observation.spec.js
@@ -59,7 +59,7 @@ describe("Observation", function () {
 
         it("should return datetime in specific format", function () {
             var observation = new Observation({"type": "Datetime", "value": "2014-12-05 17:00:00"});
-            expect(observation.getDisplayValue()).toBe("05 Dec 14 5:00 pm");
+            expect(observation.getDisplayValue()).toBe("05 Dec 2014 5:00 pm");
         });
 
         it("should return empty if value is null", function () {
@@ -71,15 +71,15 @@ describe("Observation", function () {
 
         it("should format date to bahmniDate  if value is of type date", function () {
             var observation = new Observation({"type": "Date", "value": "2012-12-10"});
-            expect(observation.getDisplayValue()).toBe("10 Dec 12");
+            expect(observation.getDisplayValue()).toBe("10 Dec 2012");
         });
         it("should format date in months and years if config is set to display in months and years", function(){
             var observation = new Observation({"type": "Date", "value": "2012-12-10"},{"displayMonthAndYear": true});
-            expect((observation.getDisplayValue())).toBe("Dec 12");
+            expect((observation.getDisplayValue())).toBe("Dec 2012");
         });
         it("should format date in text to months and years if config is set to display in months and years", function(){
             var observation = new Observation({"type": "text", "value": "2012-12-10"},{"displayMonthAndYear": true});
-            expect((observation.getDisplayValue())).toBe("Dec 12");
+            expect((observation.getDisplayValue())).toBe("Dec 2012");
         });
     });
 

diff --git a/ui/test/unit/common/uicontrols/programmanagement/controllers/manageProgramController.spec.js b/ui/test/unit/common/uicontrols/programmanagement/controllers/manageProgramController.spec.js
@@ -390,7 +390,7 @@ describe("ManageProgramController", function () {
             scope.updatePatientProgram(patientProgramToBeUpdated);
 
             expect(messageService.showMessage)
-                .toHaveBeenCalledWith("error", "PROGRAM_MANAGEMENT_STATE_CANT_START_BEFORE_KEY (15 Jul 15)");
+                .toHaveBeenCalledWith("error", "PROGRAM_MANAGEMENT_STATE_CANT_START_BEFORE_KEY (15 Jul 2015)");
         });
 
         it("should transit from one state to another successfully only if different state is selected", function () {
@@ -473,7 +473,7 @@ describe("ManageProgramController", function () {
 
             scope.updatePatientProgram(programToBeUpdated);
 
-            expect(messageService.showMessage).toHaveBeenCalledWith("error", "PROGRAM_MANAGEMENT_PROGRAM_CANT_END_BEFORE_KEY (15 Jul 15)");
+            expect(messageService.showMessage).toHaveBeenCalledWith("error", "PROGRAM_MANAGEMENT_PROGRAM_CANT_END_BEFORE_KEY (15 Jul 2015)");
         });
 
         it('should end a program successfully', function () {

diff --git a/ui/test/unit/common/util/dateUtil.spec.js b/ui/test/unit/common/util/dateUtil.spec.js
@@ -258,12 +258,12 @@ describe('DateUtil', function () {
     describe("formatDateWithTime", function () {
         it("should take a long representation of date and format", function () {
             var date = new Date(1427803080000);
-            expect(dateUtil.formatDateWithTime("1427803080000")).toEqual(moment(date).format("DD MMM YY h:mm a"));
+            expect(dateUtil.formatDateWithTime("1427803080000")).toEqual(moment(date).format("DD MMM YYYY h:mm a"));
         });
 
         it("should take a string representation of date and format", function () {
             var date = new Date();
-            expect(dateUtil.formatDateWithTime(moment(date).format(dateFormat))).toEqual(moment(date).format("DD MMM YY h:mm a"));
+            expect(dateUtil.formatDateWithTime(moment(date).format(dateFormat))).toEqual(moment(date).format("DD MMM YYYY h:mm a"));
         });
 
         it("should not break for undefined and return null", function () {
@@ -278,12 +278,12 @@ describe('DateUtil', function () {
     describe("formatDateWithoutTime", function () {
         it("should take a long representation of date and format", function () {
             var date = new Date(1427803080000);
-            expect(dateUtil.formatDateWithoutTime("1427803080000")).toEqual(moment(date).format("DD MMM YY"));
+            expect(dateUtil.formatDateWithoutTime("1427803080000")).toEqual(moment(date).format("DD MMM YYYY"));
         });
 
         it("should take a string representation of date and format", function () {
             var date = new Date();
-            expect(dateUtil.formatDateWithoutTime(moment(date).format(dateFormat))).toEqual(moment(date).format("DD MMM YY"));
+            expect(dateUtil.formatDateWithoutTime(moment(date).format(dateFormat))).toEqual(moment(date).format("DD MMM YYYY"));
         });
 
         it("should not break for undefined and return null", function () {
@@ -345,17 +345,17 @@ describe('DateUtil', function () {
 
     describe("getDateWithMonthsAndYears", function(){
         it("should return date with months and years", function(){
-           expect(dateUtil.getDateInMonthsAndYears(new Date('2014', '7', '15', '12','30','25'))).toBe('Aug 14');
+           expect(dateUtil.getDateInMonthsAndYears(new Date('2014', '7', '15', '12','30','25'))).toBe('Aug 2014');
         });
     });
 
     describe("formatDateInStrictMode", function(){
-       it("should return date in dd MMM yy format when date with yyyy-MM-dd format is passed", function(){
-          expect(dateUtil.formatDateInStrictMode('2016-02-10')).toBe('10 Feb 16');
+       it("should return date in dd MMM YYYY format when date with yyyy-MM-dd format is passed", function(){
+          expect(dateUtil.formatDateInStrictMode('2016-02-10')).toBe('10 Feb 2016');
        });
 
-       it("should return date in dd MMM yy format when date with yyyy-MM-ddTHH:mm:ss.SSSZ is passed", function(){
-           expect(dateUtil.formatDateInStrictMode('2016-03-01T10:30:00.000+0530')).toBe(moment('2016-03-01T10:30:00.000+0530').format('DD MMM YY'));
+       it("should return date in dd MMM YYYY format when date with yyyy-MM-ddTHH:mm:ss.SSSZ is passed", function(){
+           expect(dateUtil.formatDateInStrictMode('2016-03-01T10:30:00.000+0530')).toBe(moment('2016-03-01T10:30:00.000+0530').format('DD MMM YYYY'));
        });
 
        it("should return the string if the format does not match yyyy-MM-dd or yyyy-MM-ddTHH:mm:ss.SSSZ", function(){

diff --git a/ui/test/unit/reports/controllers/myReportsController.spec.js b/ui/test/unit/reports/controllers/myReportsController.spec.js
@@ -201,7 +201,7 @@ describe("MyReportsController", function () {
 
     it("should convert unixTimeStamps to given format", function () {
         expect(scope.convertToDate(1474529795000, 'hh:mm A')).toEqual(moment(1474529795000).format('hh:mm A'));
-        expect(scope.convertToDate(1474529795000, "DD MMM YY")).toEqual(moment(1474529795000).format('DD MMM YY'));
+        expect(scope.convertToDate(1474529795000, "DD MMM YYYY")).toEqual(moment(1474529795000).format('DD MMM YYYY'));
         expect(scope.convertToDate(1474529795000, "MMMM Do YYYY, dddd")).toEqual(moment(1474529795000).format('MMMM Do YYYY, dddd'));
     });