GeoTools

Remove type morphing code from Filters class

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 2.5-M0
  • Fix Version/s: 2.7.6
  • Component/s: None
  • Labels:
    None

Description

The FIlters utility class makes use of a few interesting tricks in order to approximate the filter specification. Specifically:

  • Filters.asInt( Expression )
  • Filters.asString( Expression )
  • FIlters.asType( Expression, Class )

These implementations play a range of tricks; from checking if the Expression is a Literal, a Function with Literal Parameters. Currently the code has a "// TODO use the new converters stuff".

Now is the time.

Issue Links

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. UDIG-1374 Inaccurate value for the line width in Change Style dialog

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jody Garnett added a comment -
Initial attempt is fine; testing shows that Color converting code contained in the Filters class has not been made available to the Converter subsystem yet (gasp!).
Show
Jody Garnett added a comment - Initial attempt is fine; testing shows that Color converting code contained in the Filters class has not been made available to the Converter subsystem yet (gasp!).
Hide
Permalink
Jody Garnett added a comment -
The NumericConverter could not handle the case from "3.0" to 3; added some additional checks allowing various intergral types to just be created from the integral (ie "3") component of the above.
Show
Jody Garnett added a comment - The NumericConverter could not handle the case from "3.0" to 3; added some additional checks allowing various intergral types to just be created from the integral (ie "3") component of the above.

People

  • Assignee:
    Unassigned
    Reporter:
    Jody Garnett
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: