Changeset 1504 for trunk/src/org/expeditee/gui/FrameUtils.java

Timestamp:

01/29/20 13:20:24 (4 years ago)

Author:

bnemhaus

Message:

Revised implementation of authenticated Expeditee mail. Motivated by bugs relating to messages not being marked as read and incorrect counting of new messages for users, the Expeditee mail system has been rewritten. The new code not only does not exhibit the previous bugs but is also better engineered. Whilst the MailBay is static (which is in line with the MessageBay), the Mail class is no longer static and must be initialised for each user as they log in.

File:

• trunk/src/org/expeditee/gui/FrameUtils.java (modified) (1 diff)

trunk/src/org/expeditee/gui/FrameUtils.java

@@ -1286 +1286 @@
 
                         // check the link to the message/mail frame
-                        Item linkItem = DisplayController.isMailMode() ? MailBay.getMailLink() : MessageBay.getMessageLink();
-                        if (linkItem != null && linkItem.contains(new Point(x, y))) {
-                                linkItem.setOverlayPermission(UserAppliedPermission.copy);
-                                possibles.add(linkItem);
+                        Item[] linkItems = DisplayController.isMailMode() ? new Item[] { MailBay.getMailLink(), MailBay.getCheckMailAction() } : new Item[] { MessageBay.getMessageLink() };
+                        for (Item linkItem: linkItems) {
+                                if (linkItem != null && linkItem.contains(new Point(x, y))) {
+                                        linkItem.setOverlayPermission(UserAppliedPermission.copy);
+                                        possibles.add(linkItem);
+                                }
                         }