ICEpush
  1. ICEpush
  2. PUSH-383

Support the old Notification Provider property names

          Details

          • Type: Improvement Improvement
          • Status: Resolved
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.1
          • Fix Version/s: EE-4.2.0.GA
          • Component/s: Push Library, Push Server
          • Labels:
            None
          • Environment:
            ICEpush

            Description

            The new Cloud Notification Service included in ICEpush as per PUSH-378 uses a new consistent scheme of property names. To make it easier for ICEpush/ICEfaces developers and on our existent documentation, it is likely better to allow for the old property names to be support and to take precedence over the new ones.

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              Jack Van Ooststroom added a comment - - edited

              A couple of things have been included in the fix:

              • The NotificationProviders will look for alternative property names now first before using its own defined property names. (Though it is still recommended using the new property names as defined by each NotificationProvider.)
              • If there is now property value associated with the alternative property name the NotificationProviders will use their own defined property names and possibly their own defined property default values if no property value was associated with their own defined property name.
              • Logging indicates the property name found and used as well as the property value and if the property value used is the default.
              • ICEpush has an added @WebListener AutoSystemConfigurator which will use the AnnotationScanner to look for @ExtendedConfigurator annotated classes in order to perform the auto configuration before signalling the {{NotificationProvider}}s.
              • Both ICEpush and ICEpush-EE have an @ExtendedConfigurator annotated class that ensures the old property names are made known to the new Cloud Notification Service.

              Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - - edited A couple of things have been included in the fix: The NotificationProviders will look for alternative property names now first before using its own defined property names. (Though it is still recommended using the new property names as defined by each NotificationProvider .) If there is now property value associated with the alternative property name the NotificationProviders will use their own defined property names and possibly their own defined property default values if no property value was associated with their own defined property name. Logging indicates the property name found and used as well as the property value and if the property value used is the default. ICEpush has an added @WebListener AutoSystemConfigurator which will use the AnnotationScanner to look for @ExtendedConfigurator annotated classes in order to perform the auto configuration before signalling the {{NotificationProvider}}s. Both ICEpush and ICEpush-EE have an @ExtendedConfigurator annotated class that ensures the old property names are made known to the new Cloud Notification Service. Marking this one as FIXED.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Adding icepush-ee/util/src/main/java/org/icesoft/util/NameValuePair.java
              Transmitting file data .
              Committed revision 48136.

              Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/AbstractNotificationProvider.java
              Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/EmailNotificationProvider.java
              Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/LoggerNotificationProvider.java
              Transmitting file data ...
              Committed revision 48137.

              Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/APNSNotificationProvider.java
              Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/GCMNotificationProvider.java
              Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/TwilioSMSNotificationProvider.java
              Transmitting file data ...
              Committed revision 44533.

              Sending icepush-ee/core/src/main/java/org/icepush/SystemConfigurator.java
              Transmitting file data .
              Committed revision 48138.

              Sending icepush-ee/core-ee/src/main/java/com/icesoft/icepush/SystemConfigurator.java
              Transmitting file data .
              Committed revision 44534.

              Show
              Jack Van Ooststroom added a comment - Adding icepush-ee/util/src/main/java/org/icesoft/util/NameValuePair.java Transmitting file data . Committed revision 48136. Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/AbstractNotificationProvider.java Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/EmailNotificationProvider.java Sending icepush-ee/cloud/src/main/java/org/icesoft/notify/cloud/core/LoggerNotificationProvider.java Transmitting file data ... Committed revision 48137. Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/APNSNotificationProvider.java Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/GCMNotificationProvider.java Sending icepush-ee/cloud-ee/src/main/java/com/icesoft/notify/cloud/core/TwilioSMSNotificationProvider.java Transmitting file data ... Committed revision 44533. Sending icepush-ee/core/src/main/java/org/icepush/SystemConfigurator.java Transmitting file data . Committed revision 48138. Sending icepush-ee/core-ee/src/main/java/com/icesoft/icepush/SystemConfigurator.java Transmitting file data . Committed revision 44534.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Adding icepush-ee/core/src/main/java/org/icepush/AutoSystemConfigurator.java
              Adding icepush-ee/core/src/main/java/org/icepush/Configurator.java
              Adding icepush-ee/core/src/main/java/org/icepush/ExtendedConfigurator.java
              Sending icepush-ee/core/src/main/java/org/icepush/LocalPushGroupManager.java
              Adding icepush-ee/core/src/main/java/org/icepush/SystemConfigurator.java
              Sending icepush-ee/core/src/main/java/org/icepush/servlet/MainServlet.java
              Transmitting file data ......
              Committed revision 48135.

              Adding icepush-ee/core-ee/src/main/java/com/icesoft/icepush/SystemConfigurator.java
              Transmitting file data .
              Committed revision 44529.

              Show
              Jack Van Ooststroom added a comment - Adding icepush-ee/core/src/main/java/org/icepush/AutoSystemConfigurator.java Adding icepush-ee/core/src/main/java/org/icepush/Configurator.java Adding icepush-ee/core/src/main/java/org/icepush/ExtendedConfigurator.java Sending icepush-ee/core/src/main/java/org/icepush/LocalPushGroupManager.java Adding icepush-ee/core/src/main/java/org/icepush/SystemConfigurator.java Sending icepush-ee/core/src/main/java/org/icepush/servlet/MainServlet.java Transmitting file data ...... Committed revision 48135. Adding icepush-ee/core-ee/src/main/java/com/icesoft/icepush/SystemConfigurator.java Transmitting file data . Committed revision 44529.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending cloud/src/main/java/org/icesoft/notify/cloud/core/AbstractNotificationProvider.java
              Sending cloud/src/main/java/org/icesoft/notify/cloud/core/CloudNotificationService.java
              Sending cloud/src/main/java/org/icesoft/notify/cloud/core/EmailNotificationProvider.java
              Sending cloud/src/main/java/org/icesoft/notify/cloud/core/LoggerNotificationProvider.java
              Transmitting file data ....
              Committed revision 48128.

              Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/APNSNotificationProvider.java
              Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/GCMNotificationProvider.java
              Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/TwilioSMSNotificationProvider.java
              Transmitting file data ...
              Committed revision 44528.

              Sending core/src/main/java/org/icepush/LocalPushGroupManager.java
              Sending core/src/main/java/org/icepush/servlet/ICEpushServlet.java
              Transmitting file data ..
              Committed revision 48129.

              Show
              Jack Van Ooststroom added a comment - Sending cloud/src/main/java/org/icesoft/notify/cloud/core/AbstractNotificationProvider.java Sending cloud/src/main/java/org/icesoft/notify/cloud/core/CloudNotificationService.java Sending cloud/src/main/java/org/icesoft/notify/cloud/core/EmailNotificationProvider.java Sending cloud/src/main/java/org/icesoft/notify/cloud/core/LoggerNotificationProvider.java Transmitting file data .... Committed revision 48128. Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/APNSNotificationProvider.java Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/GCMNotificationProvider.java Sending cloud-ee/src/main/java/com/icesoft/notify/cloud/core/TwilioSMSNotificationProvider.java Transmitting file data ... Committed revision 44528. Sending core/src/main/java/org/icepush/LocalPushGroupManager.java Sending core/src/main/java/org/icepush/servlet/ICEpushServlet.java Transmitting file data .. Committed revision 48129.

                People

                • Assignee:
                  Jack Van Ooststroom
                  Reporter:
                  Jack Van Ooststroom
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: