Details

    • Type: Sub-task Sub-task
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 8.0-M1
    • Fix Version/s: 8.0-M1
    • Component/s: render
    • Labels:
      None

      Description

      Another case where introducing a base class will help out.

      From aaime's email:

      • GridCoveageLayer and GridReaderLayer really need a GridLayer
        base class, right now to recognize if a layer is raster one has
        to make two instanceof, two casts, and if the code relying on the
        ability to extract a feature source out of the layer is hard to
        change, two casts to call the toFeatureCollection() method that
        both expose
      • GridReaderLayer constuctors force one to specify the title to
        pass also the parameters. Title might not be always needed,
        but some readers will not give the best without custom param
        settings. Please add a constructor wtih reader, style and params

      I am taking a chance to call this RasterLayer as we find people look for the word "raster" when trying to place an image on a map for the first time.

        Issue Links

          Activity

          Hide
          Jody Garnett added a comment -
          Committed as of -r37381
          Show
          Jody Garnett added a comment - Committed as of -r37381
          Hide
          Jody Garnett added a comment -
          Andrea with the GEOT-3565 we may be able to remove this interface? What do you think?
          Show
          Jody Garnett added a comment - Andrea with the GEOT-3565 we may be able to remove this interface? What do you think?

            People

            • Assignee:
              Jody Garnett
              Reporter:
              Jody Garnett
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: