Pluto
  1. Pluto
  2. PLUTO-521

add warning when trying to publish an event when event not registered

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Incomplete
    • Affects Version/s: None
    • Fix Version/s: 2.0.0
    • Component/s: portal driver
    • Labels:
      None

      Description

      diff --git a/pluto-portal-driver-impl/src/main/java/org/apache/pluto/driver/services/container/EventProviderImpl.java b/pluto-portal-driver-
      index 2f5669b..5d435dc 100644
      --- a/pluto-portal-driver-impl/src/main/java/org/apache/pluto/driver/services/container/EventProviderImpl.java
      +++ b/pluto-portal-driver-impl/src/main/java/org/apache/pluto/driver/services/container/EventProviderImpl.java
      @@ -194,6 +196,11 @@ public class EventProviderImpl implements org.apache.pluto.spi.EventProvider,
                                      LOG.warn(e);
                              }
                      }
      +               else
      +               {
      +                       LOG.warn("Context not declared as publishing event \"" + qname +
      +                               "\", so not publishing.");
      +               }
              }
      
              /**
      

        Activity

        Hide
        Ate Douma added a comment -

        I didn't commit any no.
        If you come up with a new patch, just reopen this issue and attach it and directly assign it to me then.

        Show
        Ate Douma added a comment - I didn't commit any no. If you come up with a new patch, just reopen this issue and attach it and directly assign it to me then.
        Hide
        Antony Stubbs added a comment -

        Did you commit anything?

        Sorry for not supplying a real patch Ate, I intended for it to be inserted manually, not patched in anyway.

        Do you need more context? I could try to dig something up.... It's been a while though.

        Show
        Antony Stubbs added a comment - Did you commit anything? Sorry for not supplying a real patch Ate, I intended for it to be inserted manually, not patched in anyway. Do you need more context? I could try to dig something up.... It's been a while though.
        Hide
        Ate Douma added a comment -

        Sorry Antony,

        I cannot match your "patch" to the code: I really need a more "context aware" patch based on svn, not git.

        Show
        Ate Douma added a comment - Sorry Antony, I cannot match your "patch" to the code: I really need a more "context aware" patch based on svn, not git.

          People

          • Assignee:
            Unassigned
            Reporter:
            Antony Stubbs
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development