Uploaded image for project: 'Pluto'
  1. Pluto
  2. PLUTO-519

Better error msg on invalid event payload class

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 2.0.0
    • 2.0.0
    • portal driver
    • None

    Description

      pretty self explanatory, see the patch

      --- 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
      @@ -155,7 +155,9 @@ public class EventProviderImpl implements org.apache.pluto.spi.EventProvider,
      
                              if (value != null && !isValueInstanceOfDefinedClass(qname, value))
                                      throw new IllegalArgumentException(
      -                                               "Payload has not the right class");
      +                                               "Payload class (" +
      +                                       value.getClass().getCanonicalName() +
      +                                       ") does not have right class, check your defined event types in portlet.xml.");
      
                              try {
      

      It would be better to list out the allowed types, but that's a little more code than this, and it'll point the user in the right direction.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ate Ate Douma
            astubbs Antony Stubbs
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 0.5h
                0.5h
                Remaining:
                Remaining Estimate - 0.5h
                0.5h
                Logged:
                Time Spent - Not Specified
                Not Specified

                Slack

                  Issue deployment