Changeset 1443 for trunk/src/org/expeditee/encryption/items/surrogates/Label.java

Timestamp:

11/08/19 13:23:37 (5 years ago)

Author:

bnemhaus

Message:

Changed Label::progressSurrogate mode so that a null return values signifies that asking for surrogate mode makes no sense because there are no labels in play.

Fixed bug with DisplayController::toggleSurrogateMode. It was printing incorrect messages to the message bay when the user was supposed to be exiting surrogate mode. This was fixed by querying Label::isInSurrogateMode.

File:

• trunk/src/org/expeditee/encryption/items/surrogates/Label.java (modified) (2 diffs)

trunk/src/org/expeditee/encryption/items/surrogates/Label.java

@@ -38 +38 @@
         }
         
+        public static boolean isInSurrogateMode() {
+                return isInSurrogateMode;
+        }
+        
+        /**
+         * Progresses surrogate mode and returns a list of labels that are currently in effect.
+         * @param context The list of expeditee Items to be considered for context.  These are checked for labels.
+         * @return Returns either:
+         *      <ol type='a'> 
+         *              <li> A list of length 1 specifying the label that the current surrogate mode accepts or </li>
+         *              <li> A list of length 0 which specifies that surrogate mode is in unprivileged view or </li>
+         *              <li> Null, which means there are no labels in play on this frame 
+         *                      (so asking for surrogate mode does not make sense) </li> 
+         *  </ol>
+         */
         public static List<String> progressSurrogateMode(ItemsList context) {
                 // Maintain the local cache of user labels
@@ -45 +60 @@
                 
                 List<String> localLabels = getLabelsFromContext(context);
-                if (userLabels.isEmpty() || localLabels.isEmpty()) {
-                        // If either the labels in the context are empty or the user
-                        // has no labels then we are not able to enter surrogate mode.
-                        return userLabels;
-                }
+                if (localLabels.isEmpty()) {
+                        // If there are no labels on this Frame return null;
+                        return null;
+                } 
                 
                 ArrayList<String> intersection = new ArrayList<String>(userLabels);