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

Better error msg on invalid event payload class

    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

        1. Error-msg.patch
          0.9 kB
          Antony Stubbs

        Activity

          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