Pluto
  1. Pluto
  2. PLUTO-448

No way to know if expiration cache value was set via PortletDD

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.4
    • Fix Version/s: 1.1.5, 2.0.0
    • Component/s: descriptor
    • Labels:
      None

      Description

      The PortletDD object defaults the expirationCache property to 0. With this value there is no way to know if the expiration-cache element was specified in the portlet descriptor which is needed when implementing support for the expiration cache request property.

      Creating a EXPIRATION_CACHE_UNSET constant set to Integer.MIN_VALUE and initializing the expirationCache property in the PortletDD object with the constant provides a means for implementors to determine if the expiration-cache element was specified.

      1. pluto_cache.patch
        7 kB
        Eric Dalquist

        Activity

        Eric Dalquist created issue -
        Hide
        Eric Dalquist added a comment -

        Patch including test case for expiration cache property in PortletDD

        Show
        Eric Dalquist added a comment - Patch including test case for expiration cache property in PortletDD
        Eric Dalquist made changes -
        Field Original Value New Value
        Attachment pluto_cache.patch [ 12370555 ]
        Elliot Metsger made changes -
        Fix Version/s 1.2.0 [ 12312213 ]
        Fix Version/s 1.1.5 [ 12312652 ]
        Elliot Metsger made changes -
        Assignee Elliot Metsger [ emetsger ]
        Hide
        Elliot Metsger added a comment -

        Fix committed to 1.1.x branch r599698.

        Show
        Elliot Metsger added a comment - Fix committed to 1.1.x branch r599698.
        Elliot Metsger made changes -
        Fix Version/s 1.2.0 [ 12312213 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Elliot Metsger made changes -
        Fix Version/s 1.2.0 [ 12312213 ]
        Hide
        Elliot Metsger added a comment -

        Pending application to trunk.

        Show
        Elliot Metsger added a comment - Pending application to trunk.
        Elliot Metsger made changes -
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        Hide
        Craig Doremus added a comment - - edited

        Added EXPIRATION_CACHE_UNSET expiration cache default in PortletDD to 1.1-286-trunk-merge branch in SVN rev 604630. Unit test was not added as it compiled with errors due to changes to that class in creating JSR-286 version of it..

        Show
        Craig Doremus added a comment - - edited Added EXPIRATION_CACHE_UNSET expiration cache default in PortletDD to 1.1-286-trunk-merge branch in SVN rev 604630. Unit test was not added as it compiled with errors due to changes to that class in creating JSR-286 version of it..
        Elliot Metsger made changes -
        Fix Version/s 2.0.0 [ 12312914 ]
        Fix Version/s 1.2.0 [ 12312213 ]
        Hide
        Elliot Metsger added a comment -

        Updated the test and committed it to trunk. Note that this patch really isn't needed in trunk since an ExpirationCacheDD has been added in 2.0. PortletDD.getExpirationCache() has been replaced by PortletDD.getExpirationCacheDD() and the method returns null if the portlet didn't specify an expiration cache.

        Show
        Elliot Metsger added a comment - Updated the test and committed it to trunk. Note that this patch really isn't needed in trunk since an ExpirationCacheDD has been added in 2.0. PortletDD.getExpirationCache() has been replaced by PortletDD.getExpirationCacheDD() and the method returns null if the portlet didn't specify an expiration cache.
        Elliot Metsger made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Workflow jira [ 12418187 ] Default workflow, editable Closed status [ 12565033 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12565033 ] jira [ 12585918 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        12h 53m 1 Elliot Metsger 30/Nov/07 05:33
        Resolved Resolved Reopened Reopened
        8h 47m 1 Elliot Metsger 30/Nov/07 14:21
        Reopened Reopened Resolved Resolved
        92d 13h 8m 1 Elliot Metsger 02/Mar/08 03:29

          People

          • Assignee:
            Elliot Metsger
            Reporter:
            Eric Dalquist
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development