Uploaded image for project: 'ShrinkWrap'
  1. ShrinkWrap
  2. SHRINKWRAP-232

Return asset removed by delete method instead of a boolean

    XMLWordPrintable

Details

    • Feature Request
    • Resolution: Done
    • Major
    • 1.0.0-alpha-12
    • 1.0.0-alpha-11
    • api
    • None
    • Documentation (Ref Guide, User Guide, etc.), Compatibility/Configuration
    • Low

    Description

      The Archive#delete(ArchivePath) operation should return the Asset (or Node) being removed (at the specified ArchivePath) rather than returning boolean. This both avoids having to have an extra call to get the asset before removing it, and it's consistent with the familiar behavior of the JDK Map API.

      // Removes the Asset in the archive at the specified Path and returns the removed Asset
      Node delete(ArchivePath path);

      The only thing I'm not sure about is whether the Asset return type is sufficient for when you delete a directory path.

      Attachments

        Activity

          People

            arubinge@redhat.com Andrew Rubinger (Inactive)
            dan.j.allen Dan Allen (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: