GUACAMOLE-1020: Remove Weekend and Weekday definitions to avoid local…

…e issues.
apache · Oct 14, 2023 · 02cf55e · 02cf55e
1 parent d352110
commit 02cf55e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 36 deletions.
diff --git a/...mole-auth-restrict/src/main/java/org/apache/guacamole/calendar/TimeRestrictionParser.java b/...mole-auth-restrict/src/main/java/org/apache/guacamole/calendar/TimeRestrictionParser.java
@@ -43,14 +43,13 @@ public class TimeRestrictionParser {
      * <ul>
      * <li>1:0700-1700 - Monday from 07:00 to 17:00
      * <li>7:0000-2359 - Sunday, all day (00:00 to 23:59)
-     * <li>wd:0900-1700 - Monday through Friday, 09:00 to 17:00
-     * <li>we:0900-1700 - Saturday and Sunday, 09:00 to 17:00
+     * <li>*:0900-1700 - Every day, 09:00 to 17:00
      * <li>6:0900-1600;7:1200-1300 - Saturday, 09:00 to 16:00, and Sunday, 
      * 12:00 - 13:00
      * </ul>
      */
     private static final Pattern RESTRICTION_REGEX = 
-            Pattern.compile("(?:^|;)+([1-7*]|(?:[w][ed]))(?::((?:[01][0-9]|2[0-3])[0-5][0-9])\\-((?:[01][0-9]|2[0-3])[0-5][0-9]))+");
+            Pattern.compile("(?:^|;)+([1-7*])(?::((?:[01][0-9]|2[0-3])[0-5][0-9])\\-((?:[01][0-9]|2[0-3])[0-5][0-9]))+");
 
     /**
      * The RegEx group that contains the start day-of-week of the restriction.
@@ -67,25 +66,6 @@ public class TimeRestrictionParser {
      */
     private static final int RESTRICTION_TIME_END_GROUP = 3;
 
-    /**
-     * A list of DayOfWeek items that make up weekdays.
-     */
-    private static final List<DayOfWeek> RESTRICTION_WEEKDAYS = Arrays.asList(
-            DayOfWeek.MONDAY,
-            DayOfWeek.TUESDAY,
-            DayOfWeek.WEDNESDAY,
-            DayOfWeek.THURSDAY,
-            DayOfWeek.FRIDAY
-    );
-
-    /**
-     * A list of DayOfWeek items that make up weekends.
-     */
-    private static final List<DayOfWeek> RESTRICTION_WEEKEND = Arrays.asList(
-            DayOfWeek.SATURDAY,
-            DayOfWeek.SUNDAY
-    );
-
     /**
      * A list of DayOfWeek items that make up all days of the week.
      */
@@ -154,16 +134,6 @@ public static List<DailyRestriction> parseString(String restrictionString) {
                     restrictions.add(new DailyRestriction(RESTRICTION_ALL_DAYS, startTime, endTime));
                     break;
 
-                // Weekdays only.
-                case "wd":
-                    restrictions.add(new DailyRestriction(RESTRICTION_WEEKDAYS, startTime, endTime));
-                    break;
-
-                // Weekend days only.
-                case "we":
-                    restrictions.add(new DailyRestriction(RESTRICTION_WEEKEND, startTime, endTime));
-                    break;
-
                 // A specific day of the week.
                 default:
                     restrictions.add(new DailyRestriction(DayOfWeek.of(Integer.parseInt(dayString)), startTime, endTime));

diff --git a/.../guacamole-auth-restrict/src/main/resources/controllers/timeRestrictionFieldController.js b/.../guacamole-auth-restrict/src/main/resources/controllers/timeRestrictionFieldController.js
@@ -64,9 +64,7 @@ angular.module('guacRestrict').controller('timeRestrictionFieldController', ['$s
         { id : '5', day : 'Friday' },
         { id : '6', day : 'Saturday' },
         { id : '7', day : 'Sunday' },
-        { id : '*', day : 'All days' },
-        { id : 'wd', day: 'Week days' },
-        { id : 'we', day: 'Week end' }
+        { id : '*', day : 'All days' }
     ];
 
     /**
@@ -118,7 +116,7 @@ angular.module('guacRestrict').controller('timeRestrictionFieldController', ['$s
             return restrictions;
 
         // Set up the RegEx and split the string using the separator.
-        const restrictionRegex = new RegExp('^([1-7*]|(?:[w][ed]))(?::((?:[01][0-9]|2[0-3])[0-5][0-9])\-((?:[01][0-9]|2[0-3])[0-5][0-9]))$');
+        const restrictionRegex = new RegExp('^([1-7*])(?::((?:[01][0-9]|2[0-3])[0-5][0-9])\-((?:[01][0-9]|2[0-3])[0-5][0-9]))$');
         var restrArray = restrString.split(";");
 
         // Loop through split string and process each item