Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-74346

MessageListeners that attempt to re-register (rather than update) will log a warning

    Details

      Description

      Changes made in LPS-74127 caused MessageListeners that attempted to re-register rather than update to log a warning.

      This is correct, but now any *MessageListeners that register during @Modified after a configuration change need to instead use the update method on the SchedulerEngineHelper.

      Otherwise, they will log warnings when trying to re-register.

      Files that need updating:
      modules/apps/twitter/twitter-service/src/main/java/com/liferay/twitter/internal/model/listener/SynchronizeTwitterMessageListener.java
      modules/apps/collaboration/blogs/blogs-web/src/main/java/com/liferay/blogs/web/internal/messaging/CheckEntryMessageListener.java
      modules/apps/collaboration/blogs/blogs-web/src/main/java/com/liferay/blogs/web/internal/messaging/LinkbackMessageListener.java
      modules/apps/collaboration/document-library/document-library-web/src/main/java/com/liferay/document/library/web/internal/messaging/TempFileEntriesMessageListener.java
      modules/apps/collaboration/subscription/subscription-web/src/main/java/com/liferay/subscription/web/internal/messaging/DeleteExpiredTicketsMessageListener.java
      modules/apps/collaboration/message-boards/message-boards-web/src/main/java/com/liferay/message/boards/web/internal/messaging/ExpireBanMessageListener.java
      modules/apps/collaboration/recent-documents/recent-documents-web/src/main/java/com/liferay/recent/documents/web/internal/messaging/RecentDocumentsMessageListener.java
      modules/apps/foundation/portal/portal-pop-notifications/src/main/java/com/liferay/portal/pop/notifications/internal/messaging/POPNotificationsMessageListener.java
      modules/apps/foundation/portal-security/portal-security-ldap/src/main/java/com/liferay/portal/security/ldap/internal/messaging/UserImportMessageListener.java
      modules/apps/foundation/server/server-admin-web/src/main/java/com/liferay/server/admin/web/internal/messaging/PluginRepositoriesMessageListener.java

      It's possible that there are more, we should check all MessageListeners just in case.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                brian.chan Brian Chan
                Reporter:
                drew.brokke Drew Brokke
                Participants of an Issue:
                Recent user:
                Jason Pince
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Days since last comment:
                  1 year, 30 weeks, 4 days ago