Skip to content

SAK-49440 msgcntr modify the mark as read behaviour to mark as not read #14331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
susowrc wants to merge 7 commits into
base: master
Choose a base branch
from
+555 −261
Open

SAK-49440 msgcntr modify the mark as read behaviour to mark as not read #14331

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
72dc7f3
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 20, 2026
e5bfbae
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 20, 2026
e5a754b
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 21, 2026
6fcbee9
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 21, 2026
cb7480e
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 21, 2026
3ececcb
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Jan 21, 2026
6d6a90e
SAK-49440 Modify the default behaviour of the 'mark as read' in the f…
susowrc Feb 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions lessonbuilder/tool/src/java/org/sakaiproject/lessonbuildertool/service/ForumEntity.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -495,7 +495,7 @@ private void setMasks() {
noneMask.put(PermissionLevel.CHANGE_SETTINGS,Boolean.valueOf(false));
noneMask.put(PermissionLevel.POST_TO_GRADEBOOK, Boolean.valueOf(false));
noneMask.put(PermissionLevel.READ, Boolean.valueOf(false));
noneMask.put(PermissionLevel.MARK_AS_READ,Boolean.valueOf(false));
noneMask.put(PermissionLevel.MARK_AS_NOT_READ,Boolean.valueOf(false));
noneMask.put(PermissionLevel.MODERATE_POSTINGS, Boolean.valueOf(false));
noneMask.put(PermissionLevel.IDENTIFY_ANON_AUTHORS, Boolean.valueOf(false));
noneMask.put(PermissionLevel.DELETE_OWN, Boolean.valueOf(false));
Expand All @@ -513,7 +513,7 @@ private void setMasks() {
contributorMask.put(PermissionLevel.CHANGE_SETTINGS,Boolean.valueOf(false));
contributorMask.put(PermissionLevel.POST_TO_GRADEBOOK, Boolean.valueOf(false));
contributorMask.put(PermissionLevel.READ, Boolean.valueOf(true));
contributorMask.put(PermissionLevel.MARK_AS_READ,Boolean.valueOf(true));
contributorMask.put(PermissionLevel.MARK_AS_NOT_READ,Boolean.valueOf(true));
contributorMask.put(PermissionLevel.MODERATE_POSTINGS, Boolean.valueOf(false));
contributorMask.put(PermissionLevel.IDENTIFY_ANON_AUTHORS, Boolean.valueOf(false));
contributorMask.put(PermissionLevel.DELETE_OWN, Boolean.valueOf(false));
Expand All @@ -531,7 +531,7 @@ private void setMasks() {
ownerMask.put(PermissionLevel.CHANGE_SETTINGS,Boolean.valueOf(true));
ownerMask.put(PermissionLevel.POST_TO_GRADEBOOK, Boolean.valueOf(true));
ownerMask.put(PermissionLevel.READ, Boolean.valueOf(true));
ownerMask.put(PermissionLevel.MARK_AS_READ,Boolean.valueOf(true));
ownerMask.put(PermissionLevel.MARK_AS_NOT_READ,Boolean.valueOf(true));
ownerMask.put(PermissionLevel.MODERATE_POSTINGS, Boolean.valueOf(true));
ownerMask.put(PermissionLevel.IDENTIFY_ANON_AUTHORS, Boolean.valueOf(false));
ownerMask.put(PermissionLevel.DELETE_OWN, Boolean.valueOf(false));
Expand Down
10 changes: 5 additions & 5 deletions ...geforums-api/src/bundle/org/sakaiproject/api/app/messagecenter/bundle/Messages.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ cdfm_toolbar_separator = |
cdfm_moderator_comment_text = Comment By
cdfm_moderator_comment_text_anon = Comment By Moderator
cdfm_moderate = Moderate
cdfm_mark_as_read=Mark as Read
cdfm_mark_as_not_read=Mark as Not Read
cdfm_readby= - Read by:
stat_list = Statistics & Grading
stat_list_student = Statistics
Expand Down Expand Up @@ -184,7 +184,7 @@ cdfm_mark_check_as_read=Mark Read
cdfm_mark_check_as_unread=Mark Unread
cdfm_mark_check_as_delete=Delete
cdfm_mark_check_move_to_folder=Move
cdfm_mark_all_as_read=Mark All as Read
cdfm_mark_all_as_not_read=Mark All as Not Read
cdfm_reply=Reply
cdfm_print=Print
cdfm_printer_friendly=Printer Friendly Format
Expand Down Expand Up @@ -304,7 +304,7 @@ cdfm_failed_create_topic=Failed to create new topic
cdfm_failed_rend_message=Failed Rending Messages
cdfm_view_under_construct=This view is under contruction
cdfm_lost_association=Lost association with current topic
cdfm_no_message_mark_read=No message selected to mark as read. Please select a message.
cdfm_no_message_mark_no_read=No message selected to mark as not read. Please select a message.
cdfm_grade_successful=Grade submission successful.
cdfm_grade_greater_than_zero=Please input a number greater than or equal to 0.
cdfm_grade_decimal_warn=Please input number with 2 or fewer digits after decimal point.
Expand Down Expand Up @@ -415,7 +415,7 @@ perm_revise_any=Edit Any
perm_revise_own=Edit Own
perm_delete_any=Delete Any
perm_delete_own=Delete Own
perm_mark_as_read=Mark as Read
perm_mark_as_not_read=Mark as Not Read
perm_choose_assignment_head=Grading
perm_choose_assignment=Gradebook item:
perm_choose_assignment_none_f=You can associate a grade from the Gradebook to this forum, but there are no existing Gradebook items.
Expand Down Expand Up @@ -657,7 +657,7 @@ pvt_missing_body = You must write your message before you can send it.
pvt_missing_body_draft =You must write your message before you can save it.
pvt_confirm_msg_delete = Are you sure you want to delete this message? If yes, click Delete to delete the message.
pvt_enter_search_text = Please enter text for search.
pvt_no_message_mark_read=No message selected to mark as read. Please select a message.
pvt_no_message_mark_no_read=No message selected to mark as not read. Please select a message.
pvt_no_message_mark_delete=No message selected for deletion. Please select a message.
pvt_no_message_mark_move=No message selected to move to another folder. Please select a message.
pvt_deleted_success=The message(s) you selected have been successfully moved to the Deleted folder.
Expand Down
10 changes: 5 additions & 5 deletions ...orums-api/src/bundle/org/sakaiproject/api/app/messagecenter/bundle/Messages_es.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ cdfm_toolbar_separator=|
cdfm_moderator_comment_text=Comentario de
cdfm_moderator_comment_text_anon=Comentario del moderador
cdfm_moderate=Moderar
cdfm_mark_as_read=Marcar como le\u00eddo
cdfm_mark_as_not_read=Marcar como no le\u00eddo
cdfm_readby=- Le\u00eddo por\:
stat_list=Estad\u00edsticas y calificaci\u00f3n
stat_list_student=Estad\u00edsticas
Expand Down Expand Up @@ -184,7 +184,7 @@ cdfm_mark_check_as_read=Marcar le\u00eddos
cdfm_mark_check_as_unread=Marcar no le\u00eddos
cdfm_mark_check_as_delete=Eliminar
cdfm_mark_check_move_to_folder=Mover
cdfm_mark_all_as_read=Marcar todos como le\u00eddos
cdfm_mark_all_as_not_read=Marcar todos como no le\u00eddos
cdfm_reply=Responder
cdfm_print=Imprimir
cdfm_printer_friendly=Versi\u00f3n imprimible
Expand Down Expand Up @@ -302,7 +302,7 @@ cdfm_failed_create_topic=Error al crear un nuevo tema
cdfm_failed_rend_message=Error al generar mensajes
cdfm_view_under_construct=Esta vista est\u00e1 en construcci\u00f3n
cdfm_lost_association=Se ha perdido la asociaci\u00f3n con el tema actual
cdfm_no_message_mark_read=No se ha seleccionado ning\u00fan mensaje para marcarlo como le\u00eddo. Seleccione un mensaje.
cdfm_no_message_mark_no_read=No se ha seleccionado ning\u00fan mensaje para marcarlo como no le\u00eddo. Seleccione un mensaje.
cdfm_grade_successful=La calificaci\u00f3n se ha enviado correctamente.
cdfm_grade_greater_than_zero=Introduzca un n\u00famero superior o igual a 0.
cdfm_grade_decimal_warn=Introduzca un n\u00famero con 2 o menos d\u00edgitos despu\u00e9s de la coma decimal.
Expand Down Expand Up @@ -411,7 +411,7 @@ perm_revise_any=Editar cualquiera
perm_revise_own=Editar propios
perm_delete_any=Eliminar cualquiera
perm_delete_own=Eliminar propios
perm_mark_as_read=Marcar como le\u00eddo
perm_mark_as_not_read=Marcar como no le\u00eddo
perm_choose_assignment_head=Calificaci\u00f3n
perm_choose_assignment=\u00cdtem de calificaci\u00f3n\:
perm_choose_assignment_none_f=Puede asociar un \u00edtem de Calificaciones con este foro, sin embargo, no hay todav\u00eda \u00edtems creados.
Expand Down Expand Up @@ -653,7 +653,7 @@ pvt_missing_body=Debe escribir el mensaje antes de enviarlo.
pvt_missing_body_draft=Debe escribir el mensaje antes de guardarlo.
pvt_confirm_msg_delete=\u00bfEst\u00e1 seguro que desea eliminar este mensaje? En caso afirmativo, haga clic en Eliminar para eliminar el mensaje.
pvt_enter_search_text=Introduzca el texto de b\u00fasqueda.
pvt_no_message_mark_read=No se ha seleccionado ning\u00fan mensaje para marcarlo como le\u00eddo. Seleccione un mensaje.
pvt_no_message_mark_no_read=No se ha seleccionado ning\u00fan mensaje para marcarlo como no le\u00eddo. Seleccione un mensaje.
pvt_no_message_mark_delete=No hay mensajes seleccionados para eliminar. Seleccione un mensaje.
pvt_no_message_mark_move=No hay mensajes seleccionados para mover a otra carpeta. Seleccione un mensaje.
pvt_deleted_success=Los mensajes seleccionados se han movido correctamente a la carpeta de mensajes Eliminados.
Expand Down
4 changes: 2 additions & 2 deletions ...ntr/messageforums-api/src/java/org/sakaiproject/api/app/messageforums/BulkPermission.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ public class BulkPermission {
private boolean changeSettings;
private boolean deleteAny;
private boolean deleteOwn;
private boolean markAsRead;
private boolean markAsNotRead;
private boolean moderatePostings;
private boolean movePostings;
private boolean newResponse;
Expand All @@ -35,7 +35,7 @@ public class BulkPermission {
private boolean reviseOwn;

public void setAllPermissions(boolean toTrueOrFalse) {
changeSettings = deleteAny = deleteOwn = markAsRead = moderatePostings = movePostings = newTopic
changeSettings = deleteAny = deleteOwn = markAsNotRead = moderatePostings = movePostings = newTopic
= newResponse = newResponseToResponse = postToGradebook = read = reviseAny = reviseOwn
= toTrueOrFalse;
}
Expand Down
4 changes: 2 additions & 2 deletions ...forums-api/src/java/org/sakaiproject/api/app/messageforums/DefaultPermissionsManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ public interface DefaultPermissionsManager
public static final String FUNCTION_REVISE_OWN="messagecenter.reviseOwn";
public static final String FUNCTION_DELETE_ANY="messagecenter.deleteAny";
public static final String FUNCTION_DELETE_OWN="messagecenter.deleteOwn";
public static final String FUNCTION_MARK_AS_READ="messagecenter.markAsRead";
public static final String FUNCTION_MARK_AS_NOT_READ="messagecenter.markAsNotRead";

public static final String MESSAGE_FUNCTION_PREFIX="msg.";
public static final String MESSAGE_FUNCTION_EMAIL= MESSAGE_FUNCTION_PREFIX +"emailout";
Expand Down Expand Up @@ -83,6 +83,6 @@ public interface DefaultPermissionsManager

public boolean isDeleteOwn(String role);

public boolean isMarkAsRead(String role);
public boolean isMarkAsNotRead(String role);

}
6 changes: 3 additions & 3 deletions ...rums-api/src/java/org/sakaiproject/api/app/messageforums/MessageForumsMessageManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -98,11 +98,11 @@ String saveMessage(Message message, boolean logEvent, String toolId, String user

public void markMessageApproval(Long messageId, boolean approved);

public void markMessageReadForUser(Long topicId, Long messageId, boolean read);
public void markMessageNotReadForUser(Long topicId, Long messageId, boolean read);

public void markMessageReadForUser(Long topicId, Long messageId, boolean read, String userId);
public void markMessageNotReadForUser(Long topicId, Long messageId, boolean read, String userId);

public void markMessageReadForUser(Long topicId, Long messageId, boolean read, String userId, String context, String toolId);
public void markMessageNotReadForUser(Long topicId, Long messageId, boolean read, String userId, String context, String toolId);

public boolean isMessageReadForUser(Long topicId, Long messageId);

Expand Down
4 changes: 2 additions & 2 deletions ...messageforums-api/src/java/org/sakaiproject/api/app/messageforums/MessagePermissions.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@ public interface MessagePermissions {

public Boolean getReviseOwn();

public Boolean getMarkAsRead();
public Boolean getMarkAsNotRead();

public void setMarkAsRead(Boolean markAsRead);
public void setMarkAsNotRead(Boolean markAsNotRead);

public void setReviseOwn(Boolean reviseOwn);

Expand Down
6 changes: 3 additions & 3 deletions ...tr/messageforums-api/src/java/org/sakaiproject/api/app/messageforums/PermissionLevel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ public interface PermissionLevel extends MutableEntity{
public static final String CHANGE_SETTINGS = "changeSettings";
public static final String POST_TO_GRADEBOOK = "postToGradebook";
public static final String READ = "read";
public static final String MARK_AS_READ = "markAsRead";
public static final String MARK_AS_NOT_READ = "markAsNotRead";
public static final String MODERATE_POSTINGS = "moderatePostings";
public static final String IDENTIFY_ANON_AUTHORS = "identifyAnonAuthors";
public static final String DELETE_OWN = "deleteOwn";
Expand Down Expand Up @@ -58,9 +58,9 @@ public interface PermissionLevel extends MutableEntity{

public void setDeleteOwn(Boolean deleteOwn);

public Boolean getMarkAsRead();
public Boolean getMarkAsNotRead();

public void setMarkAsRead(Boolean markAsRead);
public void setMarkAsNotRead(Boolean markAsNotRead);

public Boolean getModeratePostings();

Expand Down
6 changes: 3 additions & 3 deletions ...forums-api/src/java/org/sakaiproject/api/app/messageforums/ui/DiscussionForumManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -436,15 +436,15 @@ public void saveTopicMessagePermissions(DiscussionTopic topic,
* @param message
* @param readStatus TODO
*/
public void markMessageAs(Message message, boolean readStatus);
public void markMessageAsNoRead(Message message, boolean readStatus);

/**
* Mark the read status for a given message for a given user
* Mark the not read status for a given message for a given user
* @param message
* @param readStatus
* @param userId
*/
public void markMessageReadStatusForUser(Message message, boolean readStatus, String userId);
public void markMessageNotReadStatusForUser(Message message, boolean readStatus, String userId);


/**
Expand Down
2 changes: 1 addition & 1 deletion ...geforums-api/src/java/org/sakaiproject/api/app/messageforums/ui/UIPermissionsManager.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ public interface UIPermissionsManager
* @param topic
* @return
*/
public boolean isMarkAsRead(DiscussionTopic topic, DiscussionForum forum);
public boolean isMarkAsNotRead(DiscussionTopic topic, DiscussionForum forum);

/**
* Returns whether current user has perm to moderate in this situation
Expand Down
Loading
Loading